A recent change via 3039431 has broken the FPP UI and needs to be reverted. After this change, the Manage Fields page of new and existing fieldable panels panes says "No fields are present yet." and results in the following error Notice: Undefined index: fields in _field_ui_bundle_admin_path() (line 335 of ../modules/field_ui/field_ui.module).

CommentFileSizeAuthor
#5 panopoly_widgets-fpp-path-fix-3069038-5.patch4.06 KBdsnopek
#3 panopoly_widgets_fpp_path_fix.patch5.6 KBelly

Comments

chrisgross created an issue. See original summary.

chrisgross’s picture

chrisgross commented
Title: Fieldable Panels Panes fiields appear empty in UI » Fieldable Panels Panes fields missing from UI
elly’s picture

elly commented
StatusFileSize
new panopoly_widgets_fpp_path_fix.patch5.6 KB

Thank you so much for posting this. I ran into this same issue, which is still happening in Panopoly 1.71. I got a 404 when adding fields to fieldable panels pane bundles, and when creating new pane bundles after adding fields to other pane bundles, the fpp bundle would be created with all the same fields as were already on the other panes. Pretty strange. I was able to fix it and use the FPP field UI again by applying a patch which reverts the changes in https://www.drupal.org/project/panopoly/issues/3039431. I'm attaching it here in case it's of any use to others.

delacosta456’s picture

delacosta456 commented

hi
@elly i was facing the same issue and you patch worked for me.... and field restored it looks likes the error occur when the locked field "Title" is no more present.

thanks

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Active » Needs review
StatusFileSize
new panopoly_widgets-fpp-path-fix-3069038-5.patch4.06 KB

I've tested the patch, and it does fix the problem for me as well. However, this feels the wrong fix... The paths currently in code (from #3039431: Taking "Manage Fields" paths of content type like "admin/structure/types/manage/article/fields" redirects to wrong non-existing path) are the correct ones for the latest version of FPP, so we should use them.

I think the patch from that issue was just missing a change to the 'bundle argument' from 4 to 3 now that the path has fewer parts. Here's a patch that makes that change, which is working for me!

delacosta456’s picture

delacosta456 commented

hi thanks @dsnopek

Patch looks to be ok for me no issue when/after applying.

For anyone reading if you're version of Panopoly is not the latest version including recent commit, or you are having chunks when using the patch added by @dsnpek in #5 please try first applying the patch here(link mention @at #5) and then apply the patch in this issue

Thanks

  • dsnopek committed 4c10b4d on 7.x-1.x 
    Issue #3069038 by elly, dsnopek, chrisgross, delacosta456: Fieldable...
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Needs review » Fixed

Committing this! Thanks, Everyone :-)

Status: Fixed » Closed (fixed)

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