On 05/09/2009, at 3:56 AM, Justin Deoliveira wrote: > 2) How should complex types with simple content be represented? Jody > has > said/suggested that this be a property with the name "value", which is > marked as inline. I think that can work... there will just have to > be a > another special case in the encoding logic which will use the "value" > property when it detects the complex type simple content case. > > However the feature model is incomplete in this regard. > ComplexAttribute.isInline() is never set (there is no way to set it) > and > it always returns false.
Hi Justin: Can we put together exactly what needs to get fixed here and I will update geoapi and issue another milestone. Right now I think the isInline() method is in the wrong spot.... - ComplexAttribute.isInline() is good for handling choice and repeating groups. I cannot see how to use this to handle a complex type extending simple type. - Move isInline() to AttributeType or AttributeDescriptor? I think either works and between the two AttributeDescriptor.isInline allows for more reuse. - Or add a ComplexType.getInlineDescriptor() ComplexAttribute.getInline () mirroring how FeatureType.getGeometryDescriptor() Comments: - I think I find Gabriel's option the most clear; it reflects that we can only every inline one simple attribute. - My initial preference is to move the isInline method up to AttributeDescriptor; as it is the smallest change that will do the job Concerns - I remember it being valid XML to have multiple simple attributes separated by spaces when you define an array. Jody ------------------------------------------------------------------------------ Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day trial. Simplify your report design, integration and deployment - and focus on what you do best, core application coding. Discover what's new with Crystal Reports now. http://p.sf.net/sfu/bobj-july _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
