processwire / processwire-issues

ProcessWire issue reports.
44 stars 2 forks source link

Pages hidden from the Page List still show up on the Page Tree #1927

Open michaellenaghan opened 1 month ago

michaellenaghan commented 1 month ago

Short description of the issue

The ProcessPageList module allows you to hide pages "in page list(s)." It does hide pages in the page list — but it doesn't hide them in the Pages | Tree menu.

Expected behavior

When you hide pages in the ProcessPageList module they should be hidden in the page list and in the Pages | Tree menu.

Actual behavior

When you hide pages in the ProcessPageList module they're hidden in the page list but not in the Pages | Tree menu.

Optional: Screenshots/Links that demonstrate the issue

When you hide pages in the ProcessPageList module:

Screenshot 2024-05-17 at 5 50 21 PM

they're hidden in the page list:

Screenshot 2024-05-17 at 5 47 10 PM

but not in the Pages | Tree menu:

Screenshot 2024-05-17 at 5 57 06 PM

(Note that the last two screenshots are from a non-superuser.)

Steps to reproduce the issue

  1. Create some pages under Home
  2. Go to Modules | Configure | ProcessPageList
  3. Add those pages to "Hide these pages in page list(s)"
  4. Log out
  5. Disable advanced mode
  6. Disable debug mode
  7. Log in as a non-superuser
  8. The pages should be hidden from the main page list
  9. The pages should be visible in the Pages | Tree menu

Setup/Environment

ryancramerdesign commented 3 weeks ago

@michaellenaghan Thanks, I've pushed a fix for this.