fix(AjaxObservable): Fix for #1921 exposed AjaxObservable unsubscription error calling xhr.abort().

trxcllnt · trxcllnt · commit 4d23f87b92e0 · 2016-09-10T17:28:25.000-07:00
Possibly only an artifact of test utils, but after applying fix for #1921 started getting an error that xhr.abort() in AjaxObservable's unsubscribe call wasn't a function.
diff --git a/src/observable/dom/AjaxObservable.ts b/src/observable/dom/AjaxObservable.ts
@@ -361,7 +361,7 @@ export class AjaxSubscriber<T> extends Subscriber<Event> {
 
   unsubscribe() {
     const { done, xhr } = this;
-    if (!done && xhr && xhr.readyState !== 4) {
+    if (!done && xhr && xhr.readyState !== 4 && typeof xhr.abort === 'function') {
       xhr.abort();
     }
     super.unsubscribe();

Original file line number	Diff line number	Diff line change
`@@ -361,7 +361,7 @@ export class AjaxSubscriber<T> extends Subscriber<Event> {`
`361`	`361`
`362`	`362`	`unsubscribe() {`
`363`	`363`	`const { done, xhr } = this;`
`364`		`- if (!done && xhr && xhr.readyState !== 4) {`
	`364`	`+ if (!done && xhr && xhr.readyState !== 4 && typeof xhr.abort === 'function') {`
`365`	`365`	`xhr.abort();`
`366`	`366`	`}`
`367`	`367`	`super.unsubscribe();`