Code Review

Идея правильная — single source of truth для валюты и единиц. Реализация в целом хорошая, но есть баги и недочёты.

Баги

1. ms3_minicart.tpl — old_cost потерял валюту

Было:

{$product.old_cost} {'ms3_frontend_currency' | lexicon}

Стало:

{$product.old_cost}

Но old_cost_formatted нигде не генерируется (ни в ms3_cart.php, ни в других сниппетах). Результат: старая стоимость отображается как голое число без валюты. Нужно либо добавить old_cost_formatted в сниппет корзины и использовать его, либо сохранить лексикон для этого поля.

2. ms3_customer_order_details.tpl — old_price без валюты рядом с price_formatted

Строка 68 (в PR):

<div class="text-decoration-line-through text-muted small">{$product.old_price}</div>

old_price — это format::price() (число без валюты: "150"), а рядом price_formatted — число с валютой: "100 ₽". Визуальная несогласованность. Аналогично в ms3_get_order.tpl (строка 72).

3. Docblock discount() в Format.php повреждён

После вставки weightWithUnit() и getWeightUnit() у метода discount() пропала строка описания. Было * Calculate discount percentage, стало пустая строка в docblock.

Проблемы дизайна

4. Нейминг: cost_with_currency vs *_formatted

В OrdersPageService для списка заказов используется cost_with_currency, а во всех остальных местах — *_formatted. В чанке ms3_customer_order_row.tpl — {$cost_with_currency}. Нужно привести к единому паттерну.

5. ms3_order_total.php — дублирование ternary

7 строк подряд с одинаковым паттерном $response['success'] ? $response['data']['...'] ?? 0 : 0. Стоит вынести в переменную:

$data = $response['success'] ? ($response['data'] ?? []) : [];
$total['cost_formatted'] = $ms3->format->price($data['cost'] ?? 0, true);
// ...

Пропущенные места

6. Отсутствующие _formatted плейсхолдеры

Не сгенерированы, но могут понадобиться в кастомных чанках:

• old_price_formatted — есть только в ms3_cart.php, но отсутствует в ms3_get_order.php, OrdersPageService, Notification
• old_cost_formatted — отсутствует везде
• discount_price_formatted, discount_cost_formatted — отсутствуют

Как минимум old_price_formatted стоит добавить во все точки, где генерируется price_formatted, для консистентности.

Мелкое

7. ms3_weight_unit — дефолт kg

Для русскоязычного проекта дефолт kg спорный. Может стоит кг? Или оставить kg как интернациональный вариант и указать в описании настройки.

Резюме

Основное перед мержем:

1. Исправить old_cost в minicart (баг — пропала валюта)
2. Добавить old_price_formatted в ms3_get_order.php, OrdersPageService, Notification или привести чанки к согласованному виду
3. Привести нейминг к единому *_formatted (убрать cost_with_currency)
4. Восстановить docblock discount()

Code Review (повторный)

Все замечания из первого ревью исправлены во втором коммите. PR стал значительно лучше.

Что исправлено

• ms3_minicart.tpl — old_cost_formatted теперь генерируется и используется
• ms3_customer_order_details.tpl и ms3_get_order.tpl — old_price_formatted с корректным условием
• Нейминг унифицирован: cost_with_currency убран, везде *_formatted
• ms3_order_total.php — $data вынесен, ternary не дублируется
• Docblock discount() — сохранён
• Notification.php — batch-загрузка productData для old_price, полный набор _formatted полей

Оставшееся замечание (некритичное, не блокирует мерж)

Семантика cost_formatted в OrdersPageService изменена:

Было: price($cost) → число без валюты ("1 500"), чанк добавлял {'ms3_frontend_currency' | lexicon}
Стало: price($cost, true) → число с валютой ("1 500 ₽")

Это breaking change для кастомных чанков ms3_customer_order_row, которые используют {$cost_formatted} и сами добавляют валюту — получат двойной символ. Стоит упомянуть в CHANGELOG / release notes.

Мелкое

Notification::getOrderProducts() вызывает $this->order->getMany('Products') дважды (сбор ID + основной цикл). Второй вызов вернёт кеш, но можно было итерировать один раз.

LGTM