fix(subscribe): ignore syncError when deprecated

Unless using the deprecated syncError handling, the syncErrorThrowable flag should be ignored.
ReactiveX · May 26, 2018 · 44e1a27 · 44e1a27
1 parent 445dfe4
commit 44e1a27
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/internal/Observable.ts b/src/internal/Observable.ts
@@ -194,7 +194,11 @@ export class Observable<T> implements Subscribable<T> {
  if (operator) {
  operator.call(sink, this.source);
  } else {
- sink.add(this.source || !sink.syncErrorThrowable ? this._subscribe(sink) : this._trySubscribe(sink));
+ sink.add(
+ this.source || (config.useDeprecatedSynchronousErrorHandling && !sink.syncErrorThrowable) ?
+ this._subscribe(sink) :
+ this._trySubscribe(sink)
+ );
  }
 
  if (config.useDeprecatedSynchronousErrorHandling) {