[ http://jira.codehaus.org/browse/GEOT-968?page=all ]

Andrea Aime reopened GEOT-968:
------------------------------

             
Nope, the patch has not been applied afaik.
Those fid mappers do not contain my fix:

http://svn.geotools.org/geotools/trunk/gt/plugin/postgis/src/org/geotools/data/postgis/fidmapper/PostgisFIDMapperFactory.java
http://svn.geotools.org/geotools/branches/2.2.x/plugin/postgis/src/org/geotools/data/postgis/fidmapper/PostgisFIDMapperFactory.java

The log shows a commit of yours and the modified files, but you have applied 
only geot-967 apparently (only changed an import?)

> Postgis fid mapper should not return oid fid mapper without checking database 
> version number
> --------------------------------------------------------------------------------------------
>
>                 Key: GEOT-968
>                 URL: http://jira.codehaus.org/browse/GEOT-968
>             Project: GeoTools
>          Issue Type: Bug
>          Components: data postgis
>    Affects Versions: 2.3.M0, 2.2.0
>            Reporter: Andrea Aime
>         Assigned To: Justin Deoliveira
>             Fix For: 2.3, 2.2.1
>
>         Attachments: PostgisFIDMapperFactory.java.patch
>
>
> Postgis fid mapper factory returns a no pk fid mapper without checks on the 
> database version number, making queries fail afterwards because no oid is 
> available on postgres 8 onwards.
> The attached patch fixes the problems, and also logs an eventual exception 
> trying to get the major version number.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to