Problem/Motivation
Drupal\tmgmt_acclaro\Plugin\tmgmt\Translator\AcclaroTranslator::importTranslation() does not explicitly set status to translated for received translations. The status change work in most of the cases except when a job item was in TMGMT_DATA_ITEM_STATE_PRELIMINARY state:
* @param int|null $status
* (Optional) The data item status that will be set. Defaults to NULL,
* which means that it will be set to translated unless it was previously
* set to preliminary, then it will keep that state.
* Explicitly pass TMGMT_DATA_ITEM_STATE_TRANSLATED or
* TMGMT_DATA_ITEM_STATE_PRELIMINARY to set it to that value.
* Other statuses are not supported.
Proposed resolution
Explicitly set TMGMT_DATA_ITEM_STATE_TRANSLATED status when importing remote translation.
Remaining tasks
User interface changes
API changes
Data model changes
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | explicitly_set-2892931-2.patch | 803 bytes | mbovan |
Comments
Comment #2
mbovan commentedImplemented suggested solution.
Comment #4
berdirMakes sense, committed.