Hello Andrea,

sorry. I am working on the trunk.

Tim

-----Ursprüngliche Nachricht-----
Von: Andrea Aime [mailto:[EMAIL PROTECTED]
Gesendet: Dienstag, 16. Oktober 2007 15:10
An: Tim Englich
Cc: [email protected]
Betreff: Re: [Geotools-devel] FeatureTransformer writeBounds writes
Polygon instead of BOX


Tim Englich ha scritto:
> hello ,
> 
> I think there is an bug in FeatureTransformer.writeBounds.
> 
> It encodes an Polygon instead of an Envelope.
> 
> This causes an invalid WFS 1.0.0 response.
> 
> I changed the Code to
>  Envelope env = null;
>                 if (bounds != null){
>                       env = new Envelope(new Coordinate(bounds.getMinX(),
> bounds.getMinY()),new Coordinate(bounds.getMaxX(), bounds.getMaxY()));
>                 }
>                 geometryTranslator.encode(env, srsName);
> 
> Should I generate an Ticket or will anyone review this issue?
> I also can commit my changes.

This takes me by surprise. GeoServer uses that class and afaik generates
valid GML2. The code I see in FeatureTransformer.FeatureTranslator is:

public void writeBounds(Envelope bounds) {
             try {
                 String boundedBy = 
geometryTranslator.getDefaultPrefix() + ":"
                     + "boundedBy";

                 contentHandler.startElement("", "", boundedBy, NULL_ATTS);
                 geometryTranslator.encode(bounds, srsName);
                 contentHandler.endElement("", "", boundedBy);
             } catch (SAXException se) {
                 throw new RuntimeException(se);
             }
         }

This is on 2.4.x at least... maybe you're working on trunk?
Or maybe I'm missing something, I only had a very cursory look to the code.
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