In the README.txt, the example suggests including 'bundle argument' => 5,, but I encountered errors and page redirects like:
Notice: Undefined index: in _field_ui_bundle_admin_path() (line 325 of ../modules/field_ui/field_ui.module).

Fortunately, joeysantiago suggested changing it to 'bundle argument' => 4, in #1526508: Allow more types of Fieldable panels panes to be created, which fixed the problem for me. Since this is the only example and necessary for adding additional bundles, this seems like a fairly important fix. (I would have included this patch in that issue, but it wasn't relevant to the original issue.)

CommentFileSizeAuthor
#1 fieldable_panels_panes_notice_undefined_index_in_field_ui_bundle_1762362_1.patch810 bytesdavidneedham

Comments

davidneedham’s picture

davidneedham
he/him
commented
StatusFileSize
new fieldable_panels_panes_notice_undefined_index_in_field_ui_bundle_1762362_1.patch810 bytes

Patch attached for review.

davidneedham’s picture

davidneedham
he/him
commented
Issue summary: View changes

Updated issue summary.

davidneedham’s picture

davidneedham
he/him
commented
Issue summary: View changes

Updated issue summary.

mrfelton’s picture

mrfelton commented

Yes, agreed. Should be 4 not 5.

Andrew Edwards’s picture

Andrew Edwards commented

Works for me.... Thanks.

caschbre’s picture

caschbre commented

After clearing the cache, this fix also worked for me.

betoscopio’s picture

betoscopio
Primary language Spanish
commented

Great, this fix the problem, but why?
I added some fields to these custom panes mostly based on the code of the module panopoly_widgets and I found the same error while editing fields of a pane
Notice: Undefined index: in _field_ui_bundle_admin_path() (line 325 of ../modules/field_ui/field_ui.module).
After this fix, errors has gone.

I think we could improve the documentation, maybe have more than one example with different options or how to add fields to these custom panel panes.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Reviewed & tested by the community

This looks good. Will commit shortly.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x-1.x with http://drupalcode.org/project/fieldable_panels_panes.git/commit/c113ab6. Thanks!

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Updated issue summary.