Clean up of JSON parsing. Comment out the mk2-mobile debug stuff. Will remove when...

[pazpar2-moved-to-github.git] / js / pz2.js

diff --git a/js/pz2.js b/js/pz2.js
index a78417e..7948ea0 100644 (file)
--- a/js/pz2.js
+++ b/js/pz2.js
@@ -707,6 +707,7 @@ var pzHttpRequest = function ( url, errorHandler ) {
         }
 };
 
+
 pzHttpRequest.prototype = 
 {
     safeGet: function ( params, callback )
@@ -805,12 +806,32 @@ pzHttpRequest.prototype =
                        this.request.responseXML == null) {
               if (this.request.responseText != null) {
                 //assume JSON
-                
+               
                var json = null; 
-               if (this.JSON == null)
-                   json = eval("(" + this.request.responseText + ")");
-               else 
-                   json = JSON.parse(this.request.responseText, null);
+               var text = this.request.responseText;
+               if (typeof window.JSON == "undefined") 
+                   json = eval("(" + text + ")");
+               else { 
+                   try {
+                       json = JSON.parse(text);
+                   }
+                   catch (e) {
+                       // Safari: eval will fail as well. Considering trying JSON2 (non-native implementation) instead
+                       /* DEBUG only works in mk2-mobile
+                       if (document.getElementById("log")) 
+                           document.getElementById("log").innerHTML = "" + e + " " + length + ": " + text;
+                       */
+                       try {
+                           json = eval("(" + text + ")");
+                       }
+                       catch (e) {
+                           /* DEBUG only works in mk2-mobile
+                           if (document.getElementById("log")) 
+                               document.getElementById("log").innerHTML = "" + e + " " + length + ": " + text;
+                           */
+                       }
+                   }
+               } 
                this.callback(json, "json");
               } else {
                 var err = new Error("XML response is empty but no error " +