CCK Link field title empty after migrate

The array merging logic for "Set the correct node field values." looks wrong. It's trying to do:

array(0 => array('url' => 'value')) + array(0 => array('title' => 'value'))


and should probably be doing:

array(0 =>
array('url' => 'value') + array('title' => 'value')
)


Attached patch fixes the Link field bug, but it's so generic there's probably other things affected by this bug and fix.

Also the "Unset the dummy column value." block looks like it should be inside of the columns foreach and accidentally got moved outside. Moved it back in and now it looks like dummy values are unset properly. I'm not very familiar with this code, so someone needs to check that this is how it's supposed to work.

Yup, it's a dup of #678524: problem with CCK fields with more than one column. Just didn't manage to find it before submitting this. I never knew one could write $node->{$field_name}[$delta].

So how about the second part to clean up the dummy column values? It's close, but this patch should apply to 1.0-alpha2 or HEAD.

FYI just checked and patch still applies to dev (offset 5 lines). And those 5 lines are due to the patch in #678524.