RFC: E6 destructuring for resolving promise

[pYr0x]

i would like to unify the response of the XHR response.
the idea is to response a array while resolving or rejecting.
i came up with this:

xhr.onreadystatechange = function () {
		try {
			if (xhr.readyState === 4) {
				if (timer) {
					clearTimeout(timer);
				}
				if (xhr.status < 300) {
					if (o.success) {
						o.success(_xhrResp(xhr, o), xhr.statusText, xhr);
					}
				}
				else if (o.error) {
					o.error(_xhrResp(xhr, o), xhr.statusText, xhr);
				}
				if (o.complete) {
					o.complete(_xhrResp(xhr, o), xhr.statusText, xhr);
				}

				if (xhr.status >= 200 && xhr.status < 300) {
					deferred.resolve([_xhrResp(xhr, o), xhr.statusText, xhr]);
				} else {
					deferred.reject([_xhrResp(xhr, o), xhr.statusText, xhr]);
				}
			}
			else if (o.progress) {
				o.progress(++n);
			}
		} catch(e) {
			deferred.reject(e);
		}
	};

deferred.resolve([_xhrResp(xhr, o), xhr.statusText, xhr]); will return an array which can be
destructured

ajax({
			type: "get",
			url: __dirname+"/can-ajax-test-result.json"
		}).then(function([resp, textStatus, xhr]){
			assert.equal(textStatus, "OK");
			done();
		});

this change will break all tests and all implementations
this change will only work in Edge. Not on IE11

any thoughts?

[justinbmeyer]

Could this be behind a flag so as not to break existing apps?

[justinbmeyer]

Why wouldn’t this work in IE?

[pYr0x]

destructuring is not working on IE
https://kangax.github.io/compat-table/es6/

[justinbmeyer]

Yeah, but you IE could destructure manually or with a transpiler right?

…

Sent from my iPhone

On Jun 9, 2018, at 10:02 AM, Julian ***@***.***> wrote: destructuring is not working on IE https://kangax.github.io/compat-table/es6/ — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.