Hello,

With Drupal and Drush 8, it is now possible to update translations without l10n_update.

I also think to add a support for drush_language and its options.

Is this feature already planned or can I try to make a patch for that?

Comments

Grimreaper created an issue. See original summary.

Grimreaper’s picture

Category: Bug report » Feature request

Wrong Category.

helmo’s picture

I've not heard of anyone working on that... so go for it :)

Grimreaper’s picture

Status: Active » Needs review
FileSize
4.55 KB

Hello,

Here is a patch that adds this feature.

The only "problem" I am not satisfied with is that I have duplicated the function _hosting_sync_site_has_module() into _hosting_tasks_extra_site_has_module() to avoid a dependency on hosting_sync module.

Thanks for the review.

Grimreaper’s picture

Also, I think to maybe create a separated module to integrate with the drush_language extension.

I imagine a scan of the platform to detect the .po files and then being able to select which po file to import.

Grimreaper’s picture

Project: Hostmaster (Aegir) » Aegir Hosting Tasks Extra

I forgot to change project.

helmo’s picture

Status: Needs review » Needs work

Cool

+1 for moving _hosting_sync_site_has_module to the core hosting_site module so it can be used in both places.

The core_major_version check from #2830557: Support for --entity-updates option on drush updb command might be useful here too... that also sounds like a nice addition for core.

PS: the patch still calls _hosting_sync_site_has_module() ;)

Grimreaper’s picture

Status: Needs work » Needs review
FileSize
4.5 KB

Hello,

Thanks @helmo for catching that. here is a rework of the patch.

I will open an issue and provide a patch for the refactoring of the _hosting_sync_site_has_module() function.

Grimreaper’s picture

Grimreaper’s picture

Hello,

Here is the updated patch as the issue #2832110: Refactoring of _hosting_sync_site_has_module is fixed.

helmo’s picture

Status: Needs review » Reviewed & tested by the community

Thanks, I just did some basic testing... looks OK.

helmo’s picture

Status: Reviewed & tested by the community » Fixed

Committed, thanks.

Grimreaper’s picture

Thanks @helmo.

But I don't see the commit in the repository: http://cgit.drupalcode.org/hosting_tasks_extra/log

Not pushed yet?

  • helmo committed 8472a68 on 7.x-3.x authored by Grimreaper
    Issue #2830562 by Grimreaper: Update translations on managed sites
    
helmo’s picture

I must have been distracted ... it's pushed now.

Grimreaper’s picture

Thanks @helmo.

Status: Fixed » Closed (fixed)

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