The following updates returned messages

migrate module

Update #7202
Array

D7 hook_update_N() may return a string -- http://api.drupal.org/api/drupal/modules!system!system.api.php/function/... -- the $ret array of D6 has gone.

Comments

Anonymous’s picture

Anonymous (not verified) commented

Thank you for pointing this out, Joachim.

Mike - I can make a quick patch for this if you'd like. The update hooks I scanned are just doing this:

$ret[] = t('Added lastthroughput column to migrate_status table');
return $ret;

... so converting the return value to a string is really simple.

Stephen Winters’s picture

Stephen Winters commented

Just a note to say that I have the same problem. I just updated the core and all the modules on my website and it returned the same message:

The following updates returned messages
migrate module
Update #7202
Array

I don't know how to do patches. Will it hurt anything to just leave it alone until the next module update? Or are there other solutions?

Best Wishes,
Stephen

joachim’s picture

joachim commented

Rest assured that there are no untoward consequences of this at all. Instead of seeing the correct message for the update, you saw 'Array' instead, that's all.

Anonymous’s picture

Anonymous (not verified) commented

Correct - it doesn't seem serious, but having it addressed could avoid confusion. I'm just not sure what Mike's thoughts on this are yet (ie - how he'd like to handle the flow in the hooks that are currently returning multiple messages because an array used to be allowed).

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Status: Active » Fixed

OK, fixed the updated functions to return strings.

Status: Fixed » Closed (fixed)

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