I'm getting these errors in manage display for all content types when the masonry_formatter is enabled:

Notice: Undefined index: field types in field_ui_formatter_options() (line 1505 of ~/modules/field_ui/field_ui.admin.inc).
Warning: Invalid argument supplied for foreach() in field_ui_formatter_options() (line 1505 of ~/modules/field_ui/field_ui.admin.inc).

When I disable masonry_formatter, these errors go away.

Thanks for a great module!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BWPanda’s picture

Version: 7.x-1.0 » 7.x-1.x-dev
Status: Active » Needs review
FileSize
753 bytes

Thanks for the bug report!

It took me a while, but I finally worked out that I was adding settings for the colorbox formatter when it wasn't necessarily installed. This is what was causing the errors.

The attached patch should fix this. Please test and let me know if it does so I can commit it.

BWPanda’s picture

Version: 7.x-1.x-dev » 7.x-1.0

Just realised I didn't need to change the version...

BWPanda’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

controla’s picture

Version: 7.x-1.0 » 7.x-1.x-dev
Status: Closed (fixed) » Active

This same patch, modified for "fancybox" solves this issue with fancybox module.
This solution should be treated in a more generic way to avoid creating special cases for each helper, shouldn't it?

BWPanda’s picture

Status: Active » Closed (fixed)

Probably, patches welcome!

Please open a new issue for either Fancybox integration, or a more generic solution. This specific bug report is fixed.