Query: The matched objects excluded by layer filter are listed in Query result.

[jennyhe]

Report from Autodesk QA

Steps:

1. Load the attached package.
2. Open the flexible web layout by IE.
3. Query -> Property Filter -> Autogenerated_SDF_ID, Equal, 11 -> Execute query

Results: One record named "11" are listed in Results panel.

Expected results: Query should do against the layer but not the data source.

[christinebao]

The indeed of this defect is: Query widget only considers the query condition inside the widget, but ignore the query condition of the layer.
For example, if the layer has a filter "Autogenerated_SDF_ID < 2", the layer in map should only display one feature with Autogenerated_SDF_ID = 1. When user input Autogenerated_SDF_ID=3 as query widget's condition, there should be no result matching because the real condition should be "Autogenerated_SDF_ID < 2 AND Autogenerated_SDF_ID = 3". However currently only "Autogenerated_SDF_ID = 3" is set as query condition, so another feature (Autogenerated_SDF_ID = 3) besides the original one (Autogenerated_SDF_ID = 1) shows in map.

Tech diagnosis:

1. MgLayerBase should have an API to get the filter.
   
2. query.php function Execute() should consider the filter if exist.

Based on the above tech diagnosis, attach patch for fixing this defect.
There are two patches:

1. GetFilterAPI.patch is to add an API in MgLayerBase to get the filter when parsing layer definition model.
   
2. QueryFilter.patch is to consider layer filter if exists.

Question:
As you can see, I added an new API GetFilter() to get the filter of layer definition. Is there any other way to approach this without a new API? I see a function in MgSelectionBase for generate filter, however it is for a difference usage, isn't it?

STRING GenerateFilter(MgLayerBase* layer, CREFSTRING className);

Thank you for reviewing!

[chrisclaydon]

The patches look good Christine. Please remove the /// false otherwise. from the method documentation in LayerBase.h - it look like it is just left over from a cut and paste.

[christinebao]

http://trac.osgeo.org/fusion/attachment/ticket/327/GetFilterAPI.patch is submitted in MapGuide code stream. The related ticket: ​https://trac.osgeo.org/mapguide/ticket/1154.

[christinebao]

Submit the patches for fixing this defect.

[christinebao]

MgMap::Open(resourceService, mapName) needs to serialize/deserialize, so m_filter should be serialized/deserialized in MgLayer, otherwise browser can't get the value.

Attach patch http://trac.osgeo.org/fusion/attachment/ticket/327/SerializeFilter.patch for review.

This patch is tested by the above ticket, and Query widget can get the expected filter.