array-merge() error thrown (issue when rolling back node revisions) [#16140]

Comment	File	Size	Author
#4	comment_16.patch	828 bytes	killes@www.drop.org

#2	menu.inc_2.patch	596 bytes	townxelliot

Comment #1

(not verified) CreditAttribution: commented 8 February 2005 at 04:15

Log in or register to post comments

Comment #2

townxelliot CreditAttribution: townxelliot commented 30 June 2005 at 13:43

Title:

array-merge() error thrown

» array-merge() error thrown (issue when rolling back node revisions)

File	Size
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.

Log in or register to post comments

Comment #3

Dries CreditAttribution: Dries commented 2 July 2005 at 12:27

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.

Log in or register to post comments

Comment #4

killes@www.drop.org CreditAttribution: killes@www.drop.org commented 6 September 2005 at 21:08

Component:	menu system	» comment.module
Assigned:	Unassigned	» killes@www.drop.org
Status:	Needs review	» Reviewed & tested by the community

File	Size
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.