Merge pull request #54 from canjs/51-async

requests are always async

Author: cherifGsoul

can-ajax-test.js

@@ -722,3 +722,32 @@ QUnit.test("beforeSend rejects the ajax promise on failure", function (assert) {
 		assert.equal(reason, error, "error is what we expect");
 	}).then(done);
 });
+
+QUnit.test("async should be always true #51", function(assert){
+	var done = assert.async();
+	var headers = {},
+		restore = makeFixture(function () {
+			this.open = function (type, url, async) {
+				assert.ok(async);
+			};
+
+			this.send = function () {
+			    this.readyState = 4;
+			    this.status = 204;
+			    this.responseText = '';
+			    this.onreadystatechange();
+		    };
+
+			this.setRequestHeader = function (header, value) {
+				headers[header] = value;
+			};
+	});
+	
+	ajax({
+		type: "get",
+		url: "/ep"
+	}).then(function () {
+		restore();
+		done();
+	});
+});

can-ajax.js

@@ -216,7 +216,7 @@ function ajax(o) {
 	if (!isPost && o.data) {
 		url += "?" + (isJsonContentType ? JSON.stringify(o.data) : param(o.data));
 	}
-	xhr.open(type, url);
+	xhr.open(type, url, true);
 
 	// For CORS to send a "simple" request (to avoid a preflight check), the following methods are allowed: GET/POST/HEAD,
 	// see https://developer.mozilla.org/en-US/docs/Web/HTTP/Access_control_CORS#Simple_requests