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.When enabling the module with media 7.x-2.x it throws the above error and its also impossible to remove.
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | media_archive-1327270-11.patch | 98.19 KB | Rob C |
| #9 | media_archive-1327270-9.patch | 96.98 KB | ryan.gibson |
| #1 | 1327270-media_archive-class-loading.patch | 3.59 KB | snufkin |
Comments
Comment #1
snufkin CreditAttribution: snufkin commentedPatch moves the class out of the module code into a separate file, handling the loading via the .info file.
Comment #2
rwilson0429 CreditAttribution: rwilson0429 commentedThis patch worked for me.
Comment #3
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedworked for me too but I did get a warning
git apply 13*patch
warning: media_archive.info has type 100644, expected 100755
Comment #4
snufkin CreditAttribution: snufkin commentedyeah the module files should not be executable, but unfortunately they are. that would be a separate issue though.
Comment #5
Rob C CreditAttribution: Rob C commentedI took the patch a bit further. It still needs work, but a live demo and patch are available here.
This fixes a lot of issues, file permissions, reworked the whole module to be more in line with the other media modules, still needs a lot of work, but at least it's working on media 1.x and 2.x.
Comment #6
ryan.gibson CreditAttribution: ryan.gibson commentedThe patch from #1 let me enable the module without it breaking the site.
Comment #7
ryan.gibson CreditAttribution: ryan.gibson commentedActually scratch that, it didn't work. @Rob which version of the module did you write your patch for?
Comment #8
Rob C CreditAttribution: Rob C commented@ryanissamson
Current media_archive git 7.x-1.x.
The patch on my site is updated every hour, i implemented HTML5 player yesterday, it's not finished, but it should apply to 7.x-1.x without a problem. I'm working on that site almost every day. (at least i try to do a bit of work every day) And it's all GPLv2 on my site, so take it all for a spin if you like :)
All modules that you see enabled on my site should be media 1.x and 2.x compatible. (at least at this moment, this will change.)
Comment #9
ryan.gibson CreditAttribution: ryan.gibson commentedThanks @Rob for the information. That patch didn't apply with a normal git apply so I rerolled it against the latest 7.x-1.x version and am attaching it.
Comment #10
Rob C CreditAttribution: Rob C commented@ryanissamson
Looking good, might add some more bits later on, but at least we'r getting somewhere :)
Comment #11
Rob C CreditAttribution: Rob C commentedThis still needs work, has the html5 player / flash fallback implemented as a test, should apply clean and basic functions should just work for videos. It 'should' work with media 1.x and 2.x. (at least, i expect it does at this stage)
Comment #12
kreynen CreditAttribution: kreynen commentedThese changes have been committed. I've rolled a beta2 release since beta1 wasn't usable. Please use the current version for future patches and open new issues. I'm using this module again so I should be able to test and commit more quickly.
Comment #13
Rob C CreditAttribution: Rob C commentedOk great! I am currently working on re-implementing the old style embed and a check that's also in 6.x-2.x, see
http://drupalcode.org/project/media_archive.git/blob/refs/heads/6.x-2.x:...
and related code. Currently it does not work for all videos, i'll hope to have a patch ready by the end of next week that deals with this.