From #932502-130: Changing allowed values in "List" fields :

install a version of core before #111/#126 went in (e.g., RC3), then add a "List (numeric)" field to a content type, and set allowed values to an empty string. Then update to 7.0. 

Warning: array_combine(): Both parameters should have at least 1 element in list_update_7001()

Only happened for list fields with an empty list of allowed_values (e.g because the field uses allowed_values_function).

Mostly harmless error, and most d7 sites for which list_update_7001() was relevant almost certainly ran it by now. But since #1022924: Updates are broken for deleted fields is about to have list_update_7001() rerun for a couple fields, we might as well fix this.

CommentFileSizeAuthor
list_update_warnings.patch755 bytesyched
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bfroehle’s picture

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

Yes, a simple fix to a harmless warning --- there's no point recalculating every key-value pair of $allowed_values if it is empty!

Also, it'd be nice to get this one in before #1022924-19: Updates are broken for deleted fields, as @yched points out.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.