Hello,

This is confusing me because it seems like such an obvious bug, but if
it was so obvious then how could it actually be a bug??

This code in controls.js is supposed to show an alert when my edit
fails (sending a status 400, etc.)

onFailure: function(transport, ipe) {
      alert('Error communication with the server: ' +
transport.responseText.stripTags());
    },

but transport doesn't contain the text, the ipe variable does. So this
is what works instead...

onFailure: function(transport, ipe) {
      alert('Error communication with the server: ' +
ipe.responseText.stripTags());
    },

onFailure is actually triggered using the triggerCallback function
which passes the ipe as the first param and the transport as the
second, so in reality, the code should be...

onFailure: function(ipe, transport) {
      alert('Error communication with the server: ' +
transport.responseText.stripTags());
    },

As you can see in the code...

triggerCallback: function(cbName, arg) {
    if ('function' == typeof this.options[cbName]) {
      this.options[cbName](this, arg);
    }
  },

this.triggerCallback('onFailure', transport);

triggerCallback passes "this" as the first param, "this" being the
ipe, and transport is the second param.

Is this a bug?  And if so, it's been around an awfully long time.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Prototype & script.aculo.us" group.
To post to this group, send email to prototype-scriptaculous@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/prototype-scriptaculous?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to