Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrded’s picture

Status: Active » Needs review
FileSize
947 bytes

my patch

greggmarshall’s picture

Patch was a couple of lines off on my copy, but manually adding the one line of code in the patch worked for me.

abrao’s picture

This bug is caused by the undefined variable $node in content_migrate.admin.inc (Line 366) and can be fixed by commenting out that line.
// $query->addExpression("'". $node['type'] ."'", 'bundle');

At some point in April (commit 26099e5) a performance patch was applied. The variable $node that was created prior to this line was removed, but they forgot to remove the reference to $node in the above line. So $node is basically not set prior to this.
The bundle will be set correctly on line 390 ($query->addExpression("'". $node['type'] ."'", 'bundle');), once $node is set.

jcisio’s picture

This is a simpler one.

jcisio’s picture

Status: Needs review » Reviewed & tested by the community

RTBC because of #3 (I didn't see it yesterday).

brad.bulger’s picture

will this ever be fixed?

colan’s picture

Version: 7.x-2.x-dev » 7.x-3.x-dev
Component: General » upgrade path
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Thanks! Committed in 9a2cf4c.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.