<snip>
Works for me. I admit a bit flaky... but i blame the OGC for there crazy
namespace schemes :).
> yup, I was kind of wanting to find a cleaner solution but if the
> following is ok for you I can go for it:
>
> Index: src/main/java/org/geotools/gml2/bindings/GML2ParsingUtils.java
> ===================================================================
> --- src/main/java/org/geotools/gml2/bindings/GML2ParsingUtils.java
> (revision 32695)
> +++ src/main/java/org/geotools/gml2/bindings/GML2ParsingUtils.java
> (working copy)
> @@ -251,9 +251,11 @@
> // create the type
>
> ftBuilder.minOccurs(min).maxOccurs(max).add(property.getName(), theClass);
>
> - //set the default geometry explicitly
> + //set the default geometry explicitly. Note we're comparing
> the GML namespace
> + //with String.startsWith to catch up on the GML 3.2
> namespace too, which is hacky.
> + final String propNamespace = property.getTargetNamespace();
> if (Geometry.class.isAssignableFrom(theClass)
> - &&
> !GML.NAMESPACE.equals(property.getTargetNamespace())) {
> + && (propNamespace == null ||
> !propNamespace.startsWith(GML.NAMESPACE))) {
> //only set if non-gml, we do this because of
> "gml:location",
> // we dont want that to be the default if the user has
> another
> // geometry attribute
>
>
>
> cheers,
> Gabriel
>
> PS: created http://jira.codehaus.org/browse/GEOT-2408
>>>
>>> Also, how can I proceed to commit the change in main for GEOT-2403
>>> without breaking the build while this is sorted out?
>>>
>>> Cheers,
>>>
>>> Gabriel.
>>>
>>>
>>>
>>>
>>>
>>
>>
>
--
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
------------------------------------------------------------------------------
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
