Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-9325 in a recent build.
  1. #1
    Sencha Premium Member
    Join Date
    Oct 2011
    Location
    Huntsville
    Posts
    227
    Vote Rating
    -6
    mark0978 is infamous around these parts mark0978 is infamous around these parts

      0  

    Exclamation Ext.data.Operation callback called with 2 arguments instead of 3

    Ext.data.Operation callback called with 2 arguments instead of 3


    Personally I'd be fine with you just changing the docs. This has been broken since 2011, so it isn't like anyone is using it the way you have documented

    http://docs.sencha.com/ext-js/4-1/#!...data.Operation

    The 3rd param is NOT passed to callback and isn't really needed. Either way, the code or the docs need to be fixed, the comments are the only thing that stops this from wasting lots of people's time.

  2. #2
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    36,795
    Vote Rating
    834
    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
    Join Date
    Jun 2011
    Location
    San Diego, CA
    Posts
    191
    Vote Rating
    37
    nohuhu has a spectacular aura about nohuhu has a spectacular aura about

      0  

    Default


    mark0978,

    Where exactly callback is passed 2 arguments? Please give an example.

    Regards,
    Alex.

  4. #4
    Sencha Premium Member
    Join Date
    Oct 2011
    Location
    Huntsville
    Posts
    227
    Vote Rating
    -6
    mark0978 is infamous around these parts mark0978 is infamous around these parts

      0  

    Default


    Here is a stack backtrace for extjs-debug-all v4.1.3

    Code:
    record.save.callback (alerts.js:87)
    Ext.apply.callback (ext-all-debug.js:6820)
    callback (ext-all-debug.js:60568)
    Ext.define.processResponse (ext-all-debug.js:61736)
    (anonymous function) (ext-all-debug.js:61933)
    Ext.apply.callback (ext-all-debug.js:6820)
    Ext.define.onComplete (ext-all-debug.js:24662)
    Ext.define.onStateChange (ext-all-debug.js:24612)
    (anonymous function) (ext-all-debug.js:1890)
    And here is my code that gets you to that point:

    Code:
      , listeners: {
          checkchange: function(me, rowIndex, isChecked, eOpts) {
              var grid = me.up('grid'),
                  store = grid.getStore(),
                  record = store.getAt(rowIndex);
              record.save({
                  callback: function(records, operation) {
                      if(!operation.wasSuccessful()) {
                          Ext.Msg.alert("Failed to save changes to this alert")
                      }
                  }
              });
          }
      }

    Going up the stack to line 6820:
    Code:
    callback: function(callback, scope, args, delay){
            if(Ext.isFunction(callback)){
                args = args || [];
                scope = scope || window;
                if (delay) {
                    Ext.defer(callback, delay, scope, args);
                } else {
                    callback.apply(scope, args);
                }
            }
        },
    args looks like

    Code:
    args
    [ constructor, constructor ]
    in the devtools console, so you can see the callback is only being called with 2 arguments.

    You should be able to setup a unit test to demonstrate this now.

  5. #5
    Sencha - Ext JS Dev Team
    Join Date
    Jun 2011
    Location
    San Diego, CA
    Posts
    191
    Vote Rating
    37
    nohuhu has a spectacular aura about nohuhu has a spectacular aura about

      0  

    Default


    mark0978,

    Thanks for the information; indeed there was a problem. It is fixed now, and hopefully will land in 4.2.1.

    Regards,
    Alex.

Thread Participants: 2

Turkiyenin en sevilen filmlerinin yer aldigi xnxx internet sitemiz olan ve porn sex tarzi bir site olan mobil porno izle sitemiz gercekten dillere destan bir durumda herkesin sevdigi bir site olarak tarihe gececege benziyor. Sitenin en belirgin ozelliklerinden birisi de Turkiyede gercekten kaliteli ve muntazam, duzenli porno izle siteleri olmamasidir. Bu yuzden iste. Ayrica en net goruntu kalitesine sahip adresinde yayinlanmaktadir. Mesela diğer sitelerimizden bahsedecek olursak, en iyi hd porno video arşivine sahip bir siteyiz. "The Best anal porn videos and slut anus, big asses movies set..." hd porno faketaxi