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,406
    Vote Rating
    6
    stever will become famous soon enough stever will become famous soon enough

      0  

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

    [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
    16,647
    Vote Rating
    583
    evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute

      0  

    Default


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

Thread Participants: 1