jquery-form#218; wrong context scope

malsup · malsup · commit 37c48278336f · 2012-07-29T17:50:55.000-04:00
diff --git a/jquery.form.js b/jquery.form.js
@@ -1,6 +1,6 @@
 /*!
  * jQuery Form Plugin
- * version: 3.10 (20-JUL-2012)
+ * version: 3.11 (20-JUL-2012)
  * @requires jQuery v1.3.2 or later
  *
  * Examples and documentation at: http://malsup.com/jquery/form/
@@ -165,7 +165,7 @@ $.fn.ajaxSubmit = function(options) {
     }
 
     options.success = function(data, status, xhr) { // jQuery 1.4+ passes xhr as 3rd arg
-        var context = options.context || options;    // jQuery 1.4+ supports scope context 
+        var context = options.context || this ;    // jQuery 1.4+ supports scope context 
         for (var i=0, max=callbacks.length; i < max; i++) {
             callbacks[i].apply(context, [data, status, xhr || $form, $form]);
         }

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`/*!`
`2`	`2`	`* jQuery Form Plugin`
`3`		`- * version: 3.10 (20-JUL-2012)`
	`3`	`+ * version: 3.11 (20-JUL-2012)`
`4`	`4`	`* @requires jQuery v1.3.2 or later`
`5`	`5`	`*`
`6`	`6`	`* Examples and documentation at: http://malsup.com/jquery/form/`
`@@ -165,7 +165,7 @@ $.fn.ajaxSubmit = function(options) {`
`165`	`165`	`}`
`166`	`166`
`167`	`167`	`options.success = function(data, status, xhr) { // jQuery 1.4+ passes xhr as 3rd arg`
`168`		`- var context = options.context \|\| options; // jQuery 1.4+ supports scope context`
	`168`	`+ var context = options.context \|\| this ; // jQuery 1.4+ supports scope context`
`169`	`169`	`for (var i=0, max=callbacks.length; i < max; i++) {`
`170`	`170`	`callbacks[i].apply(context, [data, status, xhr \|\| $form, $form]);`
`171`	`171`	`}`