Author: chris.laprun(a)jboss.com Date: 2008-08-31 16:32:26 -0400 (Sun, 31 Aug 2008) New Revision: 11772 Modified: branches/JBoss_Portal_Branch_2_7/core-admin/src/main/org/jboss/portal/core/admin/ui/coordination/CoordinationManagerBean.java branches/JBoss_Portal_Branch_2_7/core-admin/src/resources/portal-admin-war/jsf/editCoordination.xhtml Log: - Fixed improper selection of pages (should have better tested getSelectedPage). - Really avoid concanetation of events in ui:repeat. Modified: branches/JBoss_Portal_Branch_2_7/core-admin/src/main/org/jboss/portal/core/admin/ui/coordination/CoordinationManagerBean.java =================================================================== --- branches/JBoss_Portal_Branch_2_7/core-admin/src/main/org/jboss/portal/core/admin/ui/coordination/CoordinationManagerBean.java 2008-08-31 20:20:53 UTC (rev 11771) +++ branches/JBoss_Portal_Branch_2_7/core-admin/src/main/org/jboss/portal/core/admin/ui/coordination/CoordinationManagerBean.java 2008-08-31 20:32:26 UTC (rev 11772) @@ -256,9 +256,13 @@ { try { + // make sure we use the proper selected page + selectedObjectId = pomb.getSelectedPortalObjectId(); + // Extract metadata only on window children and only if needed if (isExplicitEventsUsed() || isExplicitParametersUsed()) { + eventManager.loadInfoFrom(selectedPage); windowManager.loadInfoFrom(selectedPage); Modified: branches/JBoss_Portal_Branch_2_7/core-admin/src/resources/portal-admin-war/jsf/editCoordination.xhtml =================================================================== --- branches/JBoss_Portal_Branch_2_7/core-admin/src/resources/portal-admin-war/jsf/editCoordination.xhtml 2008-08-31 20:20:53 UTC (rev 11771) +++ branches/JBoss_Portal_Branch_2_7/core-admin/src/resources/portal-admin-war/jsf/editCoordination.xhtml 2008-08-31 20:32:26 UTC (rev 11772) @@ -190,14 +190,18 @@ <f:facet name="header"> <h:outputText value="#{bundle.COORDINATION_EVENT_EXISTING_SOURCE_EVENTS}"/> </f:facet> - <ui:repeat var="event" value="#{wiring.sourceEvents}">#{event} </ui:repeat> + <ui:repeat var="event" value="#{wiring.sourceEvents}"> + #{event} + </ui:repeat> </h:column> <h:column> <f:facet name="header"> <h:outputText value="#{bundle.COORDINATION_EVENT_EXISTING_DESTINATION_EVENTS}"/> </f:facet> - <ui:repeat var="event" value="#{wiring.destinationEvents}">#{event} </ui:repeat> + <ui:repeat var="event" value="#{wiring.destinationEvents}"> + #{event} + </ui:repeat> </h:column> <h:column>