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>

Merge remote-tracking branch 'origin/feature_fomichev_2026-03-06_TO8-22_promobonuses' into feature_fomichev_2026-03-06_TO8-22_promobonuses

fix(TO8-22): нормализация окончаний строк CRLF→LF

Файлы были сохранены с CRLF из-за WSL, конвертированы в LF
для чистого diff в MR.

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

удаление

удаление

fix(TO8-22): критические исправления промо-списания БЛАГО по код-ревью

Исправлены баги, выявленные при ревью:

1. Race condition: SELECT FOR UPDATE в actionActivatePromocode
2. burn_balans не трогается при промо-списании (только обычные бонусы)
3. user_balans_new не уменьшается при промо (списываются промо-бонусы)
4. writeOffAlready проверяет tip_sale='promobonus' для промо-дедупликации
5. date_end фильтр на promoPlusSum (просроченные промо не учитываются)
6. Обновление полей пользователя на промо early-return пути
7. Использование promocode.duration вместо YEAR_PERIOD в активации
8. attributeLabels для activated_by/activated_at в Promocode
9. Убран file_put_contents с промо-пути
10. Тесты обновлены: добавлены проверки баланса, исправлена формула

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

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

- Миграция: добавлены поля activated_by, activated_at в таблицу promocode
- Модель Promocode: константа TIP_SALE_PROMOBONUS, метод isActivatable()
- API endpoint actionActivatePromocode(): начисление 350 промо-бонусов с tip_sale='promobonus'
- Модификация actionSale(): автоматический выбор промо-списания vs стандартного, без кэшбека при промо
- Unit-тесты: 15 тестов (isActivatable + алгоритм промо-списания)

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

Merge branch 'feature_filippov_20260304_fix_start_shift_step3_bigint' into 'develop'

fix(timetable): guard admin_checkin query against empty bigint fields

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

fix(timetable): guard admin_checkin query against empty bigint fields

Prevent PDO exception when StartForm fields are empty strings — add null
check before querying admin_checkin to avoid PostgreSQL bigint cast error.

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

Merge branch 'feature_filippov_TO8-48_force_readyto_1c_timeout' into 'develop'

fix(TO8-48): increase session timeout to 4h, add JS keep-alive ping

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

fix(TO8-48): increase session timeout to 4h, add JS keep-alive ping

Prevent BadRequestHttpException (Unable to verify your data submission)
in WriteOffsErpController caused by PHP session expiry during long form filling.

- erp24/config/web.php: add session component, timeout=14400 (4 hours)
- SiteController: add actionKeepAlive() endpoint for JS ping
- session-keep-alive.js: ping /site/keep-alive every 15 min, pause on hidden tab, skip during upload
- main.php: register session-keep-alive.js globally

Merge branch 'feature_filippov_TO8-48_force_readyto_1c_timeout' into 'develop'

fix(TO8-48): send one summary Telegram message instead of per-order

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

fix(TO8-48): send one summary Telegram message instead of per-order

Merge branch 'feature_filippov_TO8-48_force_readyto_1c_timeout' into 'develop'

fix(TO8-48): fix PHPDoc syntax error, */5 closes block comment

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

fix(TO8-48): fix PHPDoc syntax error, */5 closes block comment

Merge branch 'feature_filippov_TO8-48_force_readyto_1c_timeout' into 'develop'

fix(TO8-48): fix readyto_1c strict comparison, add cron-based timeout check

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

fix(TO8-48): fix readyto_1c strict comparison, add cron-based timeout check

- Fix `checkAndSetReadyTo1c()`: cast `readyto_1c` to int before strict
  comparison — PostgreSQL returns string "0", so `!== 0` always skipped
- Add `forceReadyTo1cByTimeout()`: batch-marks orders older than 15 min
  as ready for 1C export, independent of incoming FlowWow emails
- Add console action `marketplace/check-ready-to-1c` for cron (*/5)
- Add unit tests for both string and int readyto_1c values

Merge branch 'feature_filippov_20260303_fix_delivery_house_length' into 'develop'

fix: widen delivery text cleanup regex, cap house to 16 chars

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

fix: widen delivery text cleanup regex, cap house to 16 chars

- Change regex from matching only "Уточните адрес доставки" to
  "Уточните \S+ доставки" to catch "время", "дату", etc.
- Add mb_substr guard: house field capped at 16 characters (DB limit)
- Fixes: "house should contain at most 16 characters" validation error

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

Merge branch 'feature_filippov_ERP-252_fix_delivery_street_parsing' into 'develop'

fix(ERP-252): safe serialization of deliveryText in logs (array to string fix)

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

fix(ERP-252): use district as street fallback, add prod data fixtures

YM API often returns address without street but with district
(e.g., "микрорайон Красногорка"). Now uses district as street fallback.

Added 6 test cases based on real production data:
- #4497: no street, has district (Бор, микрорайон Красногорка)
- #3519: has street (улица Генерала Зимина)
- #4328: district only (микрорайон Щербинки-1)
- #4297: no street, no district (деревня Анкудиновка)
- STRING format with "Уточните время"
- NULL delivery in raw_data

26 tests, 133 assertions — all passing.

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

fix(ERP-252): extract fillDeliveryAddressFallback method, eliminate duplication

- New method fillDeliveryAddressFallback() handles 3 raw_data formats:
  1) string (FlowWow text) → parseAddressFromDeliveryText()
  2) array with address (YM API JSON) → extract structured fields
  3) null/missing → defaults "Уточняется"
- Replaced 3 inline fallback blocks with single method call
- Fixed surprise-delivery parsing ("Отправим смс получателю...")
- 20 unit tests, 115 assertions — all passing

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

fix(ERP-252): add delivery parsing tests and fix surprise-delivery pattern

12 unit tests for parseAddressFromDeliveryText() covering:
- Full address (city/street/house)
- "Уточните адрес у получателя" pattern
- Surprise SMS delivery ("Отправим смс получателю...")
- Empty strings, prefix-only, unknown cities
- GPS coordinates for known cities
- Pickup delivery, two-word cities
- Result structure validation (all keys, no nulls)

Fixed parser: added filters for surprise-delivery phrases
that were incorrectly parsed as street names.

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

fix(ERP-252): safe serialization of deliveryText in logs (array to string fix)

$rawData['delivery'] can be array (JSON object), not just string.
Use json_encode for non-string values in log messages.

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

Merge branch 'feature_filippov_ERP-252_fix_delivery_street_parsing' into 'develop'

fix(ERP-252): fallback when address object exists but street is empty

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

fix(ERP-252): fallback when address object exists but street is empty

$delivery->getAddress() returns non-null object with empty street field.
Changed from if/else to unconditional check: if street is still empty
after structured address fill, apply parseAddressFromDeliveryText()
or "Уточняется" defaults. Markers updated to [ERP-252-v3].

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

Merge branch 'feature_filippov_ERP-252_fix_delivery_street_parsing' into 'develop'

fix(ERP-252): add [ERP-252-v2] markers to verify deployment

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

fix(ERP-252): add unique markers [DELIVERY-SAVE-1/2/3] to each error location

fix(ERP-252): add [ERP-252-v2] markers to verify deployment

Temporary logging to confirm new code is running on production.
Search for "[ERP-252-v2]" in logs to verify.

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

Merge branch 'feature_filippov_ERP-252_fix_delivery_street_parsing' into 'develop'

fix(ERP-252): parse delivery address from text when API returns no structured address

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

fix(ERP-251): parse delivery address from text when API returns no structured address

Some orders (FlowWow) have delivery info as free text in raw_data
instead of structured address fields. When $delivery->getAddress()
returns null, fallback to parseAddressFromDeliveryText() to extract
city/street/house from the text. Applied in all 3 delivery creation
points in processOrders().

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

Merge branch 'feature_filippov_ERP-251_fix_delivery_house_length' into 'develop'

fix(20260302): cast time() to string for request_id in LogService calls

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

fix(20260302): cast time() to string for request_id in LogService calls

ApiLogs.request_id validates as string, but time() returns int causing
"Значение «Request ID» должно быть строкой" validation error on every request.

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

Merge branch 'feature_filippov_ERP-251_fix_delivery_house_length' into 'develop'

fix(ERP-251): replace file_put_contents with Yii logging in DataBuhController

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

fix(ERP-251): replace file_put_contents with Yii logging in DataBuhController

Hardcoded path /var/www/erp24/api2/json_buh does not exist on deploy
server (/var/www/erp24_deploy/), causing file_put_contents errors.
Replace debug file dumps with Yii::info/Yii::error under 'buh.upload'
category. Request data is already persisted via ApiCronBuh model.

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

Merge branch 'feature_filippov_ERP-251_fix_delivery_house_length' into 'develop'

fix(ERP-251): improve delivery address parsing from FlowWow emails

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

fix(ERP-251): improve delivery address parsing from FlowWow emails

parseAddressFromDeliveryText() did a naive comma-split on raw
deliveryText, treating date/time fragments and instructions like
"Уточните адрес доставки у получателя" as address fields. This caused
house field to exceed the 16-char DB column limit.

Now strips delivery type prefix, date/time patterns, and non-address
phrases before splitting. Handles 1-2 part addresses gracefully
instead of requiring 3+ comma-separated parts.

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

Merge branch 'feature_filippov_ERP-251_fix_bonus_log_permission' into 'develop'

fix(ERP-251): replace file_put_contents with Yii::info() in BonusController

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

fix(ERP-251): replace file_put_contents with Yii::info() in BonusController

Replace USERS_AUTH_CALL_LOG2 file writes that caused permission denied
errors on production with standard Yii::info() logging under
'bonus.auth' category. Remove unused $LOG and $USERS_AUTH_CALL_LOG2
static properties.

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

Merge branch 'feature_filippov_ERP-250_fix_writeoffs_null_check' into 'develop'

fix(ERP-250): add null check for WriteOffsErp model in actionUpdate

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

fix(ERP-250): add null check for WriteOffsErp model in actionUpdate

$query->one() can return null when the record is deleted, sent to 1C,
or the user has no store access. Without the check, line 1076 crashes
with "Call to a member function getRelation() on null".

Now throws NotFoundHttpException (404) as documented in the PHPDoc.

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

Merge branch 'feature_filippov_ERP-248_fix_store_scope' into 'develop'

fix(ERP-248): move $store lookup before if/else to fix undefined variable

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

fix(ERP-248): move $store lookup before if/else to fix undefined variable

$store was defined only inside the "new order" block (line 1252), but
referenced in 5 places inside the "existing order" else-block (lines
1692, 1735, 1770, 1828, 2017), causing "Undefined variable $store"
on production.

Moved $warehouseGuid and $store assignment before the if/else split
so both branches share the same variable scope.

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

Merge branch 'feature_filippov_ERP-248_fix_subsidy_fallback' into 'develop'

fix(ERP-248): add partner_warehouse_id field to marketplace-store form

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

fix(ERP-248): use marketplace_store.partner_warehouse_id as fallback for DBS orders

Add partner_warehouse_id column to marketplace_store table for storing
the physical YM warehouse ID (e.g. 206008) separate from warehouse_guid
(campaign ID 142743530). Use it as fallback when getPartnerWarehouseId()
returns null for DBS orders instead of the incorrect warehouse_guid.

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

fix(ERP-248): fallback subsidy from subsidies[] when getSubsidy() returns null

YandexMarket deprecated the item-level `subsidy` field and stopped sending
it for some order types (DBS). Instead, subsidy is now in `subsidies[]`.

When getSubsidy() returns null:
- sum amounts from getSubsidies() array
- log Yii::warning with calculated value

When getPartnerWarehouseId() returns null:
- fall back to marketplace_order.warehouse_guid (valid for DBS orders)
- only Yii::error if warehouse_guid is also null

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

Merge branch 'feature_filippov_fix_write_offs_erp_delete_video_csrf' into 'develop'

fix: add _csrf token to delete-video AJAX in write_offs_erp form

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

fix: add _csrf token to delete-video AJAX in write_offs_erp form

POST request to write-offs-erp/delete-video was missing _csrf token,
causing 400 BadRequestHttpException on every video removal.

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

Merge branch 'feature_filippov_ERP-247_add_delivery_logging' into 'develop'

fix(ERP-247): remove debug Yii::error срок доставки logs from MarketplaceService

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

fix(ERP-247): skip item save on null subsidy/partner_warehouse_id, log Yii::error

When getSubsidy() or getPartnerWarehouseId() returns null — log error with
orderId + shopSku and skip saving the item (continue) instead of writing
invalid fallback values (0 / 'N/A') to DB.

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

fix(ERP-247): change срок доставки log level from info to warning for Loki visibility

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

fix(ERP-247): add warning logs for null subsidy/partner_warehouse_id, restore delivery info as Yii::info

- Extract getSubsidy() and getPartnerWarehouseId() to local vars in all 6 item-saving blocks
- Log Yii::warning with orderId + shopSku when either returns null
- Restore срок доставки shipments loop (was removed as debug, actually informational)
- Change level from Yii::error to Yii::info for shipment delivery time logging

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

fix(ERP-247): remove debug Yii::error срок доставки logs from MarketplaceService

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

Merge branch 'feature_filippov_ERP-247_add_delivery_logging' into 'develop'

fix(ERP-247): improve delivery save error logging in MarketplaceService

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

fix(ERP-247): improve delivery save error logging in MarketplaceService

- Log deliveryText when type not recognized (was silent return false)
- Log delivery model errors + deliveryText when model.save() fails
  (previously logged wrong model's errors - marketplaceOrder instead)
- Fix callers to log order id instead of empty marketplaceOrder.getErrors()

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

Merge branch 'feature_filippov_fix_write_offs_erp_csrf' into 'develop'

fix(ERP-247): add CSRF token to AJAX calls in WriteOffsErp view

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

fix(ERP-247): add CSRF token to AJAX calls in WriteOffsErp view

$.ajax() calls for confirm-write-off and re-send-write-off were posting
without _csrf token, causing BadRequestHttpException 400 on every request.
Added yii.getCsrfToken() to data payload of both calls.

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

Merge branch 'chore/update-claude-skills' into 'develop'

chore: update .claude/skills — refactor to <500 lines + references/assets

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

chore: update .claude/skills — refactor to <500 lines + references/assets

- Extracted detailed content from 21 skills into references/ and assets/
- Each SKILL.md now acts as a hub (<500 lines) with JiT-loaded references
- Added new skills: code-review-expert
- Added references/ dirs for: jira-*, github-*, swarm-*, sparc-*, pair-programming, etc.
- Added assets/ dirs for: agentdb-optimization, flow-nexus-platform, github-*, etc.

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

Merge branch 'feature_filippov_ERP-246_fix_api1_site_view' into 'develop'

fix(ERP-246): add missing api1/views/site/index.php to fix ViewNotFoundException

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

fix(ERP-246): replace hardcoded OUT_DIR with relative path in UploadService

Hardcoded path /var/www/erp24/api2/json caused Permission denied errors
on production when log files were owned by non-www-data users.
Using __DIR__ . '/../api2/json' makes the path deployment-agnostic.

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

fix(ERP-246): add missing api1/views/site/index.php to fix ViewNotFoundException

api1 and main app share namespace app\controllers, so SiteController
from the main app is accessible via api1. When GET / hits api1, Yii2
routes to SiteController::actionIndex() which calls render('index'),
but api1/views/site/index.php did not exist → ViewNotFoundException (500).

Added empty view file to stop the error. api1 is a REST API (FORMAT_JSON)
so the rendered empty string is JSON-encoded as "".

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

Merge branch 'feature_filippov_ERP-245_fix_domru_cams_type_error' into 'develop'

[ERP-245]: separate broken-file marker from image src URL

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

fix(ERP-245): separate broken-file marker from image src URL

- WriteOffsErp::getImagesList() previously set $imageThumbRow='broken_file-error'
  as sentinel, then built src="/broken_file-error" causing Yii2 to try routing
  it as a controller action → repeated 404 errors in production logs
- Separated marker (kept in $relaFileName for alt/debug) from URL (now points
  to valid static placeholder /images/no-image.svg)
- Added erp24/web/images/no-image.svg: lightweight SVG fallback served by nginx

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

Merge branch 'feature_filippov_ERP-245_fix_domru_cams_type_error' into 'develop'

fix(ERP-245): add is_array check to second foreach in domru-cams

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

fix(ERP-245): add is_array check to second foreach in domru-cams

Same TypeError fix for the second foreach loop that inserts data
into store_visitors table.

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

Merge branch 'feature_filippov_ERP-245_fix_domru_cams_type_error' into 'develop'

[ERP-245]: prevent TypeError when Domru XML has single count element

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

fix(ERP-245): prevent TypeError when Domru XML has single count element

When XML response contains only one count element, json_decode(json_encode($xml))
produces a flat array instead of nested, causing $arr to be a string.
Skip non-array entries to avoid "Cannot access offset of type string on string".

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

Merge branch 'feature_filippov_ERP-244_fix_shift_reminder_csrf' into 'develop'

[ERP-244]: fix CSRF validation error in ShiftReminderController

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

fix(ERP-244): fix CSRF validation error in ShiftReminderController

- Disable CSRF validation on ShiftReminderController: endpoint is
  protected by session authentication (AccessControl, roles=['@']).
  CSRF cookies may be absent in browsers with strict privacy settings,
  causing false 400 errors for legitimate authenticated users.
- Stop retrying on 400/401/403 responses in shift-reminder.js to prevent
  cascading error floods in logs when auth/validation fails.

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

Merge branch 'feature_filippov_ERP-242_fix_domru_xml_validation' into 'develop'

fix(ERP-242): validate XML response from Domru API before parsing

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

Merge branch 'feature_filippov_ERP-243_fix_timetable_transaction' into 'develop'

fix(ERP-243): catch all exceptions in TimetableService transaction rollback

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

fix(ERP-243): catch all exceptions in TimetableService transaction rollback

catch (Exception) only caught yii\db\Exception, missing
InvalidArgumentException from validation/upload errors. This left
orphaned timetable records with tabel=1 but no timetable_fact entry,
blocking shift reopening.

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

fix(ERP-242): validate XML response from Domru API before parsing

- Skip iteration on empty curl response instead of writing empty file
- Use libxml_use_internal_errors to catch invalid XML gracefully
- Auto-create log/text/ directory if missing

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

Merge branch 'feature_filippov_2026-02-26_fix_bubble_png_path' into 'develop'

fix: correct bubble.png path in minified CSS

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

fix: correct bubble.png path in minified CSS

Minified CSS in web/min/ referenced ../images/png/bubble.png which
resolved to a non-existent web/images/png/ directory, causing 404.
Fixed to ../azea/assets/images/png/bubble.png where the file actually exists.

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