Closed (fixed)
Project:
Update Status
Version:
5.x-2.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
15 Aug 2007 at 11:21 UTC
Updated:
29 Aug 2007 at 16:47 UTC
Jump to comment: Most recent file
The following errors were displayed when updating to 5.x-2.0:
* warning: array_merge() [function.array-merge]: Argument #2 is not an array in /var/www/vhosts/example.com/httpdocs/update.php on line 309.
* warning: Invalid argument supplied for foreach() in /var/www/vhosts/example.com/httpdocs/update.php on line 542.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | patch_162.txt | 572 bytes | webernet |
Comments
Comment #1
aclight commentedI got the same errors in the same situation (upgrading from 5.x-2.0-rc2).
It looks like this is caused by
update_status_update_5202()not returning anything (an array is expected). But I don't think the error message indicates any functional problem with the upgrade.Comment #2
webernet commentedAttached patch should fix the problem.
See: http://drupal.org/node/159757 for a similar issue...
Comment #3
dwwgrrrrrr :(
This is a bug in update.php. But, alas, I guess we need to fix it here for now. Thanks for the patch. Committed to DRUPAL-5--2.
I'd *hate* to make a 5.x-2.1 just for this. Maybe I'll make 5.x-2.0-2 or something, but that seems evil, too. *sigh*
I guess I could use my CVS super-powers and just move the tag on that one file and rebuild the tarball, since it's only been out for 9 hours, not *that* many people must have gotten it yet. ;)
I'll leave this issue open as "active" until I (and merlin, if he's around) decide what to do in terms of the release. Stay tuned.
Comment #4
nancydruWell, the count of people who have gotten it just went up by one...
Comment #5
dwwSee http://drupal.org/node/167610 about fixing this in core.
Comment #6
misterlawrence commentedI'm fairly new to Drupal, so I am not sure if I should be concerned now that I have received these error messages.
So two quick questions:
I am too much of a newbie to apply a patch...
Thanks.
-David
Comment #7
dwwAfter getting some input via IRC, I decided to just move the DRUPAL-5--2-0 tag on this 1 file, rebuild the tarball, and added a note at the bottom of the release notes about it:
I just ensured that the tarball was rsycn'ed all the way to ftp.d.o, so I think everything's cool now.
@misterlawrence: The relevant phrase from that note for you is: "These warnings are harmless, and can be safely ignored.". ;)
Comment #8
(not verified) commented