Update of 
/var/cvs/contributions/CMSContainer/cmsc/portal/src/java/com/finalist/cmsc/portalImpl
In directory 
james.mmbase.org:/tmp/cvs-serv3644/cmsc/portal/src/java/com/finalist/cmsc/portalImpl

Modified Files:
        PageNavigationItemManager.java 
Log Message:
CMSC-1031 Changing an URL in an Alias and publishing it does not resolve to the 
new URL


See also: 
http://cvs.mmbase.org/viewcvs/contributions/CMSContainer/cmsc/portal/src/java/com/finalist/cmsc/portalImpl
See also: http://www.mmbase.org/jira/browse/CMSC-1031


Index: PageNavigationItemManager.java
===================================================================
RCS file: 
/var/cvs/contributions/CMSContainer/cmsc/portal/src/java/com/finalist/cmsc/portalImpl/PageNavigationItemManager.java,v
retrieving revision 1.10
retrieving revision 1.11
diff -u -b -r1.10 -r1.11
--- PageNavigationItemManager.java      25 Jul 2008 15:54:05 -0000      1.10
+++ PageNavigationItemManager.java      27 Aug 2008 14:56:02 -0000      1.11
@@ -13,29 +13,52 @@
 import com.finalist.cmsc.navigation.*;
 import com.finalist.cmsc.navigation.tree.PageTreeItemRenderer;
 
+/**
+ * NavigationItemManager implementation for Page types.
+ * This NavigationItemManager creates instance of Page class and is a base 
implementation for subtypes
+ */
 public class PageNavigationItemManager implements NavigationItemManager {
 
-    private static final Logger log = 
Logging.getLoggerInstance(PageNavigationItemManager.class.getName());
+   private static final Logger log = 
Logging.getLoggerInstance(PageNavigationItemManager.class);
 
        private NavigationItemRenderer renderer = new PageNavigationRenderer();
 
        private NavigationTreeItemRenderer treeRenderer = new 
PageTreeItemRenderer();
 
+   /**
+    * @see com.finalist.cmsc.navigation.NavigationItemManager#getRenderer()
+    */
        public NavigationItemRenderer getRenderer() {
                return renderer;
        }
 
+   /**
+    * @see com.finalist.cmsc.navigation.NavigationItemManager#getTreeManager()
+    */
        public String getTreeManager() {
                return PagesUtil.PAGE;
        }
 
+   /**
+    * @see com.finalist.cmsc.navigation.NavigationItemManager#getRelatedTypes()
+    */
+   public List<String> getRelatedTypes() {
+      return null;
+   }
+
+   /**
+    * @see com.finalist.cmsc.navigation.NavigationItemManager#isRoot()
+    */
     public boolean isRoot() {
         return false;
     }
 
-       public NavigationItem loadNavigationItem(Integer key, Node node) {
-        if (node == null || !PagesUtil.isPageType(node)) {
-            log.debug("Page not found: " + key);
+   /**
+    * @see 
com.finalist.cmsc.navigation.NavigationItemManager#loadNavigationItem(org.mmbase.bridge.Node)
+    */
+   public NavigationItem loadNavigationItem(Node node) {
+      if (!PagesUtil.isPageType(node)) {
+         log.debug("Node is not a Page: " + node.getNumber());
             return null;
         }
 
@@ -65,7 +88,8 @@
         Node layoutNode = PagesUtil.getLayout(pageNode);
         if (layoutNode != null) {
             page.setLayout(layoutNode.getNumber());
-        } else {
+      }
+      else {
             log.error("NO LAYOUT");
         }
     }
@@ -87,25 +111,47 @@
 
      /**
       * publishing of sites and pages is done by the publish module
+    *
+    * @see 
com.finalist.cmsc.navigation.NavigationItemManager#getPublisher(org.mmbase.bridge.Cloud,
 java.lang.String)
       */
     public Object getPublisher(Cloud cloud, String type) {
                return null;
        }
 
+   /**
+    * @see com.finalist.cmsc.navigation.NavigationItemManager#getTreeRenderer()
+    */
     public NavigationTreeItemRenderer getTreeRenderer() {
         return treeRenderer;
     }
 
+   /**
+    * POJO class for the Page type
+    * @return POJO class
+    */
     protected Class<? extends Page> getPageClass() {
         return Page.class;
     }
 
+   /**
+    * @see com.finalist.cmsc.navigation.NavigationItemManager#getItemClass()
+    */
     public Class<? extends NavigationItem> getItemClass() {
         return Page.class;
     }
 
+   /**
+    * @see 
com.finalist.cmsc.navigation.NavigationItemManager#deleteNode(org.mmbase.bridge.Node)
+    */
    public void deleteNode(Node pageNode) {
       PagesUtil.deletePage(pageNode);
    }
 
+   /**
+    * @see 
com.finalist.cmsc.navigation.NavigationItemManager#findItemForRelatedNode(org.mmbase.bridge.Node)
+    */
+   public Node findItemForRelatedNode(Node node) {
+      return null;
+   }
+
 }
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to