Error: Function name must be a string in modules/field_ui/field_ui.admin.inc on line 154

Looks like commit http://drupalcode.org/project/field_tools.git/commit/9406c11 broke the Manage Fields page for certain content types. I'm not really sure of the criteria for this error occurring; it happens for one content type and not another. Maybe it has to do with the status of the feature in question? Currently, it's Needs review.

Hmm...if I re-export it so that it becomes Default, then...yep, then Manage Fields works again.

So looks like the Features integration needs to be fine-tuned. Some quick testing shows that Overridden status (existing field changed) is OK. Needs review (existing field added via Field Tools or normal Existing Field form field) breaks things. Simply adding a new field is fine. It just says Not exported.

So it's pretty much the Needs review status. Hope this report helps.

CommentFileSizeAuthor
#3 1944610.field_tools.features-review.patch721 bytesjoachim
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim’s picture

joachim CreditAttribution: joachim commented

Eek, that's a nasty one!

Thanks for reporting. Will investigate when I have time :)

joachim’s picture

joachim CreditAttribution: joachim commented

Looks like the Field UI admin table is expecting weird stuff:

        // Determine the region for the row.
        $function = $row['#region_callback'];
joachim’s picture

joachim CreditAttribution: joachim commented
Status: Active » Needs review
FileSize
1944610.field_tools.features-review.patch721 bytes

I'm not actually managing to get my test Feature to go to a Needs Review state. Any ideas?

Can you try this patch? Just a hunch, but it might fix it.

joachim’s picture

joachim CreditAttribution: joachim commented
Status: Needs review » Fixed

Managed to reproduce it. Fixed.

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
CreditAttribution: wizonesolutions commented

I'm glad, and sorry for not getting around to testing the patch sooner :)

Status: Fixed » Closed (fixed)

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