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

FrancescoUK’s picture

Hi,

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

FrancescoUK’s picture

!!!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).

totap’s picture

thx FrancescoUK

- dependencies[] = multiform (1.0-beta2)
+ dependencies[] = multiform

It works :)

jlhs’s picture

Thankyou Francesco

pjek’s picture

my website got stuck too, just did #3 manually and it is working again

kalman.hosszu’s picture

I think the best way to hack is:

- dependencies[] = multiform (1.0-beta2)
+ dependencies[] = multiform (>=1.0-beta2)
FrancescoUK’s picture

Yep, probably a better and safer work around.

StewartNightingale’s picture

Worked a treat, Thank you

dimitriseng’s picture

Status: Active » Needs review

This is a simple change and is working as expected, so setting to needs review.

jcaprio’s picture

Excellent 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

kalman.hosszu’s picture

I think we should change the status to RTBC...

scorchio’s picture

Status: Needs review » Reviewed & tested by the community

Definitely. This should have been done earlier.

Balbo’s picture

Do you need a patch to fix this? Or it has already been committed?

WebJohn’s picture

It has not been committed in beta7. Dear maintainers - please commit it!

webadpro’s picture

Yes please commit since it is a must to use this module now.

smartinm’s picture

This is a temporary workaround without hacking the .info file:

/**
 * Implements hook_system_info_alter().
 */
function CUSTOM_MODULE_system_info_alter(&$info, $file, $type) {
  if ($type == 'module' && $file->name == 'media_gallery') {
    $key = array_search('multiform (1.0-beta2)', $info['dependencies']);
    if ($key !== FALSE) {
      $info['dependencies'][$key] = 'multiform';
    }
  }
}
webadpro’s picture

Actually 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.

smartinm’s picture

@webadpro You are right. Multiform dependency is fixed in development snapshot. Unfortunately this commit is the only change since 5 months ago.

anfederman’s picture

After applying fix I get this:

EntityFieldQueryException: Unknown field: media_gallery_expose_block in EntityFieldQuery->addFieldCondition() (line 767 of ../newdrupal/includes/entity.inc).

kalman.hosszu’s picture

I think the next release will contain this fix.

lsolesen’s picture

Status: Reviewed & tested by the community » Fixed

Setting this to fixed, as it has been committed to dev.

Status: Fixed » Closed (fixed)

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

ahillio’s picture

Status: Closed (fixed) » Active

Is 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!!!

simon fraser’s picture

believe 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)

ahillio’s picture

My 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?

ahillio’s picture

Status: Active » Fixed

Changing 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!

webadpro’s picture

Please Create beta 8 simply to create a new release with this fix.

nopulse’s picture

I realize that this topic is a bit old but this solution worked like a charm! Thanks.

lsolesen’s picture

@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.

webadpro’s picture

Can't you simply checkout the release version rather than the dev version, and apply the patch on it, and rollout a new beta?

Status: Fixed » Closed (fixed)

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