Create/Edit form -- default the empty menu to a group menu

Open on Drupal.org →

Created on 17 April 2025, 5 months ago

Problem/Motivation

When creating group content via the special path or editing via node/nid/edit, nodes without a menu link (both core menu item content and group menu item content) the default menu parent link is set to the main menu (or whatever the defaults for the default-selected and available menu are.

I propose that we add a configurable option to allow menus to default to a group menu instead of a main menu.

There have been far too many times where my group content editors have added something to the main menu when they have access to both group and a wider range of menus on the site.

So when content is being created / edited in a group context I'd like to propose that we update the Node Form Alter to allow selection of the first group menu content is associated with.

Steps to reproduce

Create or use page content type and set default menu to main menu.
Create a group content menu type.
Create a group type.
Add the group content menu type to the group as available content.
Add the page content type as available content.
Configure permissions so the editing user has "administer menus and menu items"
Create a group of this type.
Add a node to the group and observe that menu settings default to main menu.

Proposed resolution

This patch will update the default parent item to be the first available group menu.
This functionality would change default (expected) behavior and should be guarded behind a configuration option.

Remaining tasks

With the patch I have here it doesn't include a configuration form, but I propose we add one to allow this functionality for setting a group menu default. Maybe there's a better way to do it?

User interface changes

API changes

Data model changes

✨ Feature request

Status

Active

Version

3.0

Component

Code

Created by

🇺🇸United States dasginganinja Bethlehem, PA

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Comments & Activities

Issue created by @dasginganinja
Comment 5 months ago →
🇺🇸United States dasginganinja Bethlehem, PA
Comment 4 months ago →
🇺🇸United States dasginganinja Bethlehem, PA
Comment 3 months ago →
🇺🇸United States kevin w
Very helpful patch. I had to change `reset(array_keys($group_menus))` into `array_key_first($group_menus)` to avoid the PHP warning "Only variables should be passed by reference".
Comment 2 months ago →
🇺🇸United States dasginganinja Bethlehem, PA
Thank you for your changefix! I can confirm that works for me as well on version 3.x.

I just upgraded a site Group 1 to Group 2.x and have started using GCM V2.x on that site. I needed to create a patch with this functionality that worked for that version of the module, so I used your patch from #4 as the basis.

This is a reroll of #4 against 2.x.
Comment 2 months ago →
🇺🇸United States dasginganinja Bethlehem, PA
Comment 2 months ago →
🇺🇸United States dasginganinja Bethlehem, PA
Setting to needs review.
Comment about 1 month ago →
danherrero
I re-rolled #4 against the 3.0.6 release of Group Content Menu.

This was necessitated by the work done for https://www.drupal.org/node/3515714 → which moved the updateMenuElement function into a NodeFormAlter class.
Comment about 1 month ago →
heddn Nicaragua
Be nice if we could add tests and the toggle for this functionality as mentioned in the IS.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024