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!

CommentFileSizeAuthor
#1 masonry_formatter_undefined_index-1856800-1.patch753 bytesAnonymous (not verified)

Comments

Anonymous’s picture

Version: 7.x-1.0 » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new753 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.

Anonymous’s picture

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

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

Anonymous’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?

Anonymous’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.