Profile Migrate migrates the selection values from profiles into a text field with a widget: options_select. This is not correct, it should be optionswidgets_select.

The profile data is migrated correctly, however since options_select is not a valid widget, this kind of values are hidden in nodes.

This patch fix the problem and also add a missing $ y removes a trailing space.

CommentFileSizeAuthor
pm.patch1.54 KBdagmar

Comments

kewlguy’s picture

kewlguy commented

subsribing

rtackett’s picture

rtackett commented
Status: Needs review » Reviewed & tested by the community

I have verified dagmar's patch does indeed work. I had the same issue when I migrated a select list field.

Please commit.

nhck’s picture

nhck commented

+1

Without the patch Profile migration wouldn't have been possible.

izmeez’s picture

izmeez commented

subscribing

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented

I'm confirming that I ran into this very bug and solved it completely independently of this issue thread with exactly the same code, and came to the issue queue to create a new issue and supply a patch (#874226: select lists dont appear after conversion) when I found this thread.... doh.

igorik’s picture

igorik commented

@jrguitar21

in this patch is not this part of this patch
@@ -238,8 +238,8 @@ function profile_migrate_fields($type_na
case 'selection':
- $field['widget']['module'] = 'text';
+ $field['widget']['module'] = 'optionwidgets';

is it ok (is it not necessary) or it make sense and it could be in included in this patch?

igorik’s picture

igorik commented

these two lines from this patch seems to be the same, is it error in the patch?
- if ($cat_count > 0 && !array_key_exists($type_name, $types)) {
+ if ($cat_count > 0 && !array_key_exists($type_name, $types)) {