remove debugging code

[roojs1] / Roo / JsonView.js

diff --git a/Roo/JsonView.js b/Roo/JsonView.js
index 8b5e861..4d9513c 100644 (file)
--- a/Roo/JsonView.js
+++ b/Roo/JsonView.js
@@ -14,10 +14,12 @@
  * @extends Roo.View
  * Shortcut class to create a JSON + {@link Roo.UpdateManager} template view. Usage:
 <pre><code>
-var view = new Roo.JsonView("my-element",
-    '&lt;div id="{id}"&gt;{foo} - {bar}&lt;/div&gt;', // auto create template
-    { multiSelect: true, jsonRoot: "data" }
-);
+var view = new Roo.JsonView({
+    container: "my-element",
+    tpl: '&lt;div id="{id}"&gt;{foo} - {bar}&lt;/div&gt;', // auto create template
+    multiSelect: true, 
+    jsonRoot: "data" 
+});
 
 // listen for node click?
 view.on("click", function(vw, index, node, e){
@@ -48,6 +50,7 @@ moreView.load({
 });
 </code></pre>
 * 
+* Note: old code is supported with arguments : (container, template, config)
 * 
 * 
  * @constructor
@@ -56,10 +59,10 @@ moreView.load({
  * @param {Object} config The config object
  * 
  */
-Roo.JsonView = function(container, tpl, config){
+Roo.JsonView = function(config, depreciated_tpl, depreciated_config){
     
     
-    Roo.JsonView.superclass.constructor.call(this, container, tpl, config);
+    Roo.JsonView.superclass.constructor.call(this, config, depreciated_tpl, depreciated_config);
 
     var um = this.el.getUpdateManager();
     um.setRenderer(this);
@@ -93,8 +96,7 @@ Roo.JsonView = function(container, tpl, config){
 };
 Roo.extend(Roo.JsonView, Roo.View, {
     /**
-     * The root property in the loaded JSON object that contains the data
-     * @type {String}
+     * @type {String} The root property in the loaded JSON object that contains the data
      */
     jsonRoot : "",
 
@@ -146,15 +148,20 @@ Roo.extend(Roo.JsonView, Roo.View, {
         um.update.apply(um, arguments);
     },
 
+    // note - render is a standard framework call...
+    // using it for the response is really flaky... - it's called by UpdateManager normally, except when called by the XComponent/addXtype.
     render : function(el, response){
+        
         this.clearSelections();
         this.el.update("");
         var o;
         try{
-            o = Roo.util.JSON.decode(response.responseText);
-            if(this.jsonRoot){
-                
-                o = /** eval:var:o */ eval("o." + this.jsonRoot);
+            if (response != '') {
+                o = Roo.util.JSON.decode(response.responseText);
+                if(this.jsonRoot){
+                    
+                    o = o[this.jsonRoot];
+                }
             }
         } catch(e){
         }