In #1211008: Split field_bundle_settings out per bundle the horrible field_bundle_settings variable is being split up per entity bundle and the function update_media_7016 needs to be updated accordingly.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Priority: Normal » Major
Issue tags: +D7 stable release blocker, +needs backport to 1.x
tim.plunkett’s picture

Status: Active » Needs review
FileSize
2.39 KB

How about this?

Dave Reid’s picture

It looks like we're missing a critical piece that this update function requires field_update_7002() to run first.

Dave Reid’s picture

Status: Needs review » Needs work
Dave Reid’s picture

Status: Needs work » Needs review
FileSize
3.38 KB

Revised patch with media_update_dependencies().

aaron’s picture

Status: Needs review » Reviewed & tested by the community

okay this works. I had to do some sleuthing to get it, here are the steps that I used:

git clone --recursive --branch 7.x http://git.drupal.org/project/drupal.git
cd drupal
git checkout 7.12
cd sites/all/modules
git clone --branch 7.x-1.x http://git.drupal.org/project/media.git
git clone --branch 7.x-1.x http://git.drupal.org/project/styles.git
cd media
git checkout 3ae599fe9f2cdb3780b20ab64c8fb1bb6bc541dd

* install the site, and enable the media module (after moving 3 include files, because that specific commit was broken, and I didn't feel like going back further)

git stash
git checkout 7.x-2.x
git apply -v 1418708-field-bundle_settings-update.patch
cd ..
git clone --branch 7.x-2.x http://git.drupal.org/project/file_entity.git
git clone --branch 7.x-1.x http://git.drupal.org/project/ctools.git
git clone --branch 7.x-3.x http://git.drupal.org/project/views.git
cd ../../../
git checkout 7.14

* run all updates

Dave Reid’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

scor’s picture

Status: Closed (fixed) » Needs work
+++ b/media.install
@@ -190,6 +190,20 @@ function media_uninstall() {
+  $dependencies['mymodule'][7016] = array(

Does this actually work?

patch untested.

scor’s picture

FileSize
480 bytes
Dave Reid’s picture

Status: Needs work » Closed (fixed)
Issue tags: -needs backport to 1.x