At this point there are some other modules that may want a chance to edit or act upon submitted translations. This is the case of i18n and l10n_update.

Instead of having these modules replacing the submission callback, which is not very clean and doesn't work well when more than one module does the same, this patch adds some hooks for them to work. Also it reworks the submission functions so they become a reusable API.

One more extra added by this patch is not submitting translations to the server when they have failed the locale saving part. By using these hooks, other modules (like i18n) may take care also that other strings (not default text group) added by them get saved but not submitted to the server.

CommentFileSizeAuthor
l10n_client_translation_hook.patch6.08 KBJose Reyero
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Gábor Hojtsy’s picture

Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Needs review » Needs work

Let's get this rolling for Drupal 6 and 7 again, since the fantastic new user feedback developments from #568028: Inform user about submission status are now invisible for users of l10n_update (and/or l10n_install). Looks like this will need some coordinated changes in the two modules.

Can you roll this again for the 6.x-2.x branch? Thanks!

Gábor Hojtsy’s picture

Issue summary: View changes
Status: Needs work » Closed (outdated)

Drupal 6 is not supported anymore. I believe we have solutions for this on Drupal 7 and Drupal 8 uses an entirely different system.