[mapguide-commits] r5444 - trunk/MgDev/Web/src/viewerfiles

[svn_mapguide at osgeo.org]

Author: hubu
Date: 2010-12-06 21:19:30 -0800 (Mon, 06 Dec 2010)
New Revision: 5444

Modified:
   trunk/MgDev/Web/src/viewerfiles/ajaxmappane.templ
Log:
On behalf of: Spark Liu

When there is no point defined, there should be no query instead of query with error parameter.
Non-Chrome browser should also show message when returning status is not 200.

Fix ticket 1553.

Modified: trunk/MgDev/Web/src/viewerfiles/ajaxmappane.templ
===================================================================
--- trunk/MgDev/Web/src/viewerfiles/ajaxmappane.templ	2010-12-06 22:57:34 UTC (rev 5443)
+++ trunk/MgDev/Web/src/viewerfiles/ajaxmappane.templ	2010-12-07 05:19:30 UTC (rev 5444)
@@ -962,10 +962,10 @@
                     StartToolFeedback(x, y, CIRCLE);
                     break;
                 case 6:
-            if((e.ctrlKey && !macOS) || (e.metaKey && macOS))
-            {
-            EndPolygonSelection(x, y);
-            }
+                    if((e.ctrlKey && !macOS) || (e.metaKey && macOS))
+                    {
+                        EndPolygonSelection(x, y);
+                    }
                     else
                     {
                         if(polySel.xs.length == 0)
@@ -987,6 +987,8 @@
 
 function EndPolygonSelection(x, y)
 {
+    if(polySel.xs.length == 0)
+        return;
     EndToolFeedback(x, y);
     ExecutePolygonTool();
 }
@@ -2931,7 +2933,7 @@
     selRequest.setRequestHeader("Content-Type", "application/x-www-form-urlencoded");
     selRequest.send(reqParams);
 
-    if(selRequest.responseXML)
+    if(selRequest.status == 200 && selRequest.responseXML)
     {
         ProcessFeatureInfo(selRequest.responseXML.documentElement, append, which);
     }