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
    69
    Vote Rating
    2
    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
    Gainesville, FL
    Posts
    37,047
    Vote Rating
    850
    mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute

      0  

    Default


    This seems to be fixed in Sencha Touch 2
    Mitchell Simoens @SenchaMitch
    Sencha Inc, Senior Forum Manager
    ________________
    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 in print!

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

Thread Participants: 1