WFS Filter : DWITHIN does not respect the units parameter

[assefa]

Reported by Norm

When executing the requeste with a buffer in degre, I get the corrct result. But if I use a buffer in Meter, the results are incorrect

Requete buffer en degre # Verify DWITHIN Result: Sydney # RUN_PARMS: wfs_filter_dwithin.xml [MAPSERV] QUERY_STRING="map=[MAPFILE]&SERVICE=WFS&VERSION=1.0.0&REQUEST=GetFeature&TYPENAME=popplace&FILTER=<Filter><DWithin><PropertyName>Geometry</PropertyName><gml:Point><gml:coordinates>-60.18,46.10</gml:coordinates></gml:Point><Distance units='dd'>0.05</Distance></DWithin></Filter>" > [RESULT]

Requete buffer en metre # Verify DWITHIN Result: Sydney # RUN_PARMS: wfs_filter_dwithin.xml [MAPSERV] QUERY_STRING="map=[MAPFILE]&SERVICE=WFS&VERSION=1.0.0&REQUEST=GetFeature&TYPENAME=popplace&FILTER=<Filter><DWithin><PropertyName>Geometry</PropertyName><gml:Point><gml:coordinates>-60.18,46.10</gml:coordinates></gml:Point><Distance units='meter'>1000</Distance></DWithin></Filter>" > [RESULT]

[assefa]

committed in r 7739

add msauto a test r7740.

Marking as fixed. Please reopen if there is an issue.

[tomkralidis]

Assefa:

I get msautotest errors (after svn up, building) with msautotest against wfs_filter_dwithin_units.xml. See diff at ​http://devgeo.cciw.ca/ms_tmp/wfs_filter_dwithin_units.diff

[assefa]

I can not reproduce this locally. Tom, the diff seems to indicate that the units is not used at all. Norm can you confirm this with latest mapserver svn ?

[tomkralidis]

Norm: see comment:5

[nsavard]

I'll check.

[nsavard]

I got the same results as Tom.

[assefa]

There was memory corruption fixed in r7748.

Tom, Norm please test if possible.

Tamas signaled today a problem with msautotest re this specific test. I am hopeful this fix would eliminate the problem. I will send an e-mail to him to do a test with the fix.

[tomkralidis]

Assefa: I just tested this and it works (the only msautotest that is still broken in wfs_filter.map is test wfs_filter_equals_poly.xml).

Feel free to close unless others find issues.

[assefa]

Thanks Tom,

I have added Tamas in cc on this bug. If his tests passes, I will close the bug.

[nsavard]

Assefa: It works for me as well.

[assefa]

Thx. closing.