Author: mputz Date: 2012-09-28 10:54:40 -0400 (Fri, 28 Sep 2012) New Revision: 8853 Modified: epp/portal/branches/EPP_5_2_2_GA_BZ851985/component/portal/src/main/java/org/exoplatform/portal/mop/navigation/NodeContext.java epp/portal/branches/EPP_5_2_2_GA_BZ851985/component/portal/src/main/java/org/exoplatform/portal/mop/user/UserNodeFilter.java epp/portal/branches/EPP_5_2_2_GA_BZ851985/component/portal/src/main/java/org/exoplatform/portal/mop/user/UserNodeFilterConfig.java epp/portal/branches/EPP_5_2_2_GA_BZ851985/webui/portal/src/main/java/org/exoplatform/portal/webui/page/UIPageActionListener.java epp/portal/branches/EPP_5_2_2_GA_BZ851985/webui/portal/src/main/java/org/exoplatform/portal/webui/portal/UIApplicationList.java Log: Resolved BZ#851985 - Performance issue in EPP 5.2 due to additional pages being loaded Modified: epp/portal/branches/EPP_5_2_2_GA_BZ851985/component/portal/src/main/java/org/exoplatform/portal/mop/navigation/NodeContext.java =================================================================== --- epp/portal/branches/EPP_5_2_2_GA_BZ851985/component/portal/src/main/java/org/exoplatform/portal/mop/navigation/NodeContext.java 2012-09-28 14:48:13 UTC (rev 8852) +++ epp/portal/branches/EPP_5_2_2_GA_BZ851985/component/portal/src/main/java/org/exoplatform/portal/mop/navigation/NodeContext.java 2012-09-28 14:54:40 UTC (rev 8853) @@ -304,7 +304,7 @@ private void doFilter(int depth, NodeFilter filter) { - boolean accept = filter.accept(depth, getId(), name, getState()); + boolean accept = filter.accept(depth, getId(), getName(), getState()); setHidden(!accept); if (expanded) { Modified: epp/portal/branches/EPP_5_2_2_GA_BZ851985/component/portal/src/main/java/org/exoplatform/portal/mop/user/UserNodeFilter.java =================================================================== --- epp/portal/branches/EPP_5_2_2_GA_BZ851985/component/portal/src/main/java/org/exoplatform/portal/mop/user/UserNodeFilter.java 2012-09-28 14:48:13 UTC (rev 8852) +++ epp/portal/branches/EPP_5_2_2_GA_BZ851985/component/portal/src/main/java/org/exoplatform/portal/mop/user/UserNodeFilter.java 2012-09-28 14:54:40 UTC (rev 8853) @@ -96,6 +96,10 @@ public boolean accept(int depth, String id, String name, NodeState state) { + if (config.customFilter != null && !config.customFilter.accept(depth, id, name, state)) + { + return false; + } Visibility visibility = state.getVisibility(); // Correct null -> displayed Modified: epp/portal/branches/EPP_5_2_2_GA_BZ851985/component/portal/src/main/java/org/exoplatform/portal/mop/user/UserNodeFilterConfig.java =================================================================== --- epp/portal/branches/EPP_5_2_2_GA_BZ851985/component/portal/src/main/java/org/exoplatform/portal/mop/user/UserNodeFilterConfig.java 2012-09-28 14:48:13 UTC (rev 8852) +++ epp/portal/branches/EPP_5_2_2_GA_BZ851985/component/portal/src/main/java/org/exoplatform/portal/mop/user/UserNodeFilterConfig.java 2012-09-28 14:54:40 UTC (rev 8853) @@ -21,7 +21,7 @@ import org.exoplatform.commons.utils.Safe; import org.exoplatform.portal.mop.Visibility; - +import org.exoplatform.portal.mop.navigation.NodeFilter; import java.util.Collections; import java.util.EnumSet; import java.util.Set; @@ -50,6 +50,9 @@ /** . */ final boolean temporalCheck; + + /** . */ + final NodeFilter customFilter; public UserNodeFilterConfig(Builder builder) { @@ -62,6 +65,7 @@ this.visibility = Safe.unmodifiableSet(builder.withVisibility); this.authorizationMode = builder.withAuthorizationMode; this.temporalCheck = builder.withTemporalCheck; + this.customFilter = builder.withCustomFilter; } public Set<Visibility> getVisibility() @@ -100,6 +104,9 @@ /** . */ private boolean withTemporalCheck; + + /** . */ + private NodeFilter withCustomFilter; private Builder() { @@ -191,6 +198,12 @@ this.withAuthorizationMode = AUTH_NO_CHECK; return this; } + + public Builder withCustomFilter(NodeFilter customFilter) + { + this.withCustomFilter = customFilter; + return this; + } public UserNodeFilterConfig build() { Modified: epp/portal/branches/EPP_5_2_2_GA_BZ851985/webui/portal/src/main/java/org/exoplatform/portal/webui/page/UIPageActionListener.java =================================================================== --- epp/portal/branches/EPP_5_2_2_GA_BZ851985/webui/portal/src/main/java/org/exoplatform/portal/webui/page/UIPageActionListener.java 2012-09-28 14:48:13 UTC (rev 8852) +++ epp/portal/branches/EPP_5_2_2_GA_BZ851985/webui/portal/src/main/java/org/exoplatform/portal/webui/page/UIPageActionListener.java 2012-09-28 14:54:40 UTC (rev 8853) @@ -19,15 +19,19 @@ package org.exoplatform.portal.webui.page; +import org.apache.commons.lang.StringUtils; import org.exoplatform.portal.application.PortalRequestContext; import org.exoplatform.portal.config.DataStorage; import org.exoplatform.portal.config.UserPortalConfig; +import org.exoplatform.portal.config.UserPortalConfigService; import org.exoplatform.portal.config.model.Container; import org.exoplatform.portal.config.model.ModelObject; import org.exoplatform.portal.config.model.Page; import org.exoplatform.portal.config.model.PortalConfig; import org.exoplatform.portal.mop.SiteKey; import org.exoplatform.portal.mop.SiteType; +import org.exoplatform.portal.mop.navigation.NodeFilter; +import org.exoplatform.portal.mop.navigation.NodeState; import org.exoplatform.portal.mop.user.UserNavigation; import org.exoplatform.portal.mop.user.UserNode; import org.exoplatform.portal.mop.user.UserNodeFilterConfig; @@ -36,6 +40,7 @@ import org.exoplatform.portal.webui.portal.PageNodeEvent; import org.exoplatform.portal.webui.portal.UIPortal; import org.exoplatform.portal.webui.util.PortalDataMapper; +import org.exoplatform.portal.webui.util.Util; import org.exoplatform.portal.webui.workspace.UIPortalApplication; import org.exoplatform.portal.webui.workspace.UIWorkingWorkspace; import org.exoplatform.web.application.ApplicationMessage; @@ -64,11 +69,17 @@ UIPortalApplication uiPortalApp = event.getSource(); UIPortal showedUIPortal = uiPortalApp.getCurrentSite(); - UserNodeFilterConfig.Builder builder = UserNodeFilterConfig.builder(); - builder.withReadCheck(); + PageNodeEvent<UIPortalApplication> pageNodeEvent = (PageNodeEvent<UIPortalApplication>)event; String nodePath = pageNodeEvent.getTargetNodeUri(); + + UserNodeFilterConfig.Builder pathFilterBuild = UserNodeFilterConfig.builder(); + pathFilterBuild.withReadCheck(); + pathFilterBuild.withCustomFilter(new PathFilter(nodePath)); + + UserNodeFilterConfig.Builder defNodeBuild = UserNodeFilterConfig.builder(); + defNodeBuild.withCustomFilter(new DefaultNodeFilter()); UserNode targetNode = null; SiteKey siteKey = pageNodeEvent.getSiteKey(); @@ -77,13 +88,13 @@ UserNavigation navigation = userPortal.getNavigation(siteKey); if (navigation != null) { - targetNode = userPortal.resolvePath(navigation, builder.build(), nodePath); + targetNode = userPortal.resolvePath(navigation, pathFilterBuild.build(), nodePath); if (targetNode == null) { // If unauthenticated users have no permission on PORTAL node and URL is valid, they will be required to login if (pcontext.getRemoteUser() == null && siteKey.getType().equals(SiteType.PORTAL)) { - targetNode = userPortal.resolvePath(navigation, builder.withAuthMode(UserNodeFilterConfig.AUTH_NO_CHECK).build(), nodePath); + targetNode = userPortal.resolvePath(navigation, null, nodePath); if (targetNode != null) { uiPortalApp.setLastRequestURI(null); @@ -94,7 +105,7 @@ else { // If path to node is invalid, get the default node instead of. - targetNode = userPortal.getDefaultPath(navigation, builder.build()); + targetNode = userPortal.getDefaultPath(navigation, defNodeBuild.build()); } } } @@ -102,7 +113,7 @@ if (targetNode == null) { - targetNode = userPortal.getDefaultPath(builder.build()); + targetNode = userPortal.getDefaultPath(defNodeBuild.build()); if (targetNode == null) { if (showedUIPortal != null) @@ -184,8 +195,84 @@ //Reset selected navigation on userPortalConfig PortalDataMapper.toUIPortal(uiPortal, userPortalConfig.getPortalConfig()); return uiPortal; + } + + private class PathFilter implements NodeFilter + { + private List<String> paths; + + public PathFilter(String nodePath) + { + String[] tmp = StringUtils.split(nodePath, "/"); + + paths = new ArrayList<String>(); + if (tmp != null) + { + for (String str : tmp) + { + if (str != null && !"".equals(str.trim())) + { + paths.add(str.trim()); + } + } + } + } + + @Override + public boolean accept(int depth, String id, String name, NodeState state) + { + return depth == 0 || (depth <= paths.size() && paths.get(depth -1).equals(name)); + } } - } + + private class DefaultNodeFilter implements NodeFilter + { + private boolean found; + + @Override + public boolean accept(int depth, String id, String name, NodeState state) + { + if (depth == 0) + { + return true; + } + else + { + if (found) + { + return false; + } + else + { + found = hasPermission(state.getPageRef()); + return found; + } + } + } + + private boolean hasPermission(String pageRef) + { + UIPortalApplication uiPortalApp = Util.getUIPortalApplication(); + UserPortalConfigService service = uiPortalApp.getApplicationComponent(UserPortalConfigService.class); + if (pageRef != null) + { + try + { + Page page = service.getPage(pageRef); + if (page != null) + { + return service.getUserACL().hasPermission(page); + } + } + catch (Exception e) {} + } + return false; + } + } + } + + + static public class DeleteGadgetActionListener extends EventListener<UIPage> { Modified: epp/portal/branches/EPP_5_2_2_GA_BZ851985/webui/portal/src/main/java/org/exoplatform/portal/webui/portal/UIApplicationList.java =================================================================== --- epp/portal/branches/EPP_5_2_2_GA_BZ851985/webui/portal/src/main/java/org/exoplatform/portal/webui/portal/UIApplicationList.java 2012-09-28 14:48:13 UTC (rev 8852) +++ epp/portal/branches/EPP_5_2_2_GA_BZ851985/webui/portal/src/main/java/org/exoplatform/portal/webui/portal/UIApplicationList.java 2012-09-28 14:54:40 UTC (rev 8853) @@ -90,8 +90,35 @@ { if (selectedCategory == null) return null; - List<Application> apps = selectedCategory.getApplications(); + String remoteUser = Util.getPortalRequestContext().getRemoteUser(); + if (remoteUser == null || remoteUser.equals("")) + { return null; } + + ApplicationRegistryService service = getApplicationComponent(ApplicationRegistryService.class); + UserACL userACL = Util.getUIPortalApplication().getApplicationComponent(UserACL.class); + + List<Application> allApps = selectedCategory.getApplications(); + List<Application> apps = new ArrayList<Application>(); + + for (Application app : allApps) + { + List<String> accessPermission = app.getAccessPermissions(); + if(accessPermission == null || accessPermission.size()==0) + { + continue; + } + accessCheck: + for (String p : accessPermission) + { + if (userACL.hasPermission(p)) + { + apps.add(app); + } + break accessCheck; + } + } + //Correct IconURL of Gadget GadgetRegistryService gadgetService = getApplicationComponent(GadgetRegistryService.class); for (Application app : apps) @@ -199,4 +226,4 @@ } } -} \ No newline at end of file +}