Just updated to RC9. Unfortunately update.php returned the following:
* warning: array_merge() [function.array-merge]: Argument #2 is not an array in /xxx/xxx/public_html/update.php on line 174.
* warning: Invalid argument supplied for foreach() in /xxx/xxx/public_html/update.php on line 338.

Which resulted in:
The following queries were executed
domain module
Update #6202

* No queries

Running the latest drupal with many modules on the latest dot deb.

Comments

voxpelli’s picture

voxpelli commented

I think this is caused by the update function not returning an array - it should return an empty array.

So everything works - the update function just fails to mention that to Drupal and Drupal therefor thinks it needs to go and throw errors.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

Yes. True. You can safely ignore the error message. The function should be:

function domain_update_6202() {
  $ret = array();
  $options = variable_get('domain_form_elements', array());
  if (isset($options['author'])) {
    unset($options['author']);
  }
  if (isset($options['menu'])) {
    unset($options['menu']);
  }
  variable_set('domain_form_elements', $options);
  return $ret;
}
agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Active » Fixed

Fixed. Looks like we need an rc10. Grrrr.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Title: update.php returned error for line 174 and line 338 when updating to RC9 » Missing $ret in update 6202

Title change for links.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.