[portal-commits] JBoss Portal SVN: r11828 - branches/JBoss_Portal_Branch_2_7/core-admin/src/main/org/jboss/portal/core/admin/ui/coordination.

[portal-commits at lists.jboss.org]

Author: chris.laprun at jboss.com
Date: 2008-09-06 08:57:20 -0400 (Sat, 06 Sep 2008)
New Revision: 11828

Modified:
   branches/JBoss_Portal_Branch_2_7/core-admin/src/main/org/jboss/portal/core/admin/ui/coordination/AliasBindingManagerBean.java
Log:
- rename and create should now work for aliases as well now! Woot! ^_^

Modified: branches/JBoss_Portal_Branch_2_7/core-admin/src/main/org/jboss/portal/core/admin/ui/coordination/AliasBindingManagerBean.java
===================================================================
--- branches/JBoss_Portal_Branch_2_7/core-admin/src/main/org/jboss/portal/core/admin/ui/coordination/AliasBindingManagerBean.java	2008-09-06 12:44:36 UTC (rev 11827)
+++ branches/JBoss_Portal_Branch_2_7/core-admin/src/main/org/jboss/portal/core/admin/ui/coordination/AliasBindingManagerBean.java	2008-09-06 12:57:20 UTC (rev 11828)
@@ -115,7 +115,7 @@
 
    public List<QNameSelectItem> getAvailableRenderParameters()
    {
-      if(availableRenderParameters == null)
+      if (availableRenderParameters == null)
       {
          return Collections.emptyList();
       }
@@ -157,29 +157,19 @@
 
    public String delete(String name) throws IllegalCoordinationException
    {
-      DisplayAliasBinding binding = displayAliasBindings.get(name);
-      if (binding != null)
-      {
-         AliasBindingInfo alias = binding.getAlias();
-         managerBean.getCoordinationService().removeAliasBinding(alias);
-      }
+      managerBean.getCoordinationService().removeAliasBinding(managerBean.getSelectedPage(), name);
       return resetSelection();
    }
 
    public String rename(String oldName, String newName) throws IllegalCoordinationException
    {
-      DisplayAliasBinding binding = displayAliasBindings.get(oldName);
-      if (binding != null)
-      {
-         AliasBindingInfo alias = binding.getAlias();
-         managerBean.getCoordinationService().renameAliasBinding(alias, newName);
-      }
+      managerBean.getCoordinationService().renameAliasBinding(managerBean.getSelectedPage(), oldName, newName);
       return resetSelection();
    }
 
    public void extractInfoFrom(Window window, PortletInfo info)
    {
-       // retrieve metadata on public render parameters
+      // retrieve metadata on public render parameters
       NavigationInfo navInfo = info.getNavigation();
       for (ParameterInfo pInfo : navInfo.getPublicParameters())
       {
@@ -220,7 +210,7 @@
 
       public int compareTo(Object o)
       {
-         DisplayAliasBinding other = (DisplayAliasBinding) o;
+         DisplayAliasBinding other = (DisplayAliasBinding)o;
          return getName().compareTo(other.getName());
       }
    }