Fix migrate hooks for D6 filefield [#1230398]

- formatter names changed
- some settings moved from the widget to the instance

Attached patch fixes this.

Comment	File	Size	Author
#5	file_hotfix-1230398.patch	865 bytes	yched
#2	migrate_filefield_fixes.patch	2.71 KB	yched
#1	cck_migrate_exclude-1226678.patch	2.88 KB	yched
	cck_migrate_exclude-1226678.patch	2.88 KB	yched

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

yched CreditAttribution: yched commented 26 July 2011 at 20:49

File	Size
cck_migrate_exclude-1226678.patch	2.88 KB

Also, change formatter's module.

Comment #2

yched CreditAttribution: yched commented 26 July 2011 at 20:50

File	Size
migrate_filefield_fixes.patch	2.71 KB

Gah, totally picking the wrong patch file :-)

Comment #3

yched CreditAttribution: yched commented 26 July 2011 at 20:53

Status:

Active

» Fixed

Committed

Comment #4

andypost

he/him

Russian

CreditAttribution: andypost commented 30 July 2011 at 00:33

Status:

Fixed

» Reviewed & tested by the community

+++ b/modules/content_migrate/modules/content_migrate.filefield.inc
@@ -30,6 +30,11 @@ function content_migrate_filefield_field_alter(&$field_value, $instance_value) {
+    // Some settings have changed names.
+    unset($field_value['settings']['list_field']);
+    $field_value['settings']['display_field'] = $field_value['settings']['list_field'];
+    $field_value['settings']['display_default'] = $field_value['settings']['list_default'];
+    unset($field_value['settings']['list_default']);

Suppose this should be in different order for first 2 lines:

$field_value['settings']['display_field'] = $field_value['settings']['list_field'];
unset($field_value['settings']['list_field']);

$field_value['settings']['display_default'] = $field_value['settings']['list_default'];
unset($field_value['settings']['list_default']);

EDIT: Unset should be done after transferring data