Closed (fixed)
Project:
Media Gallery
Version:
7.x-1.0-beta7
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Jan 2012 at 08:33 UTC
Updated:
18 Jul 2012 at 13:41 UTC
FYI I created the 7.x-1.0 version of Multiform. Multiform version is hardcoded (1.0-beta2) in Media Gallery beta7's dependencies array. I cloned the 7.x-1.x branch to create a patch, but the hardcoded data is removed, so please create a beta8 wihout the hardcoded version data!
Cheers
Kálmán
Comments
Comment #1
FrancescoUK commentedHi,
I've my web stuck because of this as I've upgraded Multiform to the latest and it fails to complete the installation.
Could you please provide the patch here?
I'll try to search for the hardcoded string and change it to non beta.
Thanks,
Francesco
Comment #2
FrancescoUK commented!!!This is just hacking!!!
I worked around the issue going to media_gallery.info and removing the "-beta2" from this line: dependencies[] = multiform (1.0-beta2).
Comment #3
totap commentedthx FrancescoUK
- dependencies[] = multiform (1.0-beta2)
+ dependencies[] = multiform
It works :)
Comment #4
jlhs commentedThankyou Francesco
Comment #5
pjek commentedmy website got stuck too, just did #3 manually and it is working again
Comment #6
kalman.hosszu commentedI think the best way to hack is:
Comment #7
FrancescoUK commentedYep, probably a better and safer work around.
Comment #8
StewartNightingale commentedWorked a treat, Thank you
Comment #9
dimitriseng commentedThis is a simple change and is working as expected, so setting to needs review.
Comment #10
jcaprio commentedExcellent fix!
For those who might get confused with the + and - symbols, it simply means delete one line of code and add the other. At first I thought they were both lines of code that both needed to exist along with the + and - symbols.
Thanks again!
Joe
Comment #11
kalman.hosszu commentedI think we should change the status to RTBC...
Comment #12
scorchio commentedDefinitely. This should have been done earlier.
Comment #13
Balbo commentedDo you need a patch to fix this? Or it has already been committed?
Comment #14
WebJohn commentedIt has not been committed in beta7. Dear maintainers - please commit it!
Comment #15
webadpro commentedYes please commit since it is a must to use this module now.
Comment #16
smartinm commentedThis is a temporary workaround without hacking the .info file:
Comment #17
webadpro commentedActually guys this has been done a while ago:
View the commit.
http://drupalcode.org/project/media_gallery.git/commit/a15b003
Looks like the latest changes were not included in the latest package.
Comment #18
smartinm commented@webadpro You are right. Multiform dependency is fixed in development snapshot. Unfortunately this commit is the only change since 5 months ago.
Comment #19
anfederman commentedAfter applying fix I get this:
EntityFieldQueryException: Unknown field: media_gallery_expose_block in EntityFieldQuery->addFieldCondition() (line 767 of ../newdrupal/includes/entity.inc).
Comment #20
kalman.hosszu commentedI think the next release will contain this fix.
Comment #21
lsolesen commentedSetting this to fixed, as it has been committed to dev.
Comment #23
ahillio commentedIs the dev version available? I only see beta7. I tried both changes...
dependencies[] = multiform (>=1.0-beta2)
dependencies[] = multiform
clearing cache both times, but the modules page still lists beta2 as requirement which is still incompatible so I can't enable it.
Should I even be using this module if I don't know how to do this? The directions seemed simple enough. I'd like try this module out and maybe use it on production site... which could tolerate bugs as long as it generally works. Any advice? Thanks!!!
Comment #24
simon fraser commentedbelieve you only need to change the one line in your media_gallery.info file, then it works a treat in any production site (though only once you have run the 'update script' in modules)... I gladly confess I am very grateful indeed to these 'sanity-saviour' folk in forums!
dependencies[] = multiform (>=1.0-beta2)
Comment #25
ahillio commentedMy sites are on a multi-site setup with media modules shared by all sites. Changing the dependency as described above and running update script allows some of the sites to enable Media Gallery. But one site is already using Media and Mediafront and this one cannot enable Media Gallery. I've tried disabling and re-enabling these (though mediafront couldn't be disabled because it's being used by a content field) and then running update again but it didn't help. Any ideas?
Comment #26
ahillio commentedChanging the dependency to dependencies[] = multiform (>=1.0-beta2) definitely works in multisite - but having the module redundantly installed in a sites/specificsite.com/module folder will override the shared module (fellow newbies take note). And grateful for Simon Fraser's reminder about update script!
Comment #27
webadpro commentedPlease Create beta 8 simply to create a new release with this fix.
Comment #28
nopulse commentedI realize that this topic is a bit old but this solution worked like a charm! Thanks.
Comment #29
lsolesen commented@webadpro Too many changes has happened for us to release just now. We need to make sure that it works. You can help by getting your hands dirty with git and test out all the new bugfixes.
Comment #30
webadpro commentedCan't you simply checkout the release version rather than the dev version, and apply the patch on it, and rollout a new beta?