Updated to the recommended 6.x-1.3 version and got the following warnings:

  • Warning: array_merge(): Argument #2 is not an array in update_do_one() (line 173 of /var/www/update.php).
  • Warning: Invalid argument supplied for foreach() in update_results_page() (line 337 of /var/www/update.php).

This error is caused by the countryban_update_1() function failing to return an array according to the hook_update_N() documentation.

The attached patch fixes the error by using update_sql() instead of db_query().

CommentFileSizeAuthor
#3 countryban_update_1.patch387 bytespillarsdotnet
countryban.patch337 bytespillarsdotnet

Comments

jboukes’s picture

jboukes commented
Assigned: Unassigned » jboukes

Thanks!

jboukes’s picture

jboukes commented
Status: Needs review » Fixed

Fixed in 1.4 - thank you for the notice ;)

pillarsdotnet’s picture

pillarsdotnet commented
Status: Fixed » Reviewed & tested by the community
StatusFileSize
new countryban_update_1.patch387 bytes

Bah. Should have wrapped the result in array().