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.

Sign up for GitHub

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

Jump to bottom

Page type Display Pages Beneath Page setting doesn't work #11704

Closed
moriaiy opened this issue Oct 12, 2023 · 0 comments · Fixed by #11981
Closed

Page type Display Pages Beneath Page setting doesn't work #11704

moriaiy opened this issue Oct 12, 2023 · 0 comments · Fixed by #11981
Labels
Bug:Level of Effort:Unknown Bugs where the level of effort to solve or even reproduce is unknown. Bug Priority:Low This would be a nice fix, but it mildly improves things that already work. Bug:Reproduceable:Yes Able to reproduce the bug. Product Areas:Sitemap Type:Bug Existing functionality not performing as expected.

Comments

@moriaiy
Copy link

moriaiy commented Oct 12, 2023

Affected Version of Concrete CMS

9.x

Description

The full sitemap is displayed even though I have restricted the sitemap display and the sitemap nodes to be displayed in "Display Pages Beneath Page" in the "Basic Information > Publishing Method" setting under Page Type.

How to reproduce

Set the following publishing methods on the Blog Entry page:

  • Publish Method: Choose from pages of a certain type
  • Publish Beneath Pages of Type: Blog
  • Selector From Factor: In-Page Sitemap
  • Display Pages Beneath Page: Blog
  1. Create a page named "2023" with the Blog page type under the Blog page.
  2. From the full sitemap, click on the page Blog > 2023
  3. Click "Add page" in the menu.
  4. Check the "Page Location" in the composer.
    -> Normally, only the sitemap of the Blog page node should be displayed here.
    -> However, as of 9.2.1, the full sitemap is displayed.
    -> Maybe that's why the "2023" page is checked in 8.5.12, but not in 9.2.1.

Possible Solution

No response

Additional Context

9.2.1
image

8.5.12
image
@moriaiy moriaiy added the Type:Bug Existing functionality not performing as expected. label Oct 12, 2023
@aembler aembler added Bug Priority:Low This would be a nice fix, but it mildly improves things that already work. Product Areas:Sitemap Bug:Level of Effort:Unknown Bugs where the level of effort to solve or even reproduce is unknown. Bug:Reproduceable:Yes Able to reproduce the bug. labels Oct 13, 2023
hissy added a commit to hissy/concrete5 that referenced this issue Mar 9, 2024
@hissy
Fix "Display Pages Beneath Page" doesn't affect composer
a1c5f3d 
Fixes concretecms#11704
@hissy hissy mentioned this issue Mar 9, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug:Level of Effort:Unknown Bugs where the level of effort to solve or even reproduce is unknown. Bug Priority:Low This would be a nice fix, but it mildly improves things that already work. Bug:Reproduceable:Yes Able to reproduce the bug. Product Areas:Sitemap Type:Bug Existing functionality not performing as expected.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix "Display Pages Beneath Page" doesn't affect composer hissy/concrete5
2 participants
@aembler @moriaiy