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.
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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: totap commentedthx FrancescoUK
- dependencies[] = multiform (1.0-beta2)
+ dependencies[] = multiform
It works :)
Comment #4
jlhs CreditAttribution: jlhs commentedThankyou Francesco
Comment #5
pjek CreditAttribution: pjek commentedmy website got stuck too, just did #3 manually and it is working again
Comment #6
kalman.hosszu CreditAttribution: kalman.hosszu commentedI think the best way to hack is:
Comment #7
FrancescoUK CreditAttribution: FrancescoUK commentedYep, probably a better and safer work around.
Comment #8
StewartNightingale CreditAttribution: StewartNightingale commentedWorked a treat, Thank you
Comment #9
dimitriseng CreditAttribution: dimitriseng commentedThis is a simple change and is working as expected, so setting to needs review.
Comment #10
jcaprio CreditAttribution: 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 CreditAttribution: kalman.hosszu commentedI think we should change the status to RTBC...
Comment #12
scorchio CreditAttribution: scorchio commentedDefinitely. This should have been done earlier.
Comment #13
Balbo CreditAttribution: Balbo commentedDo you need a patch to fix this? Or it has already been committed?
Comment #14
WebJohn CreditAttribution: WebJohn commentedIt has not been committed in beta7. Dear maintainers - please commit it!
Comment #15
webadpro CreditAttribution: webadpro commentedYes please commit since it is a must to use this module now.
Comment #16
smartinm CreditAttribution: smartinm commentedThis is a temporary workaround without hacking the .info file:
Comment #17
webadpro CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: kalman.hosszu commentedI think the next release will contain this fix.
Comment #21
lsolesen CreditAttribution: lsolesen commentedSetting this to fixed, as it has been committed to dev.
Comment #23
ahillio CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: webadpro commentedPlease Create beta 8 simply to create a new release with this fix.
Comment #28
nopulse CreditAttribution: nopulse commentedI realize that this topic is a bit old but this solution worked like a charm! Thanks.
Comment #29
lsolesen CreditAttribution: 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 CreditAttribution: 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?