I've just updated Webform Localisation to alpha2.

It has a dependency on Webform 40.11, but the latest version of Webform is currently 40.12. Therefore I can't complete "run updates" successfully.

I've changed the dependency in the .info file tot 40.12, which seems to work fine. Is that Ok?

Comments

yazzybe created an issue. See original summary.

joseph.olstad’s picture

true, I will make a patch to test 4.12 asap.

joseph.olstad’s picture

Status: Active » Needs review
StatusFileSize
new738 bytes
joseph.olstad’s picture

Version: 7.x-4.0-alpha2 » 7.x-4.x-dev

test against dev

joseph.olstad’s picture

StatusFileSize
new738 bytes
joseph.olstad’s picture

Status: Needs review » Reviewed & tested by the community
joseph.olstad’s picture

Status: Reviewed & tested by the community » Fixed

Hi yazzybe , thanks for reporting. A new release alpha12 depends now on webform 7.x-4.12

Stevel’s picture

Status: Fixed » Needs review
StatusFileSize
new738 bytes

Do we need the exact version number for Webform? Perhaps it would be enough to specify just that it needs to be Webform 4.x, but not the exact minor version?

That way, we don't need to make a new release just because Webform has.

Status: Needs review » Needs work

The last submitted patch, 8: 2600414-no-explicit-minor-version-8.patch, failed testing.

The last submitted patch, 8: 2600414-no-explicit-minor-version-8.patch, failed testing.

The last submitted patch, 8: 2600414-no-explicit-minor-version-8.patch, failed testing.

The last submitted patch, 8: 2600414-no-explicit-minor-version-8.patch, failed testing.

Stevel’s picture

StatusFileSize
new736 bytes

New patch starting from the 7.x-4.x branch instead of the latest commit not in that branch.

Stevel’s picture

Status: Needs work » Needs review

Back to needs review...

joseph.olstad’s picture

actually I think its safer to specify the webform dependency version because 4.13 has not yet been released and we should test it first when it comes out to make sure that it doesn't break compatibility with webform_localization.

4.11 and 4.12 both passed simpletest , but we cannot be 100% confident that 4.13 will not break something that is currently working.

once 4.13 is released we should release 7.x-4.0-alpha13

Stevel’s picture

I understand your point. It makes sense to try protect users from potential upgrading problems. It does not however prevent them from updating Webform to a later version than specified in the info file.

Most (if not all) updates to Webform however will not break this module. As will most updates to other dependencies (which don't have specific version numbers). It also increases maintenance burden for this module.

I believe it is not very common nor good practice for drupal module to specify specific versions of dependencies just because something might go wrong when the dependency is updated.
Think what would happen if all modules did this: then users could only update after ALL modules with that dependency they use release a simultaneous update.

  • Stevel authored 5b724ab on 7.x-4.x
    Issue #2600414 by joseph.olstad, Stevel: Dependency on webform 4.11
    
joseph.olstad’s picture

Status: Needs review » Fixed

Thanks Stevel , I've committed your patch.
4.x is now the webforms dependancy rather than 4.11 or 4.12

  • joseph.olstad committed dc3fe0d on 7.x-4.x
    Issue #2600414 by joseph.olstad: Dependency on webform 4.11
    

Status: Fixed » Closed (fixed)

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