id,summary,reporter,owner,description,type,status,priority,milestone,component,version,resolution,keywords,cc,blocking,blockedby
5654,Ajax callbacks and the this keyword,IvanTheBearable,,"Ajax callbacks have been changed to something along these lines:

s.success.call( callbackContext, data, status, xhr );

where callbackContext can be defined with a new ""context"" property of the ajax options.

In general, this is great.  However, if a context is not specified, it defaults to window. In 1.3 the context of an ajax callback was the options object itself. It is documented like this:

=======
function (data, textStatus) {
  // data could be xmlDoc, jsonObj, html, text, etc...
  this; // the options for this ajax request
}
=======

This is likely to break a lot of existing code. Perhaps defaulting the callbackContext to the options object would be better.

Cheers

Ivan
",bug,closed,minor,1.4,ajax,1.4a1,fixed,callback,,,
