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.

Files: 
CommentFileSizeAuthor
#10 1418708_9_media_mymodule.patch480 bytesscor
#5 1418708-field-bundle_settings-update.patch3.38 KBDave Reid
PASSED: [[SimpleTest]]: [MySQL] 11 pass(es).
[ View ]
#2 media-1418708-2.patch2.39 KBtim.plunkett
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

Comments

Priority:Normal» Major
Issue tags:+D7 stable release blocker, +Needs backport to 1.x

Status:Active» Needs review
StatusFileSize
new2.39 KB
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

How about this?

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

Status:Needs review» Needs work

Status:Needs work» Needs review
StatusFileSize
new3.38 KB
PASSED: [[SimpleTest]]: [MySQL] 11 pass(es).
[ View ]

Revised patch with media_update_dependencies().

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

Status:Reviewed & tested by the community» Fixed

Status:Fixed» Closed (fixed)

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

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.

StatusFileSize
new480 bytes

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