Results 1 to 2 of 2

Thread: [FIXED][3.0] JSON usage in JsonReader and TreeLoader

Hybrid View

Previous Post Previous Post   Next Post Next Post
    Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Ext JS Premium Member stever's Avatar
    Join Date
    Mar 2007
    Posts
    1,408
    Vote Rating
    9
      0  

    Default [FIXED][3.0] JSON usage in JsonReader and TreeLoader

    I think JsonReader.read ought to use the Ext JSON functionality rather than calling eval.

    Original:

    PHP Code:
        read : function(response){
            var 
    json response.responseText;
            var 
    = eval("("+json+")");
            if(!
    o) {
                throw {
    message"JsonReader.read: Json object not found"};
            }
            return 
    this.readRecords(o);
        } 
    Proposed:
    PHP Code:
        read : function(response){
            var 
    Ext.decode(response.responseText);
            if(!
    o) {
                throw {
    message"JsonReader.read: Json object not found"};
            }
            return 
    this.readRecords(o); 
    Same thing in TreeLoader.processResponse.

  2. #2
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    18,369
    Vote Rating
    858
      0  

    Default

    Fixed in SVN.
    Evan Trimboli
    Sencha Developer
    Twitter - @evantrimboli
    Don't be afraid of the source code!

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •