Баг: Некорректный формат файла при добавлении нескольких конверсий с валютой и суммой

[likont]

Наглядно, при использовании:

$metrikaConversionUpload->addConversion('133591247640966458', 'GOAL1', '1481718166', '66', 'RUB'); // Добавяем конверсию
$metrikaConversionUpload->addConversion('579124169844706072', 'GOAL3', '1481718116', '678.90', 'RUB'); // Добавяем ещё конверсию
/* ... и далее добавляем необходимое количество конверсий ... */

Будет результирующий файл:

ClientId,Target,DateTime,Price,Currency,Price,Currency
133591247640966458,GOAL1,1481718166,66,RUB,66,RUB
579124169844706072,GOAL3,1481718116,678.90,RUB,678.90,RUB

Дублируется столбец Price и Currency, в результате Метрика возвращает ошибку:

{"errors":[{"error_type":"invalid_uploading","message":"Incorrect data format, error in line number 1"}],"code":400,"message":"Incorrect data format, error in line number 1"}

Причина кроется в \Meiji\YandexMetrikaOffline\Scope\Upload::addConversion, и методе который он вызывает при добавлении цены/валюты \Meiji\YandexMetrikaOffline\ValueObject\ConversionHeader::addUsesColumn — не проверяет то, есть ли уже такие столбцы.

Решение №1: Проверка перед добавлением:

if (in_array($name, self::$availableColumns) && !in_array($name, $this->usesColumns)) {
    $this->usesColumns[] = $name;
}

Решение №2: Удаление дубликатов после добавления:

if (in_array($name, self::$availableColumns)) {
    $this->usesColumns[] = $name;
    $this->usesColumns = array_unique($this->usesColumns);
}