[Feature] При проверке публикации ресурса проверять опубликованность его родителей

[ychelovek]

Описание функции

Сделать галочку (кстати, было бы круто если бы она работала и для простых ресурсов), при простановке которой при вызове pdoResources, msProducts, да и просто при переходе на страницу, если один из родительских ресурсов недоступен, то и страница должна выдавать 404.

Проблема, которую решает

Учитывание доступности товаров и категорий при снятии публикации родительского ресурса

Предлагаемое решение

Возможно можно делать проверку по дереву ресурсов, которое, если я не ошибаюсь, доступно в кэше при первой его генерации.

Альтернативные варианты

Возможно плагином на событие page init.

Примеры использования

Если зайти на страницу /category1/category2/product, в тот момент когда category2 не опубликована - сайт на modx2, modx3 не обратит на это внимание и не покажет 404.

Дополнительный контекст

• Есть ли подобная функция в других e-commerce системах?
  Конечно есть, в том же самом bitrix можно указать шаблон формирования URL и параметр Строгая проверка URL'a.

[Ibochkarev]

Рассматриваем только msProduct и msCategory. Обычные ресурсы, pdoResources и правки ядра MODX вне этого issue.

Сейчас

Проверяется сам товар/категория, не предки. URI лежит строкой, дерево при запросе не обходится. Поэтому /category1/category2/product открывается, если category2 снята с публикации.

• msProducts: published товара. expandPublishedCategoryTree отсекает неопубликованных детей при depth > 0, но не корень &parents и не предков выше.
• Catalog API и корзина (validateProduct): снова только сам товар.
• Плагин на page init закрывает только заход. Списки, product/get и POST в корзину остаются дырявыми.

Предложение

1. Сервис ResourceAncestorVisibility (или рядом с CategoryTreeService)

Один контракт для всех входов:

• isPubliclyReachable(modResource $resource): bool
• filterReachableIds(array $ids): array

Ресурс достижим, если он сам published=1, deleted=0, и все предки до корня контекста тоже. hidemenu не трогаем. Пользователь с view_unpublished проходит проверку. Данные брать из getParentIds + короткая выборка по id (или кэш на запрос), не обходить дерево на каждую строку списка.

2. Системная настройка ms3_check_parent_published

По умолчанию false. Без TV на каждом ресурсе.

3. Точки вызова (только при включённой настройке)

Слой	Поведение
Плагин OnLoadWebDocument	Только msProduct / msCategory. Иначе sendErrorPage().
msProducts	До buildMsProductsWhereForCategories: отфильтровать id категорий, включая корень &parents. msCategoryMember не должен вытаскивать товар из недоступной ветки.
Catalog API (product/get, product/list)	Тот же сервис в criteria / post-filter.
CartItemManager::validateProduct	Отклонять товар с недоступным предком.

Не копировать проверку в четыре места. Все зовут один сервис.

4. Кэш

На OnDocUnPublished категории чистить кэш потомков. Иначе 404 на заходе будет, а закэшированный листинг останется.

5. Вне scope этого issue

• Обычные ресурсы / pdoResources / ядро MODX
• Строгая проверка канонического URL (как Bitrix)

До кода зафиксировать: товар уже в корзине, у которого родителя сняли после добавления — оставить до оформления или выкидывать при recalculate. Предлагаю выкидывать при validateProduct / recalculate, чтобы витрина и корзина не расходились.

[Ibochkarev]

@biz87 привет!

Что скажешь?