Hybrid View

    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
    Jan (HL) will become famous soon enough

      0  

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

    [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 - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    36,633
    Vote Rating
    817
    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


    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
    Notts/Redwood City
    Posts
    30,496
    Vote Rating
    44
    Animal has a spectacular aura about Animal has a spectacular aura about Animal has a spectacular aura about

      0  

    Default


    Are you sure?



    Those first two args are the node appended to, and the node being appended.

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

      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
    Notts/Redwood City
    Posts
    30,496
    Vote Rating
    44
    Animal has a spectacular aura about Animal has a spectacular aura about Animal has a spectacular aura about

      0  

    Default


    Ah! Well spotted! It's the append of the root node. All others are correct.

Thread Participants: 2