Results 1 to 4 of 4

Thread: Unable to remove root definition from reader

    Success! Looks like we've fixed this one. According to our records the fix was applied for DSGNR-1536 in a recent build.
  1. #1
    Sencha Premium Member
    Join Date
    Dec 2011
    Location
    London, UK
    Posts
    269
    Vote Rating
    14
      0  

    Default Unable to remove root definition from reader

    After adding a json reader to a store:

    Code:
    Ext.define('Photos.store.base.MyJsonReader', {    extend: 'Ext.data.reader.Json',
    
    
        constructor: function(cfg) {
            var me = this;
            cfg = cfg || {};
            me.callParent([Ext.apply({
                type: 'json',
                id: 'PhotosFoldersReader'
            }, cfg)]);
        }
    });
    I wanted to try changing the root. However it was not helpful but I am unable to remove the root. If I try by using the delete menu option or by clicking on the 'default' button or by removing the content I see an error message stating that 'this config value for root is mandatory'. Clearly that's not the case. The only solution is to remove the reader and add it again.

    Adding the reader again is not difficult but it would be great if it is possible to restore the default configuration .

  2. #2
    Sencha User Phil.Strong's Avatar
    Join Date
    Mar 2007
    Location
    Olney, MD
    Posts
    1,953
    Vote Rating
    65
      0  

    Default

    Yeah we're being too strict here.

    Work around for now would be to use undo!
    Phil Strong
    @philstrong

  3. #3
    Sencha Premium Member
    Join Date
    Dec 2011
    Location
    London, UK
    Posts
    269
    Vote Rating
    14
      0  

    Default

    That assumes you are 'undo'ing when relevant change history is available. If you realize later, after making other changes or maybe the next day, there's no relevant history. The work around then is to delete the component and start over.

  4. #4
    Sencha User aconran's Avatar
    Join Date
    Mar 2007
    Posts
    9,487
    Vote Rating
    145
      0  

    Default

    This will be fixed in build > 311.
    Aaron Conran
    @aconran

Posting Permissions

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