Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#10 | 1418708_9_media_mymodule.patch | 480 bytes | scor |
#5 | 1418708-field-bundle_settings-update.patch | 3.38 KB | Dave Reid |
#2 | media-1418708-2.patch | 2.39 KB | tim.plunkett |
Comments
Comment #1
Dave ReidComment #2
tim.plunkettHow about this?
Comment #3
Dave ReidIt looks like we're missing a critical piece that this update function requires field_update_7002() to run first.
Comment #4
Dave ReidComment #5
Dave ReidRevised patch with media_update_dependencies().
Comment #6
aaron CreditAttribution: aaron commentedokay 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
Comment #7
Dave ReidCommitted #6 to Git.
http://drupalcode.org/project/media.git/commit/34893b0
http://drupalcode.org/project/media.git/commit/97a1c6a
Comment #9
scor CreditAttribution: scor commentedDoes this actually work?
patch untested.
Comment #10
scor CreditAttribution: scor commentedComment #11
Dave ReidLooks like this has already been fixed with #1759170: media_update_7016() needs to run after rules_update_7205()