Top menu changin changes also lower level menus

code/MoyaWeb/src/fi/codecrew/moya/web/cdiview/menu/MenuView.java

@@ -84,6 +84,7 @@ public class MenuView extends GenericCDIView {
 	}
 
 	public void menuChangeEvent() {
+		layoutview.setPageName(menuChange);
 		navihandler.forward(menuChange);
 		//		super.navihandler.forward(menuChange);
 	}
@@ -233,6 +234,9 @@ public class MenuView extends GenericCDIView {
 	}
 
 	public String getMenuChange() {
+		if (menuChange == null || menuChange.isEmpty()) {
+			menuChange = layoutview.getPagepath();
+		}
 		return menuChange;
 	}
 

code/MoyaWeb/src/fi/codecrew/moya/web/cdiview/menu/PrimeMenuView.java

 package fi.codecrew.moya.web.cdiview.menu;
 
-import java.util.HashSet;
-import java.util.List;
 import java.util.Set;
 
 import javax.ejb.EJB;
@@ -77,19 +75,7 @@ public class PrimeMenuView extends GenericCDIView {
 		if (menuModel == null)
 		{
 			menuModel = new DefaultMenuModel();
-			Set<MenuNavigation> selectedSet = new HashSet<>();
-
-			final MenuNavigation thisPage = menubean.findNavigation(layoutview.getPagepath());
-			MenuNavigation selectedTop = thisPage;
-			while (selectedTop != null && selectedTop.getParent() != null) {
-				selectedTop = selectedTop.getParent();
-				selectedSet.add(selectedTop);
-			}
-
-			List<MenuNavigation> tops = menubean.getTopmenus();
-			if (!tops.contains(selectedTop) && !tops.isEmpty()) {
-				selectedTop = tops.get(0);
-			}
+			MenuNavigation selectedTop = layoutview.getSelectedTopmenu();
 
 			for (MenuNavigation m : selectedTop.getChildren()) {
 				if (m.getItem() != null && m.getChildren().isEmpty()) {
@@ -99,8 +85,7 @@ public class PrimeMenuView extends GenericCDIView {
 						menuModel.addElement(menuitem);
 					}
 				} else {
-
-					Submenu subm = addSubmenu(m, selectedSet);
+					Submenu subm = addSubmenu(m, layoutview.getSelectedSet());
 
 					if (subm != null) {
 						menuModel.addElement(subm);

code/MoyaWeb/src/fi/codecrew/moya/web/helper/LayoutView.java

 package fi.codecrew.moya.web.helper;
 
 import java.io.ByteArrayInputStream;
+import java.util.Collections;
+import java.util.HashSet;
+import java.util.List;
 import java.util.MissingResourceException;
 import java.util.ResourceBundle;
+import java.util.Set;
 
 import javax.ejb.EJB;
 import javax.enterprise.context.RequestScoped;
@@ -17,10 +21,12 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 import fi.codecrew.moya.beans.EventBeanLocal;
+import fi.codecrew.moya.beans.MenuBeanLocal;
 import fi.codecrew.moya.beans.PermissionBeanLocal;
 import fi.codecrew.moya.enums.apps.ContentPermission;
 import fi.codecrew.moya.model.LanEventProperty;
 import fi.codecrew.moya.model.LanEventPropertyKey;
+import fi.codecrew.moya.model.MenuNavigation;
 
 @Named
 @RequestScoped
@@ -37,6 +43,9 @@ public class LayoutView {
 	private StreamedContent headerimage;
 	private String headertext;
 
+	@EJB
+	private transient MenuBeanLocal menubean;
+
 	private static final Logger logger = LoggerFactory.getLogger(LayoutView.class);
 
 	public void init() {
@@ -48,6 +57,38 @@ public class LayoutView {
 		return permbean.hasPermission(ContentPermission.MANAGE_PAGES);
 	}
 
+	private Set<MenuNavigation> selectedSet;
+
+	public Set<MenuNavigation> getSelectedSet()
+	{
+		if (selectedSet == null) {
+			getSelectedTopmenu();
+		}
+		return selectedSet;
+	}
+
+	private MenuNavigation selectedTop;
+
+	public MenuNavigation getSelectedTopmenu()
+	{
+		if (selectedTop == null)
+		{
+			selectedSet = new HashSet<>();
+			selectedTop = menubean.findNavigation(getPagepath());
+			while (selectedTop != null && selectedTop.getParent() != null) {
+				selectedTop = selectedTop.getParent();
+				selectedSet.add(selectedTop);
+			}
+
+			List<MenuNavigation> tops = menubean.getTopmenus();
+			if (!tops.contains(selectedTop) && !tops.isEmpty()) {
+				selectedTop = tops.get(0);
+			}
+			selectedSet = Collections.unmodifiableSet(selectedSet);
+		}
+		return selectedTop;
+	}
+
 	public String getPagepath() {
 		if (pagename == null) {
 			HttpServletRequest req = (HttpServletRequest) context.getExternalContext().getRequest();
@@ -137,4 +178,9 @@ public class LayoutView {
 	public void setHeadertext(String headertext) {
 		this.headertext = headertext;
 	}
+
+	public void setPageName(String menuChange) {
+		this.pagename = menuChange;
+		selectedTop = null;
+	}
 }