Undefined index notice when adding a shortcut [#1030216]

Comment	File	Size	Author
#6	menu.zip	19.33 KB	jon betts

Comment #1

anrikun commented 21 January 2011 at 10:26

This should not happen as in menu.inc, module index is set before drupal_alter is called :-(
Could you please try to clear your menu cache and see if notice goes away?

Log in or register to post comments

Comment #2

jon betts commented 21 January 2011 at 16:30

Issue is still occurring after going to Performance and clicking on the Clear All Caches button.

Log in or register to post comments

Comment #3

anrikun commented 21 January 2011 at 22:33

Try to disable Menu Firstchild then create a link.
Does the notice show up?

Log in or register to post comments

Comment #4

jon betts commented 22 January 2011 at 16:15

Disabled Menu Firstchild, cleared caches and then created a link and did not receive the message.

Log in or register to post comments

Comment #5

anrikun commented 22 January 2011 at 17:00

Could you please post or send me a dump of your tables:
menu_custom
menu_links

Log in or register to post comments

Comment #6

jon betts commented 22 January 2011 at 17:26

Status	File	Size
new	menu.zip	19.33 KB

Sure. I've attached a zip of a sql dump of both.

Log in or register to post comments

Comment #7

anrikun commented 22 January 2011 at 23:00

Sorry, but I have just realized something:
Your issue is about Menu Firstchild used with the Shortcut module, right?
So that it is fully clear for me, list each step to reproduce the issue.
Also tell me what Core modules you have enabled and what your theme is.

Log in or register to post comments

Comment #8

jon betts commented 23 January 2011 at 14:40

Sure. I'm just testing out D7 to see if I may be able to use it for upcoming projects so at this point I'm just using the defaults, Bartik/Seven as my themes.

To reproduce, just go to any admin page where you can add a link to the shortcuts bar and click on the plus button that appears to the right of the title. The same thing happens in the Edit Shortcuts screen. As far as I know, these are the only ways to add shortcuts through the interface. e.g. admin/modules, add modules to shortcuts.

I went through and deactivated (but did not uninstall) all contrib modules and have the following core modules active:

Block
Contextual links
Dashboard
Database logging
Field
Field SQL storage
Field UI
Filter
Help
List
Locale
Menu
Node
Number
Options
Overlay
Path
Profile
Search
Shortcut
System
Taxonomy
Text
Toolbar
Update manager
User

Hope this helps!

Log in or register to post comments

Comment #9

anrikun commented 23 January 2011 at 15:58

Status:

Active

» Needs review

All right, I was able to reproduce the bug and I have fixed it.
Can you get the last menu_firstchild.module from CVS and review it?

Log in or register to post comments

Comment #10

anrikun commented 23 January 2011 at 16:05

Version:

7.x-1.0-beta1

» 7.x-1.0

Log in or register to post comments

Comment #11

jon betts commented 23 January 2011 at 20:58

Unfortunately I haven't had any luck getting CVS set up on my Mac. I've got CVS set up but haven't found clear (for me) instructions on how to set up specifically for Drupal. I think there is a way to make a patch though on Drupal.org, I'll look into it.

Log in or register to post comments

Comment #12

jon betts commented 23 January 2011 at 21:11

Status:

Needs review

» Reviewed & tested by the community

I took a look at the changes that were made in CVS and they were easy enough for me to make so I did. I tested it out with the stripped down installation above and then I re-enabled a few dozen modules and the message has gone away. Thanks!!