fix(ERP-372): синхронизировать фильтры getSalesCountSum с getSalesSum

На дашборде /dashboard/sales для магазинов с большой долей доставки
(например, 07 Аэродромная 28) средний чек расходился с суммой продаж:
avg = sum_count_query / cnt_count_query, при этом колонка «сумма продаж»
показывала результат другого запроса (getSalesSum) с более узким
фильтром.

Привожу getSalesCountSum к тем же условиям:
- LEFT JOIN sales sc ON sc.id = s.sales_check
- sc.order_id IS NULL OR IN ('','0')   — отбрасываем строки, привязанные
  через sales_check к доставочной продаже;
- s.marketplace_order_id IS NULL OR =''  — отбрасываем продажи маркетплейсов.

После фикса avg = sum / cnt совпадает с тем, что выводится в колонке
«сумма продаж».

Тесты (9/9 GREEN): unit на структуру SQL и биндинг параметров +
интеграционные на реальной БД (insert/rollback в транзакции),
проверяющие исключение маркетплейс- и доставочно-связанных продаж.

test(ERP-372): добавить тесты для getSalesCountSum

Unit-тесты на SQL-структуру, биндинг параметров, payType-ветку
и пустой результат. Интеграционный тест на реальной БД проверяет,
что метод исключает маркетплейс-продажи и записи, связанные через
sales_check с доставкой. На текущем (нефиксенном) коде воспроизводят
баг — RED-состояние перед применением правки.

Merge branch 'feature_filippov_task22_widen_sync_window_to_4months' into 'develop'

fix(task_22): расширить окно выборки админов с 1 до 4 месяцев

See merge request gitlab-instance/yii-erp24!1161

fix(task_22): расширить окно выборки админов с 1 до 4 месяцев

Сотрудники с пустым guid выпадали из синхронизации с 1С после месяца —
расширяем окно date_add до 4 месяцев. Добавлена документация по каналу
ERP→1С create_employee (pull через API2 DataController::actionRequest).

Merge branch 'feature_fomichev_ERP-300J_suppliers_dict' into 'develop'

feat(ERP-300): справочник закупщика

See merge request gitlab-instance/yii-erp24!1159

feat: добавить группу 82 (менеджер контроля качества) в RBAC ERP-300

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: подсказка кнопки экспорта — .xlsx → .csv

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: экспорт — убрать sendFile(), отдавать CSV через response->content

sendFile() конфликтовал с nginx на сервере. Теперь CSV генерируется
в php://temp и отдаётся напрямую через FORMAT_RAW без файла на диске.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: экспорт — один JOIN-запрос со скалярами вместо AR + eager loading

Убирает OOM: GUID-ы фильтруются отдельно, данные — одним SQL с
STRING_AGG для маркировок. Никаких ActiveRecord-объектов в памяти.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: экспорт маппинга переведён с XLSX на CSV

PhpSpreadsheet роняет процесс по памяти на больших выборках.
CSV: нет зависимостей, минимум памяти, UTF-8 BOM для Excel.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: заменить yii.getCsrfToken() на meta[csrf-token] во всех модулях buyer-reference

yii объект недоступен в контексте внешних JS-файлов. Читаем токен напрямую
из <meta name="csrf-token"> — так же работает во всех браузерах.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: перехват ошибки экспорта — возврат JSON вместо 500

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: правильный регистр исключённых категорий (строчные буквы)

В БД категории хранятся строчными: 'букет', 'сборка', 'сервис'.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix: фильтрация категорий Букет/Сборка/Сервис после кэша

Перенесена фильтрация исключённых категорий из SQL-запроса внутри кэша
в PHP-фильтр после getOrSet — работает немедленно независимо от состояния кэша.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix cache

fix mapping

fix marking

fix producer

fix plantation

fix plantation

delete

migration

fix button

fix button

fix button

фикс появления после создания

фикс удаления

фикс удаления

фикс вкладки

Фикс по фильтры

Фикс по фильтры

Фикс по js

Фикс по маппингу

Каталог поставщика

Каталог поставщика

feat(ERP-318): справочник поставщиков — Model + Controller + View + JS

Реализован CRUD справочника поставщиков с модальными окнами (AJAX),
GridView + PJAX и каскадной деактивацией связанных записей.

Компоненты:
- Supplier (AR модель с TimestampBehavior + BlameableBehavior)
- SupplierSearch (search модель без пагинации)
- SupplierController (AJAX CRUD, наследует BaseController)
- views/supplier/index.php (GridView + модалка + JS)
- views/supplier/_form.php (форма для модалки)
- BuyerReferenceController + view (контейнер с табами)

Бейджи типа и статуса — по UI-guideline из spec-supplier-reference.html.
Валидация: name unique varchar(200), lead_time >= 0, type/currency enum.
Soft delete через is_active=false с каскадом на markings и product_mappings.
Write-операции обёрнуты в DB-транзакции.

Артефакты workflow: docs/jira/ERP-318/

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Merge branch 'fix_filippov_BC-248_max_chatbot_source' into 'develop'

fix(BC-248): защита миграции purchase_date от повторного применения

See merge request gitlab-instance/yii-erp24!1158

fix(BC-248): защита миграции purchase_date от повторного применения

Проверка наличия колонки через getTableSchema перед ALTER TABLE.

Merge branch 'fix_filippov_BC-248_max_chatbot_source' into 'develop'

feat(BC-248): добавить поле purchase_date в обращения KikFeedback

See merge request gitlab-instance/yii-erp24!1156

feat(BC-248): добавить поле purchase_date в обращения KikFeedback

- Миграция: колонка purchase_date (date, nullable) в kik_feedback_request
- Модель KikFeedbackRequest: PHPDoc, rules, attributeLabels
- API2 KikController: приём purchase_date из bodyParams
- API3 KikService + FeedbackInput: приём purchase_date, маппинг max_chatbot
- View: отображение "Дата покупки" в карточке обращения

Merge branch 'fix_filippov_BC-248_max_chatbot_source' into 'develop'

fix(BC-248): actionGetStores — вернуть UUID из Products1c вместо числового ID из CityStore

See merge request gitlab-instance/yii-erp24!1155

fix(BC-248): actionGetStores — вернуть UUID из Products1c вместо числового ID из CityStore

Max-бот ожидает UUID магазинов (Products1c.id), а не числовой ID из CityStore.
Удалён закомментированный дубликат метода.

Merge branch 'fix_filippov_BC-248_max_chatbot_source' into 'develop'

fix(BC-248): добавить источник max_chatbot в маппинг KikFeedback

See merge request gitlab-instance/yii-erp24!1154

fix(BC-248): добавить источник max_chatbot в маппинг KikFeedback

Обратная связь из Max-бота записывалась с source=0 (неизвестный)
вместо source=11 (Макс), т.к. маппинг не содержал max_chatbot.

Merge branch 'feature_filippov_ERP-280_reduce_cleanup_period' into 'develop'

feat(ERP-280): клиентская валидация суммарного размера файлов (лимит 2.5 ГБ)

See merge request gitlab-instance/yii-erp24!1153

feat(ERP-280): клиентская валидация суммарного размера файлов (лимит 2.5 ГБ)

Предотвращает отправку формы списания, если суммарный размер вложений
превышает 2.5 ГБ. Показывает пользователю фактический и допустимый размер.

Merge branch 'feature_filippov_ERP-280_reduce_cleanup_period' into 'develop'

fix(ERP-280): сократить срок подготовки файлов списания до 5 дней

See merge request gitlab-instance/yii-erp24!1152

fix(ERP-280): сократить срок подготовки файлов списания до 5 дней

Было: -1 month, стало: -5 days в export-old, print-old и связанных методах модели.

Merge branch 'fix_filippov_batch_sync_pdo_binding' into 'develop'

fix: BatchSyncService PDO binding — $N плейсхолдеры → :pN

See merge request gitlab-instance/yii-erp24!1150

fix: BatchSyncService PDO binding — $N плейсхолдеры → :pN

PDO::bindValue() требует индекс ≥ 1, а $params был 0-индексированным
массивом с PostgreSQL-нативными $1,$2,$3 плейсхолдерами. Это вызывало:

  PDOStatement::bindValue(): Argument #1 ($param) must be >= 1

Результат на проде: Balances::deleteAll() удалял данные магазина,
BatchSyncService->upsertBalances() падал → магазин терял ВСЕ строки.

Fix: именованные :p1,:p2,:p3 плейсхолдеры вместо $1,$2,$3.
Проверено на реальной БД: 5 магазинов (419-499 товаров) + characteristics.

Merge branch 'feature_filippov_ERP-33_stock_state_scd2' into 'develop'

Feature filippov erp 33 stock state scd2

See merge request gitlab-instance/yii-erp24!1149

fix(ERP-33): SendTelegramMessageJob namespace app\jobs, не yii_app\jobs

Merge branch 'task-JIRA-ERP-33-20260324070448' into 'develop'

Task jira erp 33 20260324070448

See merge request gitlab-instance/yii-erp24!1148

Merge branch 'develop' into 'task-JIRA-ERP-33-20260324070448'

# Conflicts:
#   erp24/services/StockStateService.php

fix(ERP-33): safe cast components — пустые строки и NULL в products_1c

fix(ERP-33): safe cast components — пустые строки и NULL в products_1c

Merge branch 'feature_filippov_ERP-33_stock_state_scd2' into 'develop'

[ERP-33]: cast components::jsonb + TelegramTarget multiline fix

See merge request gitlab-instance/yii-erp24!1147

fix(ERP-33): cast components::jsonb + TelegramTarget multiline fix

1. StockStateService: pp.components::jsonb cast — products_1c хранит
   components как text, stock_state ожидает JSONB
2. TelegramTarget: экранировать \n при записи в error_tracking.log —
   многострочные ошибки ломали формат файла и спамили stderr

fix(ERP-33): cast pp.components::jsonb в INSERT для совместимости с products_1c

Merge branch 'task-JIRA-ERP-33-20260324070448' into 'develop'

fix(ERP-33): TelegramTarget — экранировать \n в error_tracking.log

See merge request gitlab-instance/yii-erp24!1146

refactor(ERP-33): убрать actionCleanup — очистка не нужна

fix(ERP-33): TelegramTarget — экранировать \n в error_tracking.log

Многострочные ошибки (содержащие $_SERVER dump) ломали формат
файла error_tracking.log: file() разбивала одну запись на
множество строк, каждая выводилась как "Некорректная строка".

Фикс: str_replace \n → \\n при записи, убран error_log спам.

Merge branch 'task-JIRA-ERP-33-20260324070448' into 'develop'

fix(ERP-33): namespace app\ → yii_app\ для совместимости с продом

See merge request gitlab-instance/yii-erp24!1145

fix(ERP-33): namespace app\ → yii_app\ для совместимости с продом

Merge branch 'task-JIRA-ERP-33-20260324070448' into 'develop'

[ERP-33]: add unit tests for StockHistoryService (11 tests)

See merge request gitlab-instance/yii-erp24!1144

feat(ERP-33): заменить StockHistory (snapshot) на StockState (SCD-2)

Полная переделка подхода хранения остатков:
- StockHistory (полные снимки 11400 INSERT/запуск) → StockState (SCD-2, 2 SQL)
- Без партиций — одна таблица с valid_from/valid_to (TIMESTAMP)
- valid_to = '2100-01-01 00:00:00' для активных, закрытие на секунду раньше
- is_active boolean для быстрой фильтрации
- batch_ts — единый timestamp пакета на запуск
- Экономия ~78% объёма (660 MB vs 3 GB за 2 года)
- 10 unit-тестов, 29 assertions

feat(task-JIRA-ERP-33-20260324070448): Сбор данных по остаткам на день

feat(task-JIRA-ERP-33-20260324070448): Сбор данных по остаткам на день

feat(task-JIRA-ERP-33-20260324070448): Сбор данных по остаткам на день

feat(task-JIRA-ERP-33-20260324070448): Сбор данных по остаткам на день

feat(task-JIRA-ERP-33-20260324070448): Сбор данных по остаткам на день

feat(task-JIRA-ERP-33-20260324070448): Сбор данных по остаткам на день

feat(task-JIRA-ERP-33-20260324070448): Сбор данных по остаткам на день

feat(task-JIRA-ERP-33-20260324070448): Сбор данных по остаткам на день

feat(task-JIRA-ERP-33-20260324070448): Сбор данных по остаткам на день

Merge branch 'feature_filippov_ERP-237_missing_indexes' into 'develop'

feat(ERP-237): 6 индексов для устранения медленных запросов

See merge request gitlab-instance/yii-erp24!1143

feat(ERP-237): 6 индексов для устранения медленных запросов из pg_stat_statements

export_import_table(entity, export_id, export_val) — 2.3M запросов, 511s
marketplace_orders(guid) — 247K запросов, 412s
marketplace_flowwow_emails(subject, from, date) — 253K запросов, 409s
marketplace_order_items(order_id) — 376K запросов, 307s
timetable_fact(admin_id, date, is_opening) — 52K запросов, 298s
admin_payroll_days(admin_id, date) — 27K запросов, 278s

Merge branch 'feature_filippov_ERP-218-J_fix_db_long_request' into 'develop'

feat(ERP-218-J): интеграция BatchSyncService и ProductPropTypeCacheService в UploadService

See merge request gitlab-instance/yii-erp24!1142

feat(ERP-218-J): интеграция BatchSyncService и ProductPropTypeCacheService в UploadService

- Исправлена миграция: indexExists() через pg_indexes вместо $schema->indexes
- ProductPropTypeCacheService: null-safe обращение к Yii::$app->cache
- UploadService: Products1cPropType::findOne() заменён на ProductPropTypeCacheService (10M→1 запрос)
- UploadService: balances INSERT в цикле заменён на BatchSyncService::upsertBalances (6M→2k запросов)
- UploadService: characteristics INSERT/UPDATE заменён на BatchSyncService::upsertProductCharacteristics (11M→1.2k)
- 20 unit-тестов: BatchSyncServiceTest (9) + ProductPropTypeCacheServiceTest (11)

Merge branch 'feature_filippov_ERP-255_fix_marketplace_concurrency' into 'develop'

fix(ERP-255): защита от гонки процессов в маркетплейс-командах

See merge request gitlab-instance/yii-erp24!1141

test(ERP-255): unit-тесты для mutex-защиты cron-команд и null-check в SendBonusInfoToSiteJob

MarketplaceControllerMutexTest — 7 тестов: пропуск при занятом mutex,
release в finally, уникальность ключей.
SendBonusInfoToSiteJobTest — 5 тестов: null-check, детерминированность hash.

Merge branch 'feature_filippov_ERP-258_telegram_429_fix' into 'develop'

fix(ERP-258): агрегация сообщений и дедупликация в TelegramTarget

See merge request gitlab-instance/yii-erp24!1139

fix(ERP-258): агрегация сообщений и дедупликация в TelegramTarget

- Обработка всех сообщений буфера вместо только первого
- Включён shouldSendMessage() для дедупликации (1 раз в день на ошибку)
- Лимит 800 символов на сообщение, 4000 итого
- disableNotification только если нет ERROR-уровня

Merge branch 'feature_filippov_ERP-257_heic_already_readable' into 'develop'

fix(ERP-257): graceful fallback для HEIC-файлов, которые уже являются JPEG

See merge request gitlab-instance/yii-erp24!1138

chore: trigger MR creation

fix(ERP-257): graceful fallback для HEIC-файлов, которые уже являются JPEG

iPhone иногда сохраняет JPEG с расширением .HEIC. Библиотека heic2any
бросала ERR_USER вместо того, чтобы пропустить файл. Теперь при ошибке
"already browser readable" файл передаётся как есть с корректным расширением.

Merge branch 'feature_filippov_ERP-256_cabinet_last_month_button' into 'develop'

feat(ERP-256): добавлена кнопка «за прошлый месяц» на странице ошибки смен

See merge request gitlab-instance/yii-erp24!1137

feat(ERP-256): добавлена кнопка «за прошлый месяц» на странице ошибки смен

Добавлен $buttonParams2 с диапазоном дат прошлого месяца.
Кнопка отображается рядом с «за неделю» при ошибке cs1.

fix(ERP-255): защита от гонки процессов в маркетплейс-командах

- Добавлен компонент PgsqlMutex в console.php для advisory locks
- Обёрнуты 4 action MarketplaceController в мьютекс (yandex-orders,
  flowwow-orders, flowwow-retry, check-ready-to-1c) — повторный запуск
  cron пропускается без ошибок
- Добавлен try-catch IntegrityException в processOrders() вокруг
  save() нового заказа — дубликат marketplace_order_id при гонке
  теперь логируется как warning, а не падает в ошибку
- Исправлен NPE в SendBonusInfoToSiteJob: добавлена проверка на null
  перед обращением к ->status++ (Fatal Error при отсутствии записи)
- Добавлен layout => false в api1.config.php

Merge branch 'feature_filippov_ERP-254_fix_close_shift_null_fact' into 'develop'

fix(ERP-254): throw 404 when no open shift found on close

See merge request gitlab-instance/yii-erp24!1135

fix(ERP-254): throw 404 when no open shift found on close

TimetableFactModel::getLast() returns null if no open fact exists
for the admin on the current/previous date. Accessing checkin_start_id
on null caused PHP Warning and 500 response. Now returns proper 404.

Merge branch 'feature_filippov_20260307_fix_mp_telegram_429' into 'develop'

fix(marketplace): aggregate missing MarketplaceStore errors before Telegram send

See merge request gitlab-instance/yii-erp24!1134

fix(marketplace): aggregate missing MarketplaceStore errors before Telegram send

Instead of sending one Telegram message per order with missing store
inside processOrders() loop, collect unique warehouse GUIDs and send
a single aggregated message after the loop to avoid Telegram 429.

Merge branch 'feature_fomichev_2026-03-06_TO8-22_promobonuses' into 'develop'

fix(TO8-22): is_promo_balance boolean validation in Promocode model

See merge request gitlab-instance/yii-erp24!1133

fix(TO8-22): is_promo_balance boolean validation in Promocode model

Колонка is_promo_balance в PostgreSQL — boolean, а валидатор был integer.
PDO возвращает PHP false → IntegerValidator падает: "должно быть целым числом".

Merge branch 'feature_fomichev_2026-03-06_TO8-22_promobonuses' into 'develop'

feat(TO8-22): промокоды БЛАГО — активация и промо-списание бонусов

See merge request gitlab-instance/yii-erp24!1132

дубликаты

блокирование строки

логирование

fix(TO8-22): skip Yii validation on UsersBonus save in activatePromocode

Множество полей БД (dell, status, check_id и др.) не имеют дефолтов
на уровне модели. save(false) пропускает валидацию Yii — аналогично
actionSale() строка 902.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>