Author: taylor Date: Tue Jan 5 00:24:44 2010 New Revision: 895837 URL: http://svn.apache.org/viewvc?rev=895837&view=rev Log: JS2-1084 resurrect per user positioning
Modified: portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/layout/impl/PortletPlacementContextImpl.java portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/services/rest/PageLayoutService.java portals/jetspeed-2/portal/trunk/jetspeed-api/src/main/java/org/apache/jetspeed/layout/PortletPlacementContext.java Modified: portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/layout/impl/PortletPlacementContextImpl.java URL: http://svn.apache.org/viewvc/portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/layout/impl/PortletPlacementContextImpl.java?rev=895837&r1=895836&r2=895837&view=diff ============================================================================== --- portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/layout/impl/PortletPlacementContextImpl.java (original) +++ portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/layout/impl/PortletPlacementContextImpl.java Tue Jan 5 00:24:44 2010 @@ -24,6 +24,7 @@ import org.apache.jetspeed.components.portletregistry.PortletRegistry; import org.apache.jetspeed.layout.Coordinate; +import org.apache.jetspeed.layout.PageLayoutComponent; import org.apache.jetspeed.layout.PortletPlacementException; import org.apache.jetspeed.layout.PortletPlacementContext; import org.apache.jetspeed.om.page.ContentFragment; @@ -423,10 +424,17 @@ */ public ContentPage syncPageFragments() { - syncFragments( true, -1 ); + syncFragments(true, -1, null, null); //debugFragments( "syncPage" ); return this.page; } + + public ContentPage syncPageFragments(String scope, String scopeValue) + { + syncFragments(true, -1, scope, scopeValue); + //debugFragments( "syncPage" ); + return this.page; + } protected int getLatestColumn( Coordinate coordinate ) { @@ -451,7 +459,7 @@ return row; } - protected void syncFragments( boolean updateFragmentObjects, int onlyForColumnIndex ) + protected void syncFragments(boolean updateFragmentObjects, int onlyForColumnIndex, String scope, String scopeValue) { for ( int colIndex = 0; colIndex < this.columnsList.length; colIndex++ ) { @@ -475,7 +483,7 @@ } if ( updateFragmentObjects ) { - fragment.updateRowColumn(rowIndex, colIndex); + fragment.updateRowColumn(rowIndex, colIndex, scope, scopeValue); } } } @@ -613,7 +621,7 @@ CoordinateImpl coordinate = new CoordinateImpl( newCol, newRow ); this.fragmentCoordinateMap.put( fragment.getId(), coordinate ); this.fragmentMap.put( fragment.getId(), fragment ); - syncFragments( false, newCol ); + syncFragments(false, newCol, PageLayoutComponent.USER_PROPERTY_SCOPE, null); } else { @@ -638,9 +646,9 @@ this.fragmentMap.put( fragment.getId(), fragment ); - syncFragments( false, currentCol ); + syncFragments(false, currentCol, PageLayoutComponent.USER_PROPERTY_SCOPE, null); if ( columnChanged ) - syncFragments( false, newCol ); + syncFragments(false, newCol, PageLayoutComponent.USER_PROPERTY_SCOPE, null); } } return (Coordinate)this.fragmentCoordinateMap.get( fragment.getId() ); @@ -681,9 +689,9 @@ this.fragmentMap.put( fragment.getId(), fragment ); - syncFragments( false, currentCol ); + syncFragments(false, currentCol, PageLayoutComponent.USER_PROPERTY_SCOPE, null); if ( newCol != currentCol ) - syncFragments( false, newCol ); + syncFragments(false, newCol, PageLayoutComponent.USER_PROPERTY_SCOPE, null); } } return (Coordinate)this.fragmentCoordinateMap.get( fragment.getId() ); @@ -730,7 +738,7 @@ this.fragmentCoordinateMap.remove( fragment.getId() ); this.fragmentMap.remove( fragment.getId() ); - syncFragments( false, currentCol ); + syncFragments(false, currentCol, null, null); return currentCoordinate; } Modified: portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/services/rest/PageLayoutService.java URL: http://svn.apache.org/viewvc/portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/services/rest/PageLayoutService.java?rev=895837&r1=895836&r2=895837&view=diff ============================================================================== --- portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/services/rest/PageLayoutService.java (original) +++ portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/services/rest/PageLayoutService.java Tue Jan 5 00:24:44 2010 @@ -315,7 +315,7 @@ } // synchronize back to the page layout root fragment - contentPage = ppc.syncPageFragments(); + contentPage = ppc.syncPageFragments(PageLayoutComponent.USER_PROPERTY_SCOPE, null); } catch (PortletPlacementException e) { @@ -363,7 +363,7 @@ } // synchronize back to the page layout root fragment - contentPage = ppc.syncPageFragments(); + contentPage = ppc.syncPageFragments(PageLayoutComponent.USER_PROPERTY_SCOPE, null); } catch (Exception e) { @@ -383,9 +383,8 @@ { if (layoutFragmentId != null && layoutFragmentId.equals("detach")) { - // first time detach, need to reorder + //pageLayoutComponent.moveFragment(contentPage, layoutFragmentId, fromFragmentId, toFragmentId) } - pageLayoutComponent.updatePosition(contentFragment, posX, posY, posZ, posWidth, posHeight, PageLayoutComponent.USER_PROPERTY_SCOPE, null); pageLayoutComponent.updateStateMode(contentFragment, JetspeedActions.DETACH, null, PageLayoutComponent.USER_PROPERTY_SCOPE, null); } Modified: portals/jetspeed-2/portal/trunk/jetspeed-api/src/main/java/org/apache/jetspeed/layout/PortletPlacementContext.java URL: http://svn.apache.org/viewvc/portals/jetspeed-2/portal/trunk/jetspeed-api/src/main/java/org/apache/jetspeed/layout/PortletPlacementContext.java?rev=895837&r1=895836&r2=895837&view=diff ============================================================================== --- portals/jetspeed-2/portal/trunk/jetspeed-api/src/main/java/org/apache/jetspeed/layout/PortletPlacementContext.java (original) +++ portals/jetspeed-2/portal/trunk/jetspeed-api/src/main/java/org/apache/jetspeed/layout/PortletPlacementContext.java Tue Jan 5 00:24:44 2010 @@ -153,8 +153,18 @@ * Takes the internal portlet placement state and writes it back * out to the root fragment for the managed page layout. * + * @param scope the name of the property scope to update + * @param scopeValue the scope discriminator value, (unless scope is GLOBAL + * or USER where the default user name is used if null) * @return the managed page layout with updated fragment state. */ + public ContentPage syncPageFragments(String scope, String scopeValue); + + /** + * Takes the internal portlet placement state and writes it back + * out to the root fragment for the managed page layout. + * + * @return the managed page layout with updated fragment state. + */ public ContentPage syncPageFragments(); - } --------------------------------------------------------------------- To unsubscribe, e-mail: jetspeed-dev-unsubscr...@portals.apache.org For additional commands, e-mail: jetspeed-dev-h...@portals.apache.org