Skip to content

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

Open
Ibochkarev wants to merge 2 commits into
betafrom
feat/issue-616-gallery-sort-by-name
Open

Галерея: сортировка по имени после пакетной загрузки#643
Ibochkarev wants to merge 2 commits into
betafrom
feat/issue-616-gallery-sort-by-name

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

После завершения пакетной загрузки в Uppy позиции файлов галереи пересчитываются natural sort по имени (strnatcasecmp на name, fallback на file). Пакет 01.jpg10.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.jpg10.jpg.
  3. Убедиться, что сетка идёт 01…10 без ручной сортировки.
  4. Перетащить файл drag-sort — порядок меняется как раньше.

After Uppy upload-complete, re-rank product gallery files by strnatcasecmp
on name so numbered batches match ms2-style order without manual drag.
@Ibochkarev
Ibochkarev requested a review from biz87 August 30, 2026 06:36
usort already reindexes the list; array_values was a no-op warning.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant