Hi, Please make sure you add unit tests when fixing things like this...
Thanks, Florent On Thu, Apr 22, 2010 at 7:07 PM, dcaruana <[email protected]> wrote: > Author: dcaruana > Date: Thu Apr 22 17:07:46 2010 > New Revision: 936966 > > URL: http://svn.apache.org/viewvc?rev=936966&view=rev > Log: > Fix issue where paging iterator did not reset total number of items after > fetching next page of results. > > Modified: > > incubator/chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-impl/src/main/java/org/apache/chemistry/opencmis/client/runtime/util/DefaultPagingIterator.java > > Modified: > incubator/chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-impl/src/main/java/org/apache/chemistry/opencmis/client/runtime/util/DefaultPagingIterator.java > URL: > http://svn.apache.org/viewvc/incubator/chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-impl/src/main/java/org/apache/chemistry/opencmis/client/runtime/util/DefaultPagingIterator.java?rev=936966&r1=936965&r2=936966&view=diff > ============================================================================== > --- > incubator/chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-impl/src/main/java/org/apache/chemistry/opencmis/client/runtime/util/DefaultPagingIterator.java > (original) > +++ > incubator/chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-impl/src/main/java/org/apache/chemistry/opencmis/client/runtime/util/DefaultPagingIterator.java > Thu Apr 22 17:07:46 2010 > @@ -119,6 +119,7 @@ public class DefaultPagingIterator<T> im > skipCount += skipOffset; > skipOffset = 0; > this.page = pageFetch.fetchPage(skipCount); > + this.totalItems = null; > currentPage = this.page; > if (currentPage != null) { > items = currentPage.getPage(); > > > -- Florent Guillaume, Director of R&D, Nuxeo Open Source, Java EE based, Enterprise Content Management (ECM) http://www.nuxeo.com http://www.nuxeo.org +33 1 40 33 79 87
