This bug kills Drupal. When using the "Menu settings" fieldset to link a page into a menu (formerly menu-otf), the page that you are currently editing shows up in the list of available selections. If you choose that one, you end up with mid = pid in the menu table, and viewing the page becomes impossible. Drupal enters an endless loop and Apache eventually sends a timeout. The solution is to deactivate the option in the select so that it cannot be selected, or remove it altogether (less ideal). In any case, it should never be possible to have mid = pid in the menu table (is this something we can enforce at the database level?)

CommentFileSizeAuthor
#2 menu.39440.patch1.02 KBRichard Archer
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Richard Archer’s picture

This is a fapi conversion problem. It is addressed in this patch:
http://drupal.org/node/32665

Could you please have a look at that patch and make sure it fixes this for you?
While you're there, you might as well give it a quick review :)

Thanks!

Richard Archer’s picture

Status: Active » Needs review
FileSize
1.02 KB

Here's a patch that fixes this specific problem.

robertDouglass’s picture

Status: Needs review » Reviewed & tested by the community

Tested: applies, fixes the problem. This is a critical patch... please no RC without it.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)