This PR fix the exception thrown when we push translations twice, with existing translations on Loco.

Explanation of the fix:

To translate an "asset" (a translation key in Loco vocabulary), we need its Loco id. But Loco does not allow us to retrieve an id from a key. So, we fetch all ids for the current domain. Then, we build a map with key as key, and id as value. After that, we can intersect the keys of this map and our messages array to get all Loco ids of the currently processed messages.

foreach ($catalogue->all() as $domain => $messages) {
    $keysIdsMap = [];

    foreach ($this->getAssetsIds($domain) as $id) {
        $keysIdsMap[$this->retrieveKeyFromId($id, $domain)] = $id;
    }

    $ids = array_intersect_key($keysIdsMap, $messages);

    $this->translateAssets(array_combine(array_values($ids), array_values($messages)), $locale);
}

Todo:

• Make some real tests to be 100% sure there is no BC
• Add tests to guarantee the fix