To which idea is this +1 directed :-)

I am getting a bit nervous watching PropertyName being used as a placeholder 
for a generic "qualified" XPath. Still I guess I understand why it is taking 
that shape.

So is your +1 for the idea of:

Query.getAttributes(): List<PropertyName>  // property name may refer to a 
complete xpath

With this in mind getPropertyNames and setPropertyNames would then become a 
shortcut methods for the above List.

Jody

On 23/12/2010, at 11:51 PM, Gabriel Roldán wrote:

> +1
> 
> On Mon, 2010-12-20 at 17:23 +1100, Jody Garnett wrote:
>> I though about this; like the idea of using a List<Name> from an ease
>> of use / consistency point of view. However I think I may of slightly
>> missed the point ...
>> 
>> 
>> In the code example I tried to show the use of an xpath for
>> propertyNames so you could indicate how much of contained data
>> structures you wanted filled out.
>> 
>> 
>> query.setPropertyNames( new 
>> String[]{"foo:x","foo:y","foo:description/foo:label"});
>> 
>> 
>> The idea was that a "description" object would be created; but only
>> the "label" field filled in with a non null result.
>> This kind of thing we cannot obtain using a List<Name>; but would be
>> possible with a List<PropertyName>
>> 
>> 
>> Jody
>> 
>> 
>> On 20/12/2010, at 4:10 PM, Niels wrote:
>> 
>>> On 18/12/10 17:12, Andrea Aime wrote:
>>>> On Fri, Dec 17, 2010 at 9:03 AM, Niels<niels.charl...@csiro.au>
>>>> wrote:
>>>>> I think you do need a NameSpace context in your query, because
>>>>> you cannot
>>>>> assume that the namespace prefixes in the property names of the
>>>>> query are
>>>>> the same as the ones in the mapping or in the datastore or
>>>>> anywhere else.
>>>>> The thing about namespace prefixes is that you can never make
>>>>> those
>>>>> assumptions.
>>>>> Another solution for Query would be to provide a list  of
>>>>> PropertyNames
>>>>> rather than strings (which is what I suggested earlier I think?)
>>>>> but I guess
>>>>> that would break too much existing code ?
>>>> Lucklily Query is not an interface anymore so it _would_ be
>>>> possible to
>>>> just add PropertyName (or Name) support and have it fall back on
>>>> plain
>>>> strings when the plain string methods are called. The new methods
>>>> would be used only by code that know about complex features.
>>> 
>>> I think that is an excellent idea actually... Have two getters and 
>>> settesr for PropertyNames property in Query: one with string[] and
>>> one 
>>> with PropertyName[]. The inner representation of the property names
>>> in 
>>> the class could be PropertyName[], but when the string[] getter and 
>>> setter are used, conversion is done automatically.
>>> 
>>> I think that would be even better than having a
>>> setNamespaceContext() in 
>>> Query!
>>> 
>>> Niels
>>> 
>>> ------------------------------------------------------------------------------
>>> Lotusphere 2011
>>> Register now for Lotusphere 2011 and learn how
>>> to connect the dots, take your collaborative environment
>>> to the next level, and enter the era of Social Business.
>>> http://p.sf.net/sfu/lotusphere-d2d
>>> _______________________________________________
>>> Geotools-devel mailing list
>>> Geotools-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>> 
>> 
>> 
>> 
>> 
>> On 20/12/2010, at 4:10 PM, Niels wrote:
>> 
>>> On 18/12/10 17:12, Andrea Aime wrote:
>>>> On Fri, Dec 17, 2010 at 9:03 AM, Niels<niels.charl...@csiro.au>
>>>> wrote:
>>>>> I think you do need a NameSpace context in your query, because
>>>>> you cannot
>>>>> assume that the namespace prefixes in the property names of the
>>>>> query are
>>>>> the same as the ones in the mapping or in the datastore or
>>>>> anywhere else.
>>>>> The thing about namespace prefixes is that you can never make
>>>>> those
>>>>> assumptions.
>>>>> Another solution for Query would be to provide a list  of
>>>>> PropertyNames
>>>>> rather than strings (which is what I suggested earlier I think?)
>>>>> but I guess
>>>>> that would break too much existing code ?
>>>> Lucklily Query is not an interface anymore so it _would_ be
>>>> possible to
>>>> just add PropertyName (or Name) support and have it fall back on
>>>> plain
>>>> strings when the plain string methods are called. The new methods
>>>> would be used only by code that know about complex features.
>>> 
>>> I think that is an excellent idea actually... Have two getters and 
>>> settesr for PropertyNames property in Query: one with string[] and
>>> one 
>>> with PropertyName[]. The inner representation of the property names
>>> in 
>>> the class could be PropertyName[], but when the string[] getter and 
>>> setter are used, conversion is done automatically.
>>> 
>>> I think that would be even better than having a
>>> setNamespaceContext() in 
>>> Query!
>>> 
>>> Niels
>>> 
>>> ------------------------------------------------------------------------------
>>> Lotusphere 2011
>>> Register now for Lotusphere 2011 and learn how
>>> to connect the dots, take your collaborative environment
>>> to the next level, and enter the era of Social Business.
>>> http://p.sf.net/sfu/lotusphere-d2d
>>> _______________________________________________
>>> Geotools-devel mailing list
>>> Geotools-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>> 
>> 
>> ------------------------------------------------------------------------------
>> Lotusphere 2011
>> Register now for Lotusphere 2011 and learn how
>> to connect the dots, take your collaborative environment
>> to the next level, and enter the era of Social Business.
>> http://p.sf.net/sfu/lotusphere-d2d
>> _______________________________________________ Geotools-devel mailing list 
>> Geotools-devel@lists.sourceforge.net 
>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
> 
> -- 
> Gabriel Roldan
> grol...@opengeo.org
> Expert service straight from the developers
> 


------------------------------------------------------------------------------
Learn how Oracle Real Application Clusters (RAC) One Node allows customers
to consolidate database storage, standardize their database environment, and, 
should the need arise, upgrade to a full multi-node Oracle RAC database 
without downtime or disruption
http://p.sf.net/sfu/oracle-sfdevnl
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to