sheerockoff / bitrix-elastic-indexer Goto Github PK

Нужно разобраться с Code Climate Reporter в after_script. Из за него возвращается passed даже если тесты провалены.

Документация: https://dev.1c-bitrix.ru/api_help/iblock/classes/ciblockelement/getlist.php

Для некоторых текстовых свойств Битрикс как значение передаётся массив.

"PROPERTY_DESCRIPTION" => [
    "TEXT" => "Some text",
    "TYPE" => "HTML"
]

Нормализация таких значений происходит с ошибками:

1. Создаётся массив значений, как для множественного значения.
2. Попытка сохранить такое значение в свойство типа text приводит к исключению Elasticsearch\Common\Exceptions\BadRequest400Exception.

Документация: https://dev.1c-bitrix.ru/api_help/iblock/classes/ciblockelement/getpropertyvalues.php

Документация: https://dev.1c-bitrix.ru/api_help/iblock/classes/ciblockelement/getlist.php

Реализовать базовые агрегации для получения количества уникальных, минимального и максимального значений.

Документация: https://www.elastic.co/guide/en/elasticsearch/reference/7.x/search-aggregations.html

Документация: https://dev.1c-bitrix.ru/api_help/iblock/classes/ciblockelement/getlist.php

Из-за низкой точности float создаёт проблемы даже для целых чисел. Нужно что-то придумать.

В Битрикс false используется для обозначения незаполненных свойств, в том числе, при фильтрации в GetList и присвоении значений, например, в SetPropertyValuesEx.

В ElasticSearch строгая типизация. И пустые значения сейчас приводятся к 0.0. И в реальных примерах это может изменить бизнес-логику. Например, в Битрикс "Стоимость доставки" равная 0.0 это "бесплатно", а равная false это "нет доставки". А в ElasticSearch все пустые значения приведутся к 0.0 и появятся товары с "бесплатной доставкой", которых быть не должно.

Для float можно попробовать использовать null вместо битриксового false.