Here is a pull request with the fix:
https://github.com/geotools/geotools/pull/669

Torben

On Wed, Jan 7, 2015 at 2:31 PM, Torben Barsballe <
[email protected]> wrote:

> When testing the gt-property-ng migration in geoserver (Specifically,
> org.geoserver.wfs.TransactionTest.testDoubleDelete), I ran into another
> error in ContentFeatureSource:
>
> On line 390
> <https://github.com/geotools/geotools/blob/master/modules/library/data/src/main/java/org/geotools/data/store/ContentFeatureSource.java#L390>
> of getBounds, we build a list of modified fids based on the diff. I found
> that if one of these features has been deleted, then this will fail with a
> NullPointerException on line 395. Fortunately, the keySet of
> diff.getModified() appears to be the fid, so we can use this instead.
>
> I am currently testing this change.
>
> Torben.
>
------------------------------------------------------------------------------
Dive into the World of Parallel Programming! The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to