I have run into an issue where if the $form['menu'] fieldset has been removed or was never in the $form multiple_node_menu will still do a form alter to $form['menu'].

I use Menu Settings per Content Type which can remove the $form['menu'] fieldset completely.
This could also be issue if the code Menu module is disabled. Though that is an assumption.

Im sure there are probably other modules and cases where this might crop up

CommentFileSizeAuthor
#6 multiple_node_menu-843006-6.patch.txt635 byteslee20
#1 multiple_node_menu-843006.patch888 bytesJackinloadup

Comments

Jackinloadup’s picture

Jackinloadup commented
Status: Active » Needs review
StatusFileSize
new multiple_node_menu-843006.patch888 bytes

Here is a proposed patch against HEAD.

PeteS’s picture

PeteS commented

Looks good, committing. Thanks!

Jackinloadup’s picture

Jackinloadup commented
Status: Needs review » Fixed

Marking as fixed

Status: Fixed » Closed (fixed)

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

Melissamcewen’s picture

Melissamcewen commented
Status: Closed (fixed) » Active

It seems to be working for the parent item field in the "add a new menu item" section but not for the current menu link sections
https://skitch.com/melissamcewen/r7rch/roles-and-responsibilities-intern...

I am using the Menu Settings per Content Type Module (ctm)

lee20’s picture

lee20 commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Assigned: Jackinloadup » Unassigned
Status: Active » Needs review
StatusFileSize
new multiple_node_menu-843006-6.patch.txt635 bytes

This logic needs to be ported to the 7.x branch. Attached is a patch that prevents the multiple node menu form from appearing when the content type doesn't have any available menus assigned to it.

kruser’s picture

kruser commented

Thanks for the patch. It worked for me with 7.x-1.0-beta1

lee20’s picture

lee20 commented
Status: Needs review » Reviewed & tested by the community
harrrrrrr’s picture

harrrrrrr commented

tested & works

  • recidive committed 3795f0e on 7.x-1.x authored by lee20 
    Issue #843006: Don't try to alter the form when the menu fieldset is not...
recidive’s picture

recidive commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Patch committed. Thanks!

Status: Fixed » Closed (fixed)

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