Results 1 to 5 of 5

Thread: [4.2.0.265 Beta] Event signature changed: Ext.data.TreeStore.beforeappend

Hybrid View

Previous Post Previous Post   Next Post Next Post
    Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-8232 in a recent build.
  1. #1
    Ext Premium Member Jan (HL)'s Avatar
    Join Date
    Aug 2010
    Location
    Germany
    Posts
    111
    Vote Rating
    9
      0  

    Default [4.2.0.265 Beta] Event signature changed: Ext.data.TreeStore.beforeappend

    Until Ext JS 4.1, the signature of the event Ext.data.TreeStore.beforeappend is [ this, node, eOptions ] where you can use this as the parent of node. So, 3 arguments.

    Now trying 4.20.265 beta, the signature has changed to [ [parent, node], eOptions] (2 arguments, where the first is an array). I would suggest that something there is a tiny change (typo?) when applying arguments to the event callbacks..?

    Either this is a documentation bug or an internal "bug".

    Do you need more information about this issue? Personally, I think that should be sufficient.

  2. #2
    Sencha - Sr Software Engineer mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    39,897
    Vote Rating
    1330
      0  

    Default

    Thanks for the report! I have opened a bug in our bug tracker.

  3. #3
    Sencha - Ext JS Dev Team Animal's Avatar
    Join Date
    Mar 2007
    Location
    Bédoin/Redwood City
    Posts
    30,644
    Vote Rating
    61
      0  

    Default

    Are you sure?



    Those first two args are the node appended to, and the node being appended.
    Longtime Sencha geek. Outspoken advocate of pure Javascript Views. Posts my own opinions.

  4. #4
    Ext Premium Member Jan (HL)'s Avatar
    Join Date
    Aug 2010
    Location
    Germany
    Posts
    111
    Vote Rating
    9
      0  

    Default

    Yes.

    I've made a test case, take index.js and and index.html from this commit: https://github.com/knalli/extjs4-demos/commit/0602ef202c66e10e9462f7cf473768caa416245f

    S
    ee my output (Chrome24 OSX): Bildschirmfoto 2013-01-16 um 22.57.40.png

    I hadn't checked this correctly: it is only the first call which is unsual (but the first which breaks my app).

    Perhaps a different handling when applying the root node?

  5. #5
    Sencha - Ext JS Dev Team Animal's Avatar
    Join Date
    Mar 2007
    Location
    Bédoin/Redwood City
    Posts
    30,644
    Vote Rating
    61
      0  

    Default

    Ah! Well spotted! It's the append of the root node. All others are correct.
    Longtime Sencha geek. Outspoken advocate of pure Javascript Views. Posts my own opinions.

Posting Permissions

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