Re: [Geotools-devel] feature collection cleanup -> 9.0-M0-SNAPSHOT

2012-11-03 Thread Jody Garnett
Thanks Justin, I just wen rover them now and they look great, and have merged them into the main clean up branch. Sorry I took the ValueCollectionType / PropertyValueCollection change in the wrong direction - it drove me nuts figuring out what it was doing. -- Jody Garnett On Sunday, 4 Nov

Re: [Geotools-devel] feature collection cleanup -> 9.0-M0-SNAPSHOT

2012-11-03 Thread Justin Deoliveira
Ok, I had to make a few changes on top of yours to make GeoServer happy with these changes. I pushed up them to a barnch in my repo. If they look ok i will merge them into the main cleanup branch. https://github.com/geotools/geotools/pull/46 On Fri, Nov 2, 2012 at 5:31 PM, Jody Garnett wrote:

Re: [Geotools-devel] feature collection cleanup -> 9.0-M0-SNAPSHOT

2012-11-02 Thread Jody Garnett
Thanks Justin! If it helps there is a BaseFeatureCollection that just requires a features() method, which is great for wrappers that formally supplied an iteration() method. I am on IRC on and off this weekend if needed - trying to release a milestone release (not even sure where to start with th

Re: [Geotools-devel] feature collection cleanup -> 9.0-M0-SNAPSHOT

2012-11-02 Thread Justin Deoliveira
I took an initial run at updating GeoServer for the new api... for the most part it is straight forward but there seems to be one tricky bit when it comes to some feature collection wrappers that are used for security. MIght not be a problem but need to understand what they are being used for, and

Re: [Geotools-devel] feature collection cleanup -> 9.0-M0-SNAPSHOT

2012-11-02 Thread Jody Garnett
On Thursday, 1 November 2012 at 8:38 PM, Jody Garnett wrote: > I have deployed 9.0-M0-SNAPSHOT as preview for anyone wanting to try this out > locally. > > It will also help work on the GeoServer and uDig migration branches, indeed > it may allow me to do a pull request for uDig early. - pull r