Галерея: сортировка по имени после пакетной загрузки

[Ibochkarev]

Описание

После завершения пакетной загрузки в Uppy позиции файлов галереи пересчитываются natural sort по имени (strnatcasecmp на name, fallback на file). Пакет 01.jpg…10.jpg встаёт в ожидаемый порядок без ручного drag-sort.

Логика в ProductImageService, тонкий процессор Gallery/SortByName, вызов из ProductGallery на upload-complete. Upload.php и drag-sort (Sort.php) не менялись.

Тип изменений

• Новая функциональность (non-breaking change)
• Исправление бага (non-breaking change)
• Breaking change (изменение, ломающее обратную совместимость)
• Рефакторинг (без изменения функциональности)
• Документация
• Другое (опишите):

Связанные Issues

Refs #616 (только пункт 2 — Variant A: сортировка по имени). Компактный uploader и layout остаются вне этого PR.

Как это было протестировано?

cd core/components/minishop3
php -l src/Services/Product/ProductImageService.php
php -l src/Processors/Gallery/SortByName.php
composer test -- --filter ProductImageService
# exit 0 (11 tests, including 6 SortByName)

cd ../../../vueManager
npx eslint src/composables/useGalleryApi.js src/components/gallery/ProductGallery.vue
# exit 0

• Ручное тестирование
• Автоматические тесты (composer test -- --filter ProductImageService, ESLint на изменённых Vue/JS)
• Тестирование на разных версиях PHP/MODX

Конфигурация тестирования:

• MiniShop3: ветка feat/issue-616-gallery-sort-by-name
• MODX: n/a (unit)
• PHP: 8.4.23

Скриншоты (если применимо)

До	После
n/a	n/a

Чеклист

• Код соответствует стилю проекта
• Добавлены/обновлены комментарии в сложных местах
• Изменения не ломают существующую функциональность
• Лексиконы добавлены на двух языках (ru/en) — новых ключей нет
• PHPStan проходит без новых ошибок (composer stan / CI job PHPStan)
• ESLint проходит без ошибок (eslint на затронутых файлах)
• Обновлён CHANGELOG.md (для значимых изменений) — по политике репозитория на релиз

Дополнительные заметки

• После batch complete вызывается SortByName только если есть успешные uploads. Очередь в одной вкладке сериализует overlapping complete при allowMultipleUploadBatches.
• Пересчёт позиций для всего product_id / parent_id = 0 после загрузки может перезаписать предыдущий ручной drag-порядок (ожидаемое ms2-like поведение).
• Вне scope: компактный uploader, layout, Variant B/C (gallerySortOnUpload), замена Uppy.

Ручная проверка

1. Открыть галерею товара в менеджере.
2. Загрузить пакет 01.jpg…10.jpg.
3. Убедиться, что сетка идёт 01…10 без ручной сортировки.
4. Перетащить файл drag-sort — порядок меняется как раньше.

[AgelxNash]

Этот PR включён в тестовую интеграционную сборку всех открытых PR MiniShop3: AgelxNash/MiniShop3, ветка integration/open-prs-20260906 (28/28 открытых).

Сборка нужна, чтобы проверить совместимость взаимозависимых серий PR до их мержа — при последовательном слиянии они конфликтуют друг с другом. Это не ревью и не конкурирующий PR: авторство сохранено (1 PR = 1 коммит с исходным автором), ветка пересобирается по мере обновления PR.

Как вошёл в сборку: Слился чисто. (Правки сохранены при последующем #640.)

[AgelxNash]

Отличная работа! Желаю этому PR быстрого мержа и ни одного конфликта 🙌

[biz87]

Проверил — механика хорошая, но одна вещь не выполняется, и как раз для основного нашего сценария.

Что подтвердилось

• Natural sort по числам работает верно: 01.jpg | 2.jpg | 10.jpg | IMG_2.jpg | IMG_10.jpg — прогнал, порядок правильный, «2» перед «10».
• Затрагиваются только строки конкретного product_id с parent_id = 0 — чужие товары не задеты.
• is_preview / preview_file_id не меняется: сортировка ничего не удаляет, ссылка остаётся валидной.
• Права строже, чем у соседей по каталогу процессоров: msproductfile_save выставлен явно, вызов идёт через штатный коннектор.
• Очередь uploadCompleteQueue действительно нужна — allowMultipleUploadBatches: true в GalleryUploader.vue включён, перекрывающиеся complete реальны.
• msProductFile наследует xPDOSimpleObject, поэтому известная ловушка проекта с getIterator() и derivative criteria сюда не относится.
• Мерж с актуальной beta (после feat(web-api): галерея изображений товара (images[]) #598, который тоже правит ProductImageService.php) — чистый, конфликтов нет. Публичную галерею не ломает: ProductGalleryPublicService читает те же position и preview_file_id.

Локально: php -l чисто, smoke 92/92, PHPUnit 294 теста / 725 assertions, WebApi 22/22, --filter ProductImageService 11/11, ESLint чисто, vitest 43/43, build проходит. PHPStan из vendor/bin (CI-pinned версия) — 0 ошибок.

Что просьба поправить

Заявленная в описании регистронезависимость не работает для кириллицы. strnatcasecmp под locale C не делает case-folding многобайтных символов, поэтому имена с заглавной буквы группируются перед именами со строчной:

как сейчас (strnatcasecmp):   Фото 1.jpg | Фото 10.jpg | фото 2.jpg | фото 3.jpg
с mb_strtolower перед сравнением: Фото 1.jpg | фото 2.jpg | фото 3.jpg | Фото 10.jpg

Прогнал напрямую, воспроизводится стабильно. Для латиницы всё в порядке — проблема именно в многобайтных символах.

Смущает то, что это не край, а основной случай: аудитория у нас русская, имена загружаемых фотографий сплошь кириллические, и разный регистр первой буквы в пачке — обычное дело. Фича называется «сортировка по имени», и на типичной галерее она даст не тот порядок, которого ждёт пользователь.

Правка — mb_strtolower() перед сравнением в ProductImageService::buildNaturalSortRanks() (строки ~134-138).

И заодно тест: testBuildNaturalSortRanksIsCaseInsensitive сейчас покрывает только ASCII (B.jpg / a.jpg), поэтому регрессию не ловит. Нужен кейс с кириллицей разного регистра — иначе это вернётся.

Кстати, это ровно тот же класс, что мы сегодня чинили в #618/#619: байтовые операции над UTF-8 без учёта многобайтности. Возможно, стоит поискать такие места системно.

Мелочи, не блокеры

• SortByName.php:25,31 использует лексиконы ms3_gallery_err_ns и ms3_gallery_err_no_product, которых нет ни в ru, ни в en — пользователь при ошибке увидит сырой ключ. Это унаследованный паттерн, ровно те же ключи дёргают Generate.php, GenerateAll.php, RemoveAll.php, SetPreview.php, Update.php и Upload.php. Твой PR ничего не ухудшил, просто фиксирую — заведём отдельно.
• rankProductImages() сохраняет позиции поштучно без транзакции. Метод существовал и раньше, но этим PR он начинает реально использоваться: при обрыве на середине галерея останется в смешанном порядке. Повторный запуск идемпотентен и всё восстановит, так что риск низкий.
• Интеграционного теста на сам процессор нет — инварианты «не трогает чужой товар» и «preview не меняется» я проверил чтением кода, автотестом они не закреплены.

После правки с mb_strtolower — вливаем.

[Ibochkarev]

Адрес ревью:

• buildNaturalSortRanks(): перед сравнением ключи проходят mb_strtolower(..., 'UTF-8') (Cyrillic case-fold; strnatcasecmp под locale C этого не делает)
• Тест testBuildNaturalSortRanksIsCaseInsensitiveForCyrillic: Фото 1 → фото 2 → фото 3 → Фото 10

Rebase на актуальную beta без конфликтов. composer ci:php зелёный (--filter ProductImageService 12/12).

#660

В этот PR не включал: Sort.php/Multiple.php permissions, глобальный тест на $permission и недостающие ms3_gallery_err_* — отдельный ACL/lexicon scope, не про сортировку по имени. Логичнее закрыть отдельным PR по issue.