[MGNLGS-86] Pages subapp hides pages implicitelly Created: 09/Jun/14 Updated: 10/Jul/14 Resolved: 09/Jul/14 |
|
| Status: | Closed |
| Project: | Google Sitemap (closed) |
| Component/s: | None |
| Affects Version/s: | 2.0 |
| Fix Version/s: | 2.0.4, 2.2 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Zdenek Skodik | Assignee: | Eric Hechinger |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | support | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Template: |
|
||||
| Acceptance criteria: |
Empty
|
||||
| Date of First Response: | |||||
| Description |
|
As far as I can see in the code, it has been meant to always render all pages belonging to tree attached to particular sitemap so that e.g. an user can hide a page on demand or revert such a change if the constraint is gone. The service however iterate over the nodes only in vertical axis, it never moves back to pages on the same level of a parent hence effectively hides a significant part of the tree implicitly. Following the demo-project tree, pages like /demo-project/about/history or the entire /demo-project/news-and-events/news-overview subtree is not rendered by the subapp, although it's included in the resulting sitemap. |
| Comments |
| Comment by Roman Kovařík [ 13/Jun/14 ] |
|
Page is lost in info.magnolia.module.googlesitemap.app.subapp.sitemapdetail.util.SiteMapEntryContainer.getChildren(Object) |
| Comment by Eric Hechinger [ 09/Jul/14 ] |
|
Already fixed during refactoring... just need code polishing and unit test |