[fusion-commits] r1676 - in trunk: . layers/MapGuide layers/MapServer

[svn_fusion at osgeo.org]

Author: madair
Date: 2008-11-19 14:37:40 -0500 (Wed, 19 Nov 2008)
New Revision: 1676

Modified:
   trunk/build.xml
   trunk/layers/MapGuide/MapGuide.js
   trunk/layers/MapGuide/MapGuideViewerApi.js
   trunk/layers/MapServer/MapServer.js
Log:
re #140: changes suggested by JSLint, remove extra commas in MapGuideViewerApi.js and remove that from the single file build

Modified: trunk/build.xml
===================================================================
--- trunk/build.xml	2008-11-19 17:56:45 UTC (rev 1675)
+++ trunk/build.xml	2008-11-19 19:37:40 UTC (rev 1676)
@@ -234,7 +234,7 @@
     <!-- merge everything into a single file -->
     <concat destfile="${buildOutput}.js" outputencoding="UTF-8" append="true">
         <fileset dir="${build.home}/widgets" includes="**/*.js"/>
-        <fileset dir="${build.home}/layers" includes="**/*.js" excludes="Layers.js"/>
+        <fileset dir="${build.home}/layers" includes="**/*.js" excludes="Layers.js MapGuide/MapGuideViewerApi.js"/>
     </concat>
   </target>
 

Modified: trunk/layers/MapGuide/MapGuide.js
===================================================================
--- trunk/layers/MapGuide/MapGuide.js	2008-11-19 17:56:45 UTC (rev 1675)
+++ trunk/layers/MapGuide/MapGuide.js	2008-11-19 19:37:40 UTC (rev 1676)
@@ -249,7 +249,7 @@
             if (!this.bSingleTile) {
               if (o.groups.length >0) {
                 this.bSingleTile = false;
-                this.groupName = o.groups[0].groupName  //assumes only one group for now
+                this.groupName = o.groups[0].groupName;  //assumes only one group for now
                 this.mapWidget.registerForEvent(Fusion.Event.MAP_EXTENTS_CHANGED,
                     OpenLayers.Function.bind(this.mapExtentsChanged, this));
               } else {
@@ -482,7 +482,9 @@
 
         this.oLayerOL.mergeNewParams(params);
 
-        if (this.queryLayer) this.queryLayer.redraw(true);
+        if (this.queryLayer) {
+          this.queryLayer.redraw(true);
+        }
     },
 
     /**
@@ -778,7 +780,7 @@
               }
 
               if (zoomTo) {
-                var ext = oNode.extents
+                var ext = oNode.extents;
                 var extents = new OpenLayers.Bounds(ext.minx, ext.miny, ext.maxx, ext.maxy);
                 this.zoomToSelection(extents);
               }
@@ -815,7 +817,7 @@
             'maxfeatures': options.maxFeatures || 0, //zero means select all features
             'layers': options.layers || '',
             'variant': options.selectionType || this.selectionType
-        }
+        };
         if (options.filter) {
             params.filter= options.filter;
         }

Modified: trunk/layers/MapGuide/MapGuideViewerApi.js
===================================================================
--- trunk/layers/MapGuide/MapGuideViewerApi.js	2008-11-19 17:56:45 UTC (rev 1675)
+++ trunk/layers/MapGuide/MapGuideViewerApi.js	2008-11-19 19:37:40 UTC (rev 1676)
@@ -249,7 +249,7 @@
         seg.to.updateGeo();
         this.clearContext();
         this.draw(this.context);
-    },
+    }
 }
     
 RectangleHandlers = Class.create();
@@ -317,7 +317,7 @@
         this.seg4.from.updateGeo();
         this.clearContext();
         this.draw(this.context);
-    },
+    }
 }
     
 MultiPointHandlers = Class.create();

Modified: trunk/layers/MapServer/MapServer.js
===================================================================
--- trunk/layers/MapServer/MapServer.js	2008-11-19 17:56:45 UTC (rev 1675)
+++ trunk/layers/MapServer/MapServer.js	2008-11-19 19:37:40 UTC (rev 1676)
@@ -671,7 +671,7 @@
                 }
                 this.newSelection();
                 if (zoomTo) {
-                var ext = oNode.extents
+                var ext = oNode.extents;
                 var extents = new OpenLayers.Bounds(ext.minx, ext.miny, ext.maxx, ext.maxy);
                 this.zoomToSelection(extents);
               }
@@ -706,7 +706,7 @@
             'maxfeatures': options.maxFeatures || -1, //-1 means select all features
             'layers': layers,
             'variant': options.selectionType || this.selectionType
-        }
+        };
         if (options.filter) {
             params.filter = options.filter;
         }