Issue status update for http://drupal.org/node/16140 Post a follow up: http://drupal.org/project/comments/add/16140 Project: Drupal Version: cvs -Component: menu system +Component: comment.module Category: bug reports Priority: normal -Assigned to: Anonymous +Assigned to: killes@www.drop.org Reported by: lyk0s@drupal.org Updated by: killes@www.drop.org -Status: patch (code needs review) +Status: patch (ready to be committed) Attachment: http://drupal.org/files/issues/comment_16.patch (828 bytes) this is a bug somewhere else. A menu entry defines callback arguments that are not wrapped in an array. Either a contrib module or comment module. Patch attached. killes@www.drop.org Previous comments: ------------------------------------------------------------------------ Tue, 25 Jan 2005 03:25:26 +0000 : lyk0s@drupal.org warning: array_merge() [function.array-merge]: Argument #1 is not an array in /Web/cybertrib.com/..../includes/menu.inc on line 349. (I am using PHP 5 :) ------------------------------------------------------------------------ Thu, 30 Jun 2005 13:43:30 +0000 : townxelliot Attachment: http://drupal.org/files/issues/menu.inc_2.patch (596 bytes) As far as I can tell, this bug is still not fixed in the latest (as of today) CVS version. I've attached a patch for it (diff -Nur, as I couldn't work out what the preferred patch format for Drupal is). I found the error occurred when running under PHP 5 and attempting to rollback to a previous revision of a node. ------------------------------------------------------------------------ Sat, 02 Jul 2005 12:27:28 +0000 : Dries I think we can avoid having to do the array_merge() in this case. Would be nice if you could a slightly cleaner solution. It might be possible.