Need way to delete menu item from node submit form

awolfey - September 22, 2008 - 23:21
Project:Auto Menu
Version:5.x-1.0
Component:User interface
Category:feature request
Priority:normal
Assigned:danielfs
Status:closed
Description

Hi, I like this module so my client won't have to get into the menu admin pages, but there also needs to be an easy way to delete the menu item. When I check Delete this menu item Auto Menu seems to override that setting. Same thing if I delete the menu title and check it.

Am I doing something wrong or is this just a missing feature?

Thanks.

#1

danielfs - September 29, 2008 - 18:44
Assigned to:Anonymous» danielfs
Status:active» postponed (maintainer needs more info)

The purpose of the auto menu is to avoid having nodes without a parent menu. So, either you set the menu manually or auto menu will do it for you.

Could you explain how would you want the module to act in your case?

#2

awolfey - September 30, 2008 - 15:02

Suppose I have a node that I need to unpublish from time to time. I need a simple way to remove that node from the menu while the node is unpublished. Unpublishing won't do it, and auto menu prevents me from deleting the menu item

Maybe automenu could check the status and enable/disable the menu item to match.

For now I'm just having the client change the menu parent to a parent that is never displayed anywhere, but that is an awkward solution.

#3

danielfs - October 1, 2008 - 07:36

Ok. This is easily doable and I can't see any undesirable side effects. I've submitted the changes to the development branch. Try tomorrow's (2008-Oct-02) snapshot and let me know your thoughts.

#4

danielfs - October 10, 2008 - 15:14
Status:postponed (maintainer needs more info)» fixed

This modification is now included in stable release 5.x-1.1. Nobody complained so far. Closing as fixed.

#5

awolfey - October 10, 2008 - 15:15

Thank you. I have not had a chance to try it yet, but I will soon.

#6

danielfs - October 10, 2008 - 15:33

BTW: it's included in the new Drupal6 branch, too.

#7

awolfey - October 23, 2008 - 20:16

I did some very light testing and it works. Thanks again.

#8

Anonymous (not verified) - November 6, 2008 - 20:21
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.