GeoServer

DescribeResponse doesn't support booleans

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.3.0-beta
  • Fix Version/s: 1.3.0 RC6
  • Component/s: WFS
  • Labels:
    None
  • Number of attachments :
    3

Description

A call to the WFS getDescribeFeature service, for a FeatureType
with no explicit schema defined, ends up calling:
String org.vfny.geoserver.responses.wfs.DescribeResponse.generateFromSchema(FeatureType schema)

that calls:
org.geotools.gml.producer.FeatureTypeTransformer.transform(schema, writer);

that in the end calls:
org.geotools.gml.producer.FeatureTypeTransformer$FeatureTypeTranslator.encode(AttributeType attribute)

This method is missing support for the Boolean type (and maybe for others), so an excpetion is thrown.
I fixed this bug (for Boolean only), but I'm unsure if the org.geotools.gml.producer package is to be replaced by something better altogether or will stay with us a little more.

Bye
Paolo Rizzi

  1. Java Source File
    DescribeResponse.java
    27/Jun/05 5:43 AM
    20 kB
    Paolo Rizzi
  2. Java Source File
    DescribeResponse.java
    27/Jun/05 5:40 AM
    20 kB
    Paolo Rizzi
  3. Java Source File
    FeatureTypeTransformer.java
    27/Jun/05 5:40 AM
    19 kB
    Paolo Rizzi

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
dblasby added a comment -

Paolo, could you commit the boolean fix?

Show
dblasby added a comment - Paolo, could you commit the boolean fix?
Hide
Permalink
Paolo Rizzi added a comment -

I''m very sorry!!! It seems that I overlooked that last comment about committing my fix!!!
And it was two months ago!!! Sorry again...
I'll do it as first thing on Monday!!!

Bye
Paolo Rizzi

Show
Paolo Rizzi added a comment - I''m very sorry!!! It seems that I overlooked that last comment about committing my fix!!! And it was two months ago!!! Sorry again... I'll do it as first thing on Monday!!! Bye Paolo Rizzi
Hide
Permalink
Paolo Rizzi added a comment -

I don't have commit rights, so here are the two fixed files (fixed lines marked as "SISfixed").
FeatureTypeTransformer.java is really from GeoTools, so it would be better fixed there, not in GeoServer.
DescribeResponse.java has no errors, I simply modified it to use it's own package version of FeatureTypeTransformer, instead of the one inside GeoTools.
As soon as FeatureTypeTransformer will be fixed inside GeoTools, DescribeResponse may as it was.

Bye
Paolo Rizzi

Show
Paolo Rizzi added a comment - I don't have commit rights, so here are the two fixed files (fixed lines marked as "SISfixed"). FeatureTypeTransformer.java is really from GeoTools, so it would be better fixed there, not in GeoServer. DescribeResponse.java has no errors, I simply modified it to use it's own package version of FeatureTypeTransformer, instead of the one inside GeoTools. As soon as FeatureTypeTransformer will be fixed inside GeoTools, DescribeResponse may as it was. Bye Paolo Rizzi
Hide
Permalink
Paolo Rizzi added a comment -

I'm sorry, there was a few HTML lines inside the first version of the attached DescribeResponse.java

Bye
Paolo Rizzi

Show
Paolo Rizzi added a comment - I'm sorry, there was a few HTML lines inside the first version of the attached DescribeResponse.java Bye Paolo Rizzi
Hide
Permalink
Brent Owens added a comment -

Moved to 1.3.0-RC6

Show
Brent Owens added a comment - Moved to 1.3.0-RC6
Hide
Permalink
Justin Deoliveira added a comment -

apply changes at geotools level

Show
Justin Deoliveira added a comment - apply changes at geotools level
Hide
Permalink
Andrea Aime added a comment -

These issue has been in resolved state for at least one month (quite a bit, a lot more than one month). Batch transitioning them to closed state

Show
Andrea Aime added a comment - These issue has been in resolved state for at least one month (quite a bit, a lot more than one month). Batch transitioning them to closed state

People

  • Assignee:
    Justin Deoliveira
    Reporter:
    Paolo Rizzi
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: