Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Проблема обработки parents=0 #350

Closed
Alexij2 opened this issue Oct 29, 2022 · 5 comments
Closed

Проблема обработки parents=0 #350

Alexij2 opened this issue Oct 29, 2022 · 5 comments

Comments

@Alexij2
Copy link

Alexij2 commented Oct 29, 2022

Сообщение об ошибке / Error message

Небольшая неприятность.
Если указать parents = 0 и depth = 0, то pdoResources выводит все ресурсы включая вложенные.
Если указать parents = '0,' То работает как надо.
Если Parents больше нуля, то все работает корректно.

Версия PdoTools 2.13.2-pl, MODX 2.8.4

@Alexij2 Alexij2 added the bug label Oct 29, 2022
@sergant210
Copy link
Member

Есть простое решение - прочитать документацию.

@sergant210 sergant210 removed the bug label Oct 31, 2022
@Alexij2
Copy link
Author

Alexij2 commented Nov 4, 2022

Ааа. семен семеныч... это фитча ). Но в моем случае я хотел через pdoResource вывести только 1й уровень, как в pdoMenu.
Прошу прощения за бессмысленную исушку.

@Alexij2
Copy link
Author

Alexij2 commented Nov 4, 2022

Хотя нет... речь идет именно о том что parents = 0 и depth = 0. те логично что parents = 0 - не ограничена выборка, но depth =0 не работает получается. А вот если указать например каталог id = 2, то да, depth = 0 срабатывает. Об этом речь.

@sergant210
Copy link
Member

sergant210 commented Nov 6, 2022

Алексей, я как и ты не знаю какую Василий закладывал логику. Чтобы узнать, нужно копать код. Открываем, смотрим - если parent пустой (а ноль это тоже пусто), то depth не учитывается. Мне кажется, ты мог бы это сделать и без меня.

П.С. Кстати, это логично. Если ты хочешь вывести всё без ограничений, то зачем тебе уровень? А если нужен уровень, то укажи точку отсчёта.

@Alexij2
Copy link
Author

Alexij2 commented Nov 6, 2022

Сергей, логика и так понятна, что depth не учитывается. Видимо, эта особенность только меня задела.
Ладно, значит так и должно быть.

@Alexij2 Alexij2 closed this as completed Feb 9, 2023
This issue was closed.
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

No branches or pull requests

2 participants