Results 1 to 5 of 5

Thread: 4.0.6-4.0.7 TreeStore change

    Looks like we can't reproduce the issue or there's a problem in the test case provided.
  1. #1
    Sencha Premium User westy's Avatar
    Join Date
    Feb 2009
    Location
    Bath, UK
    Posts
    1,038
    Vote Rating
    108
      0  

    Default 4.0.6-4.0.7 TreeStore change

    Hi,

    Just noticed something very odd going on with the TreeStore in 4.0.7, and after some investigation have figured out what is going on (I think), but would like to know why the change was made.

    First, the effect I'm seeing... I have a tree with the ability to reload at a given node.
    This should result in a single call to the server, using Ext.Direct, to load the nodes children.
    What I'm seeing now is a call per existing child node, passing just the node up, and finally the call that _should_ be made, with the correct parameters to load the data.

    Have tracked it down to the change in TreeStore.load, where node.removeAll() was changed to be node.removeAll(true), i.e. the nodes should now be destroyed.

    Stepping through, and into my favourite "class" in the framework(!), NodeInterface, I see that because destroy is called with silent set to "true", a call to the proxy is made, presumably to tell the server that the node has been destroyed or something (so it can be deleted perhaps?!).

    I'm sorry but what?!
    We're reloading a node here, not deleting one.

    If I change removeAll back to not destroy, or the destruction to not be silent, then I get the single call to the server. Obviously loads of chatter to the server for no reason is expensive, and kills performance.

    Obviously I can work around this, by reverting the change in an override, but would argue that the change makes no sense, hence my post...

    Would welcome any insight.
    Perhaps code changes should have comments on them listing the bug number being addressed, since that would probably help determine the intention.

    Cheers,
    Westy
    Product Architect
    Altus Ltd.

  2. #2
    Sencha - Community Support Team jsakalos's Avatar
    Join Date
    Apr 2007
    Location
    Slovakia
    Posts
    27,664
    Vote Rating
    404
      0  

    Default

    I suggest to move this thread to Bugs (I can do it for you if you want) because this question is aimed directly to developers. Bugs forum is scanned by them, this forum not.
    Jozef Sakalos, aka Saki

    Education, extensions and services for developers at new http://extjs.eu
    News: Grid MultiSearch Plugin, Grid MultiSort Plugin, Configuring ViewModel Hierarchy


  3. #3
    Sencha Premium User westy's Avatar
    Join Date
    Feb 2009
    Location
    Bath, UK
    Posts
    1,038
    Vote Rating
    108
      0  

    Default

    If you could please; seems that I can only close or delete the thread.

    Of course, if it's moved to the bugs forum I'll just get told off for not supplying a test case, but hey ho
    Think it needs thinking about.

    Some additional info that may be relevant, I'm using directFn rather that the CRUD config for my proxy.

    Cheers,
    Westy
    Product Architect
    Altus Ltd.

  4. #4
    Ext JS Premium Member
    Join Date
    Feb 2011
    Posts
    21
    Vote Rating
    0
      0  

    Default

    Yeap, this is definitely related to zombeerose's thread http://www.sencha.com/forum/showthre...cords&p=664477

  5. #5
    Sencha Premium User evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    19,234
    Vote Rating
    1015
      0  

    Default

    Agreed, looks like a dupe. Please track in the other thread.
    Twitter - @evantrimboli
    Former Sencha framework engineer, available for consulting.
    As of 2017-09-22 I am not employed by Sencha, all subsequent posts are my own and do not represent Sencha in any way.

Tags for this Thread

Posting Permissions

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