Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Catch up the changes made in the following tmgmt issue: https://www.drupal.org/node/2538198
TranslatorPluginInterface won't have the following methods any more:
- public function getRemoteLanguagesMappings(TranslatorInterface $translator);
- public function mapToRemoteLanguage(TranslatorInterface $translator, $language);
- public function mapToLocalLanguage(TranslatorInterface $translator, $language);
Comments
Comment #1
edurenye CreditAttribution: edurenye at MD Systems GmbH commentedIn local this patch passes the tests.
Comment #2
edurenye CreditAttribution: edurenye at MD Systems GmbH commentedNow it maps properly, and passes the tests in local.
There's still problems with the translation, but seems to not be related.
Comment #4
edurenye CreditAttribution: edurenye at MD Systems GmbH commentedFinally I could test it and it works perfectly, was just configuration problem.
Comment #5
edurenye CreditAttribution: edurenye at MD Systems GmbH commentedBetter title.
Comment #6
miro_dietikerHm, remote languages should not be filtered by the (local) source language. This is a different language domain. Instead the source languate should be mapped to remote languages first.
Comment #7
BerdirThe mapping happens in the translator now.
$source_language is already mapped to the remote source language. this is something else. Microsoft translator doesn't have mappings, just a list of languages. If the source languages is one of the supported languages, then we can translate into all the other languages, if not, then we can not translate into any language for this source language.
However, leaving at needs work to improve the comments because what's there now doesn't make much sense.
Comment #8
BerdirAlso, we should rename all $language to $langcode when it is just a language code. But not in this patch.
Comment #9
edurenye CreditAttribution: edurenye at MD Systems GmbH commentedOk, I changed the documentation.
Comment #11
edurenye CreditAttribution: edurenye at MD Systems GmbH commentedComment #13
BerdirCommitted.