[jbossseam-issues] [JBoss JIRA] Closed: (JBSEAM-308) DataModel instance changes even though the backing list did not when using PAGE scope

DataModel instance changes even though the backing list did not when using PAGE scope ------------------------------------------------------------------------------------- Key: JBSEAM-308 URL: http://jira.jboss.com/jira/browse/JBSEAM-308 Project: JBoss Seam Issue Type: Bug Components: Core Affects Versions: 1.0 beta 1, 1.0 beta 2, 1.0, 1.0.1, 1.1.0.BETA1 Reporter: Chris Rudd Assigned To: Gavin King Fix For: 1.3.0.ALPHA The outjectDataModel code attempts to determine if the value changed and therefore needs to be re outjected. When not using PAGE scope this works fine. When PAGE scope is used it always reports dirty, which causes the value to be outjected. The issue is that if the backing value didnt change, it still gets re-wrapped and outjected instead of just re-outjected. I noticed this when working with an @DataModel, the UIData components im using cache the DataModel to ensure that it doesnt change between the updateValues and the invokeApplication phases. So hwat ends up happening is that in the updateValues phase a property gets set on my component, causing the datamodel to be re-outjected, since the UIData has already cached the original value, it never picks up the "new" one. When I get to the invokeApplication phase and it calls an action method on the component the datamodel that the UIData updated the rowIndex on isnt the same one that seam pulls the rowIndex from to inject the @DataModelSelection. I propose the following fix to the 1.0.0 GA code ( removed PAGE scope check from dirty assignment, added else clause to if(dirty) ): private void outjectDataModelList(String name, Object list, ScopeType scope, DataBinder wrapper) { Context context = getDataModelContext(scope); Object existingDataModel = context.get(name); boolean dirty = existingDataModel == null || !wrapper.getWrappedData(existingDataModel).equals(list); //TODO: delegate to the wrapper to determine equality if ( dirty ) { if ( list != null ) { context.set( name, wrapper.wrap(list) ); } else { context.remove( name ); } } // If page scope, outject the existingDataModel even if there was no change else if( scope==ScopeType.PAGE ) { context.set( name, existingDataModel ); } }