Gabriel Roldán wrote:
> On Monday 09 July 2007 20:00:14 Justin Deoliveira wrote:
>> Hi Gabriel,
>>
>> It seems that some code that did attribute name lookup in
>> DataUtilities.attributeNames is no longer going through the
>> PropertyAccessor interface when traversing a filter to pull out all
>> PropertyName's from it.
>>
>> There is a comment in the code sayign that it is part of GEOT-1192 which
>> is still in progress... so confused just looking for some explanation.
>>
>> As far as I know the replacement will cause all xpath and namespace
>> prefixed property names to fail in this manner.
> 
> I see.. well, I have to say that I've relied on the unit tests so it might be 
> worth if we add one for that case. Gonna run home now, but will take care of 
> restoring that functionality asap.
Ok cool... a lot of these issues don't come up until we run cite... a 
test case will be well worth it, working on one now..

I already have a patch which involves modifying FilterPropertyExtractor 
which I will submit. I was more wondering if there was an explicit 
reason for the change but it sounds like not.

Thanks Gabriel.

-Justin
> 
> Gabriel
> 
>> -Justin
> 
> 
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> Geotools-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
> 
> !DSPAM:4007,46927cb0116651961014482!
> 


-- 
Justin Deoliveira
The Open Planning Project
http://topp.openplans.org

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to