Bug 794084 (JBEPP-1150) - Invisible navigation node still shown in Sitemap portlet
Summary: Invisible navigation node still shown in Sitemap portlet
Keywords:
Status: CLOSED NEXTRELEASE
Alias: JBEPP-1150
Product: JBoss Enterprise Portal Platform 5
Classification: JBoss
Component: Portal
Version: 5.1.0.GA,5.1.1.GA
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: 5.2.0.ER01
Assignee: Martin Weiler
QA Contact:
URL: http://jira.jboss.org/jira/browse/JBE...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-09-06 11:53 UTC by Martin Weiler
Modified: 2012-02-28 16:25 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-09-06 12:47:04 UTC
Type: Bug


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker JBEPP-1150 0 None None None Never

Description Martin Weiler 2011-09-06 11:53:55 UTC
Help Desk Ticket Reference: https://na7.salesforce.com/500A0000007sCxx
Steps to Reproduce: - Log in as root
- Click on Site -> Edit Navigation
- Under Home, create two new navigation nodes, make one invisible
- Go to SiteMap portlet
- Expand 'Home' node -> both new sub-nodes are displayed 
Workaround Description: For EPP 5.1.x, edit gatein.ear/02portal.war/groovy/webui/core/UISitemapTree.gtmpl and add the following:
{code}
@@ -28,6 +28,10 @@
 			size++;
 			
 			node = treeNode.getNode();
+
+                        // continue if node is invisible
+                        if(!node.isDisplay()) continue;
+
 			nav = treeNode.getNavigation();
{code}
project_key: JBEPP

An invisible navigation node is displayed in the Sitemap portlet when the parent node is expanded (not with Expand All button).

Comment 1 Martin Weiler 2011-09-06 11:55:16 UTC
Workaround Description: Removed: For EPP 5.1.x, edit gatein.ear/02portal.war/groovy/webui/core/UISitemapTree.gtmpl and add the following:

@@ -28,6 +28,10 @@
 			size++;
 			
 			node = treeNode.getNode();
+
+                        // continue if node is invisible
+                        if(!node.isDisplay()) continue;
+
 			nav = treeNode.getNavigation();
 Added: For EPP 5.1.x, edit gatein.ear/02portal.war/groovy/webui/core/UISitemapTree.gtmpl and add the following:
{code}
@@ -28,6 +28,10 @@
 			size++;
 			
 			node = treeNode.getNode();
+
+                        // continue if node is invisible
+                        if(!node.isDisplay()) continue;
+
 			nav = treeNode.getNavigation();
{code}


Comment 2 Martin Weiler 2011-09-06 12:46:35 UTC
Fixed in GateIn trunk / EPP 5.2 branch by https://source.jboss.org/changelog/GateIn?cs=6783

Comment 3 Martin Weiler 2011-09-06 12:47:04 UTC
Release Notes Docs Status: Added: Not Required


Comment 4 Thomas Heute 2011-09-22 07:12:21 UTC
Release Notes Docs Status: Removed: Not Required Added: Not Yet Documented


Comment 5 Jared MORGAN 2011-11-29 22:15:18 UTC
Release Notes Docs Status: Removed: Not Yet Documented Added: Documented as Resolved Issue
Release Notes Text: Added: An invisible navigation node was displayed in the Sitemap portlet when the parent node was expanded (not with Expand All button). The workaround described in the linked JIRA has been implemented in this release, which corrects the originally reported issue.



Note You need to log in before you can comment on or make changes to this bug.