First this will fix the bugs: in #405234: Wrong function name called and missing argument.

Second: I don't like to extend the node-form, I rather prefer to extend the functionality of menu module ... at least from the users point of view. For me it seems to be more intuitive to do the menu-stuff in just one place.

Therefor I extended the node-form > "menu settings"-field-group:
The menu item will be also deleted if you delete the "Menu link title". If the "Menu link title" is left/set to blank and the "Parent item" was selected then the current node will be added to the "node trail". This means the current node behaves like a child of the parent item. If the node will be shown then the breadcrumb and the menu trail will set as expected.

The module weight is now set in nodetrail.install (there seems to be no update-funtion necessary, because the weight was already set in the hook_enable)

I've also created a README.txt It would be nice if you also can update the module Page, because it's a very useful module, but it was hard to find (maybe thats why it has such few users).

You can find all that stuff in the patch file: nodetrail.menu_settings.patch

Last point: I guess you are very busy ... at least you weren't be able to answer/fix the two pending issues. I don't want to see this module dying (I prefer to see this functionality in core ;) ). Thats why I offer you to take over maintaining this module (it will be my first one, so I'll find time in busy days). What do you think?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

osopolar’s picture

wondering that nobody answers ... I forgot the patch.

davyvdb’s picture

Assigned: Unassigned » davyvdb
Status: Needs review » Closed (duplicate)

Thanks for the patch? But can you split this up in multiple issues with one feature/bug per issue? This will make this more transparent.

osopolar’s picture

Title: Some improvements and bug fixes » Integrate nodetrails into the menu settings section (provided by the core menu module)
Status: Closed (duplicate) » Needs review
FileSize
8.31 KB

As I can see you have already fixed the two issues so here comes the feature request only (see above description).

fuerst’s picture

Status: Needs review » Reviewed & tested by the community

The patch works as advertised.

marrch_caat’s picture

Davy, thanks a lot for this so-much-needed module, simple and efficient, and Davy, thank you for making this module even better! I'm really surprised no one did that before you two. THANK YOU, GUYS!

marrch_caat’s picture

FileSize
9.57 KB

The module with the provided patch (from comment #3) merged in works fine, with one small but terrible exception. If I open node add/edit page, select a parent menu item but leave menu link title field blank, and press "Preview" button - then parent menu item is cleared. Is there any way to fix that behavior? I attach the merged module files here for the case you'd like to check if the patch was applied properly. Please, help me!!

osopolar’s picture

I want to bring this up again for usability reasons, because imho it's better to have one single point of change than two different fieldsets.

We should also think about #545052: Allow to assign content to a certain menu trail without menu link (a.k.a. Node Trails in core)