[drupal-devel] [bug] Menu.inc array_merge error

Thox drupal-devel at drupal.org
Fri Sep 16 08:13:01 UTC 2005

Issue status update for 
Post a follow up: 

 Project:      Drupal
 Version:      cvs
 Component:    menu system
 Category:     bug reports
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  Thox
 Updated by:   Thox
 Status:       patch (ready to be committed)

Yep, +1 on ax's patch. I'm glad /someone/ understands the menu system ;)


Previous comments:

Thu, 15 Sep 2005 14:38:53 +0000 : Thox

Attachment: http://drupal.org/files/issues/menu.inc_3.patch (801 bytes)

On a PHP 5 system, enabling comments and visiting a comment adding
screen causes an array_merge error for me. The patch included prevents
this from happening on my system.


Thu, 15 Sep 2005 14:51:43 +0000 : m3avrck

Confirmed error on PHP5 here. Patch does indeed fix problem, which not
only creates the warning, but prevents rest of page from showing since
output occured before the headers were sent.


Thu, 15 Sep 2005 14:59:57 +0000 : killes at www.drop.org

I think this error was caused by a callback argument in comment.module
that was a string. this has been fixed a few days ago. So I think we
won't need this patch. Can somebody confirm this?


Thu, 15 Sep 2005 15:06:35 +0000 : killes at www.drop.org

Ok, I've been convinced that this is nto the same error. In fact, there
is no callback arguement set and that is why it errors out. Please


Thu, 15 Sep 2005 17:44:26 +0000 : ax

Attachment: http://drupal.org/files/issues/menu.inc_4.patch (902 bytes)

an alternative patch that goes to the root of the problem (i think) and
prevents that empty array wrongly to be set at the first place. there
is no 'callback arguments' key in the 'comment/reply' menu item, so it
shoudn't be set (to NULL) neither (to not fool later array_key_exists()

please consider this instead.


Thu, 15 Sep 2005 17:50:54 +0000 : ax

to be completely sure, there probably should be similar isset() checks
around both of

$_menu['items'][$mid]['access'] = $item['access'];
$_menu['items'][$mid]['callback'] = $item['callback'];

as both keys aren't required in menu items (according to


Thu, 15 Sep 2005 18:00:14 +0000 : moshe weitzman

ax's brief patch looks good to me. a few days ago, i tried to track this
down for a while but failed. nice work thox and ax.


Thu, 15 Sep 2005 18:32:21 +0000 : m3avrck

Tested and confirmed, patch works on PHP5, great find ax!


Fri, 16 Sep 2005 06:56:20 +0000 : killes at www.drop.org

Correct, ax' patch is the better one, eliminating the root cause.

More information about the drupal-devel mailing list