Ok I will change it back to the Code you posted.
And commit it if there a no veto from the developer-list until monday.

-----Ursprüngliche Nachricht-----
Von: Andrea Aime [mailto:[EMAIL PROTECTED]
Gesendet: Freitag, 19. Oktober 2007 17:02
An: Tim Englich
Cc: [email protected]
Betreff: Re: Little Change in OracledataStore


Tim Englich ha scritto:
> Hello,
> 
> I have got a little change in the Oracle DataStore on trunk.
> I put the Instantiation of the CRS back to the determineCRS Method so that
I
> can override the Method again to instantiate an CRS when no EPSG-Code is
> given.

Hmmm... the code on 2.4.x is first trying to interpret the CRS
by using decode("EPSG:xxxx") and then, it that fail, it falls back
on determineCRS() which in fact you can override...

try {
        srid = determineSRID( tableName, columnName );
        try {
            crs = CRS.decode("EPSG:" + srid);
        } catch(Exception e) {
            // decode failed, let's try the internal authority
            crs = determineCRS(srid);
        }
} catch (Exception e) {
        LOGGER.warning( "Could not map SRID "+srid+" to CRS:"+e );
}

I thought trunk was the same? If not, can you do the change but keep
it consistent with 2.4.x? That is, what prevents you from
overriding determineCRS instead?
Cheers
Andrea

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to