Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Touch Premium Member
    Join Date
    Sep 2011
    Posts
    44
    Vote Rating
    0
    pdm is on a distinguished road

      0  

    Default readRecords fails with empty json objects on iOS

    readRecords fails with empty json objects on iOS


    Sencha Touch version tested:
    • 1.1.1
    Platform tested against:
    • iOS 5.x
    • iOS 5 Simulator
    • Chrome
    Description:
    • when returning an empty element in json, readRecords in Ext.data.Reader fails
    Chrome seems to be tolerant of this, but iOS with phonegap/cordova isn't. Throws
    Code:
    TypeError: 'undefined' is not an object
    Fix:
    Code:
    --- a/www/lib/touch/sencha-touch-debug-w-comments.js
    +++ b/www/lib/touch/sencha-touch-debug-w-comments.js
    @@ -15541,7 +15541,7 @@ Ext.data.Reader = Ext.extend(Object, {
             data = this.getData(data);
     
             var root    = this.getRoot(data),
    -            total   = root.length,
    +            total   = root ? root.length : 0,
                 success = true,
                 value, records, recordCount;
     
    @@ -15559,7 +15559,7 @@ Ext.data.Reader = Ext.extend(Object, {
                 }
             }
     
    -        records = this.extractData(root, true);
    +        records = root ? this.extractData(root, true) : [];
             recordCount = records.length;
     
             return new Ext.data.ResultSet({

  2. #2
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    St. Louis, MO
    Posts
    34,121
    Vote Rating
    453
    mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of mitchellsimoens has much to be proud of

      0  

    Default


    This seems to be fixed in Sencha Touch 2
    Mitchell Simoens @SenchaMitch
    Sencha Inc, Senior Forum Manager
    ________________
    http://www.JSONPLint.com - Source to lint your JSONP!

    Check out my GitHub, lots of nice things for Ext JS 4 and Sencha Touch 2
    https://github.com/mitchellsimoens

    Think my support is good? Get more personalized support via a support subscription. https://www.sencha.com/store/

    Need more help with your app? Hire Sencha Services services@sencha.com

    Want to learn Sencha Touch 2? Check out Sencha Touch in Action that is almost in print!

    When posting code, please use BBCode's CODE tags.