[fusion-commits] r2065 - sandbox/adsk/2.2gp/widgets/Theme

[svn_fusion at osgeo.org]

Author: liuar
Date: 2010-02-08 21:57:05 -0500 (Mon, 08 Feb 2010)
New Revision: 2065

Modified:
   sandbox/adsk/2.2gp/widgets/Theme/thememain.php
Log:
Port the fix of ticket #359 Theme on raster layer cause an error to this branch.

[Analysis]
For layers which don't contain the geometry property, the layer's name will not be record in layerNames in theme.php
So in thememain.php the layerSelect.value will be null, and the layerInfo request will fail.


[Resolution/Workaround]
To resolve this problem, I checked the layerSelect.value, if it's null, just return.

Modified: sandbox/adsk/2.2gp/widgets/Theme/thememain.php
===================================================================
--- sandbox/adsk/2.2gp/widgets/Theme/thememain.php	2010-02-09 02:56:28 UTC (rev 2064)
+++ sandbox/adsk/2.2gp/widgets/Theme/thememain.php	2010-02-09 02:57:05 UTC (rev 2065)
@@ -124,6 +124,11 @@
         function OnLayerChange()
         {
             var layerSelect = document.getElementById("layerSelect");
+
+            if(layerSelect.value == ""){
+                return;
+            }
+
             var propertySelect = document.getElementById("propertySelect");
             var scaleSelect = document.getElementById("scaleSelect");