If dev-releases are used, the project version gets set to the latest stable release by using branch as version and "trusting the translation server to fall back to the latest stable release for that branch". This does not seem to work for local files though.

Example: admin_theme version 7.x-1.0+4-dev

Missing translation for 1 project:
Administration theme (7.x-1.x). File not found at https://ftp.drupal.org/files/translations/7.x/admin_theme/admin_theme-7.... nor at translations://admin_theme-7.x-1.x.de.po

Ironically, the file is listed under https://ftp.drupal.org/files/translations/7.x/admin_theme/, but not accessible. This part seems to be a server issue. Regarding the local file ... translations://admin_theme-7.x-1.0.de.po is there, but not found by the module.

Proposal: Instead of setting branch as version just strip dev-release information to get proper version info.

Examples from code:
7.x-1.x-dev => 7.x-1.x (no change)
7.x-1.0-alpha1+5-dev => 7.x-1.0-alpha1 instead of 7.x-1.x.
7.33-dev => 7.33 instead of 7.x

Thoughts about this?

CommentFileSizeAuthor
#6 l10n_update-improve_dev_release_fallback-2949027.patch1.54 KBFireHawkX
#2 l10n_update-improve_dev_release_fallback-2949027.patch1.68 KBrudi teschner

Comments

Rudi Teschner created an issue. See original summary.

rudi teschner’s picture

rudi teschner CreditAttribution: rudi teschner commented
StatusFileSize
new l10n_update-improve_dev_release_fallback-2949027.patch1.68 KB
rudi teschner’s picture

rudi teschner CreditAttribution: rudi teschner commented
Issue summary: View changes

Fixed typo in summary

FireHawkX’s picture

FireHawkX CreditAttribution: FireHawkX commented
Version: 7.x-2.2 » 7.x-2.x-dev
Status: Active » Needs review

will be testing this patch out today.

FireHawkX’s picture

FireHawkX CreditAttribution: FireHawkX commented
Status: Needs review » Needs work

patch does not apply, path seems to be wrong and keeps all the directory structure from the author, i'll try to edit and reupload

FireHawkX’s picture

FireHawkX CreditAttribution: FireHawkX commented
Status: Needs work » Needs review
StatusFileSize
new l10n_update-improve_dev_release_fallback-2949027.patch1.54 KB

Here is the updated version that applies cleanly to latest dev 7.x-2.x

Out of my 200+ modules, the only two that didnt work were draggableviews because the version didnt match...

Also, update_extended-7.x-1.1, because the translation are marked with RC1 (release candidate) and it does not seem to be using those...

Overall, great patch and it works fine!

Thanks!