Hi. I stumbled upon this change after someone updated media_gallery on prod: http://drupalcode.org/project/media_gallery.git/commitdiff/660d8772b728e...

Is there going to be an upgrade path for this any time soon or should I just do it myself? Yes, I realize this is how the dev branch works :-P

Members fund testing for the Drupal project. Drupal Association Learn more


jsagotsky’s picture

Here's an update hook. Should be straightforward. It works against the prod data I've checked so far, but I'd appreciate if anyone could throw me some comments before we run it for real.

Bernsch’s picture

Status: Active » Needs review
jsagotsky’s picture

Slight change to the patch. We ran into some cases where the new field's table already existed, probably due to a feature revert. When that happens, the table is dropped if it's empty and the update proceeds as normal. If it has content, the update skips.

Ideally this should probably merge both tables together instead.

markpavlitski’s picture

Hi, Just saw this issue after posting a patch for this in #1883270: errors after upgrade to media_gallery 2.x - Undefined property: stdClass::$media_gallery_file in media_gallery_node_load() .

It takes a slightly different approach in that it creates the media_gallery_file table first (which doesn't quite match the existing media_gallery_media table) and then migrates the data over.

Moloc’s picture

This issue seems to be fixed in the issue referenced in #4

Status: Fixed » Closed (fixed)

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