incubator-graffito-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alexandru Popescu <the.mindstorm.mailingl...@gmail.com>
Subject Re: no nodetype specified in queries
Date Wed, 15 Feb 2006 21:05:20 GMT
#: Christophe Lombart changed the world a bit at a time by saying (astral date: 2/15/2006 10:07
PM) :#
> On 2/15/06, Alexandru Popescu <the.mindstorm.mailinglist@gmail.com> wrote:
>> Hi!
>>
>> Looking around to see what changes should be done to support polymorphic queries,
I have noticed
>> that even if the class is passed to the Filter the mapped node type for the corresponding
class is
>> not used to filter the nodes. This would imply that even if a class is mapped to
a specific node
>> type, using a query on a set of nodes that have some of the properties of that class
will return
>> results even if the node types do not correspond.
>>
>> Is this done on purpose?
> 
> No - see in the QueryManagerImpl the method buildJCRExpression. this
> method builds a JCR Expression corresponding to a Filter object. The
> node type is defined.I think this method has to be review if we want
> to support de inheritance/interface.
>

True... I missed that line. Indeed that is one of the places where I am looking to update.

>>
>> IMO the relation between class and its corresponding node type should be more strict.
And this is
>> not only for queries, but also for updates, fetching. Currently, the only time this
relation is used
>> is for insertion. My personal approach would be to have this relation made strong.
>>
>> What's your opinion?
>>
> 
> +1
> 
>> ./alex
>> --
>> .w( the_mindstorm )p.
>>
> 
> 
> --
> Best regards,
> 
> Christophe
> 

Thanks for confirmation.

./alex
--
.w( the_mindstorm )p.



Mime
View raw message