[fusion-commits] r2549 - sandbox/adsk/2.3r/layers/MapGuide/php

[svn_fusion at osgeo.org]

Author: liuar
Date: 2012-07-16 23:23:39 -0700 (Mon, 16 Jul 2012)
New Revision: 2549

Modified:
   sandbox/adsk/2.3r/layers/MapGuide/php/GetSelectionProperties.php
Log:
On behalf of: Andy Zhang

We should add geometry name to query property list, otherwise the selection operation maybe fail. This change makes fusion code be consistent with ASP code.

Modified: sandbox/adsk/2.3r/layers/MapGuide/php/GetSelectionProperties.php
===================================================================
--- sandbox/adsk/2.3r/layers/MapGuide/php/GetSelectionProperties.php	2012-07-17 02:03:03 UTC (rev 2548)
+++ sandbox/adsk/2.3r/layers/MapGuide/php/GetSelectionProperties.php	2012-07-17 06:23:39 UTC (rev 2549)
@@ -81,16 +81,18 @@
             //the one causing a problem when the properies are limited ?
             if (isset($_SESSION['property_mappings']) && isset($_SESSION['property_mappings'][$oLayer->GetObjectId()])) {
                 $mappings = $_SESSION['property_mappings'][$oLayer->GetObjectId()];
-                if (0 && count($mappings) > 0) {
+                if (count($mappings) > 0) {
                     foreach($mappings as $name => $value) {
                         $queryOptions->AddFeatureProperty($name);
                         //echo "$name $value <br>\n";
                     }
-                    $geomName = $oLayer->GetFeatureGeometryName();
-                    $queryOptions->AddFeatureProperty($geomName);
                 }
             }
 
+            //Add geometry property in all cases.
+            $geomName = $oLayer->GetFeatureGeometryName();
+            $queryOptions->AddFeatureProperty($geomName);
+
             $filter = $selection->GenerateFilter($oLayer, $class);
             $queryOptions->SetFilter($filter);
             $featureReader = $featureService->SelectFeatures($featureResId, $class, $queryOptions);