When you run a conversion on multiple nodes, say with VBO, the static variables $node_body and $node_teaser in node_convert_field_convert aren't reset for each node. This means that each node converted has it's content, plus the content of the nodes that were converted before it. I'm submitting a patch that checks the $nid argument against a static $_nid in node_convert_field_convert, and sets a $reset flag if they don't match. If the $reset flag is set, $node_body and $node_teaser are cleared and loaded with fresh data from the current node later in the function. This patch is against the current 6.x-1.x branch.

CommentFileSizeAuthor
node_convert-static_vars-20110425.patch1.22 KBmdm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

katrien_w’s picture

Issue summary: View changes

I experience the same problem with nody body being cumulative filled with content of all previous nodes when bulk converting.
Can somebody please make this patch apply to the latest version? Many thanks.

ziomizar’s picture

Hi katrien_w,

This patch is dated 2011 against 6.x version that now is obsolete and unsupported.
Have you experienced this problem in a 7.x version?