Incorrect hook_field_widget_info_alter() breaks existing/new unrelated fields. [#1529282]

This took me far too long to track down. Finally solved it with grep -R -A 10.

When the exif module is installed, it overwrites the options_select and options_buttons widget field types with incorrect values. The result is the two option widgets are no longer available for other fields. I experienced the issue with entityreference, but many other modules will have the same problem.

This entire hook must be removed.

/**
 * Implements hook_field_widget_info_alter().
 */
function exif_field_widget_info_alter(&$info) {
  $info['options_select']['field types'] = array('text', 'text_long', 'taxonomy_term_reference','date', 'datetime', 'datestamp');
  $info['options_buttons']['field types'] = array('text', 'text_long', 'taxonomy_term_reference','date', 'datetime', 'datestamp');
}

Comments

Comment #1

Pierre_G CreditAttribution: Pierre_G commented 7 June 2012 at 13:36

Hi
I'm coming here from Entity Ref module indeed.
http://drupal.org/node/1616070

Is there any news about this ?

Comment #2

rbruhn CreditAttribution: rbruhn commented 7 June 2012 at 16:08

Came here from the same issue experienced by Pierre.
Rather than remove, couldn't you simply fix it like this:

function exif_field_widget_info_alter(&$info) {
  $info['options_select']['field types'][] = array('text', 'text_long', 'taxonomy_term_reference','date', 'datetime', 'datestamp');
  $info['options_buttons']['field types'][] = array('text', 'text_long', 'taxonomy_term_reference','date', 'datetime', 'datestamp');
}

Comment #3

raphael_alla CreditAttribution: raphael_alla commented 1 July 2012 at 05:00

This bugs affects me too. Please fix ASAP

Comment #4

jphautin CreditAttribution: jphautin commented 2 July 2012 at 20:05

Assigned:

Unassigned

» jphautin

Comment #5

aasarava CreditAttribution: aasarava commented 19 July 2012 at 21:29

What is the point of altering the options_select and options_buttons widgets just to add back in the standard types? Seems like it could be removed completely, no? Otherwise, if there's some reason it's needed, an array_merge() would probably do the trick to prevent overwriting existing values.