Results 1 to 5 of 5

Thread: [4.0.6] Ext.data.TreeStore.load - Async Concurrency Problem

    Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-4124 in a recent build.
  1. #1
    Ext JS Premium Member
    Join Date
    Mar 2008
    Location
    Germany, Frankfurt
    Posts
    143
    Vote Rating
    1
      0  

    Default [4.0.6] Ext.data.TreeStore.load - Async Concurrency Problem

    Ext version tested:
    • Ext 4.0.6


    Browser versions tested against:
    • FF6.0.2 (firebug 1.8.1 installed)


    DOCTYPE tested against:
    • none


    Description:
    • If using Ext.data.TreeStore.load as documented against a remote DataSource in some situations (one particular documented below) the nodes get screwd up.


    Steps to reproduce the problem:
    • see code and demo


    The result that was expected:
    • stable behaviour ... no screwd up situation not matter which of the documented parameters are set (workaround situation: see demo)


    The result that occurs instead:
    • see demo


    Test Case:
    see the attached demo app (as soon as I manage to upload my existing zip file :-)


    Screenshot or Video:
    • none


    See this URL for live test case: http://enpasos.com/extjsissue1/ (up until fixed or latest until end of Sept 2011)


    Debugging already done:
    • yes


    Possible fix:
    I suggest removing the following code from the load function in TreeStore:
    Code:
            // If there is not a node it means the user hasnt defined a rootnode yet. In this case lets just
            // create one for them.
            if (!node) {
                node = me.setRootNode({
                    expanded: true
                });
            }
    This would fix a race condition introduced inside the load function. In addition a stabilization against concurrent calls to the load method is needed - e.g. by an encapsulated locking like in AsyncTreeLoader in ExtJS3 or by defining a user contract that works (see workaround in demo and code).

    Additional CSS used:
    • no


    Operating System:
    • Win7



    Please reopen the corresponding issue

  2. #2
    Ext JS Premium Member
    Join Date
    Mar 2008
    Location
    Germany, Frankfurt
    Posts
    143
    Vote Rating
    1
      0  

    Default

    here the code as in demo showing issue and workaround
    Attached Files Attached Files

  3. #3
    Ext JS Premium Member
    Join Date
    Mar 2008
    Location
    Germany, Frankfurt
    Posts
    143
    Vote Rating
    1
      0  

    Default [4.0.7] Bug still there ... Why?

    I have retested the bug in [4.0.7]: It is still there. See http://enpasos.com/extjsissue1/

    I also can not find EXTJSIV-4124 in the release notes.

    This is very strange to me. Could you please explain.

  4. #4
    Ext JS Premium Member
    Join Date
    Mar 2008
    Location
    Germany, Frankfurt
    Posts
    143
    Vote Rating
    1
      0  

    Default

    the sources for my [4.0.7] testcase:

    test-extjs407.zip

  5. #5
    Ext JS Premium Member
    Join Date
    Mar 2008
    Location
    Germany, Frankfurt
    Posts
    143
    Vote Rating
    1
      0  

    Default

    Please reopen for [4.0.7].

Posting Permissions

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