react-observable-mixin - npm Package Compare versions

Comparing version 1.0.2 to 1.0.3

lib/RxObservableMixin.js

@@ -73,2 +73,4 @@ 'use strict';
   observablesSubscribeOnNext: function observablesSubscribeOnNext(state) {
+    if (!this.observablesSubscription) return;
+
     if (this.componentWillReceiveStateFromObservables) this.componentWillReceiveStateFromObservables(state);
@@ -87,2 +89,4 @@
   observablesSubscribeOnError: function observablesSubscribeOnError(error) {
+    if (!this.observablesSubscription) return;
+
     if (this.componentWillReceiveErrorFromObservables) this.componentWillReceiveErrorFromObservables(error);
@@ -120,5 +124,8 @@ },
   unsubscribeFromObservables: function unsubscribeFromObservables() {
-    this.observablesSubscription.dispose();
+    if (this.observablesSubscription) {
+      this.observablesSubscription.dispose();
+      this.observablesSubscription = null;
+    }
   }
 };
 module.exports = exports['default'];

package.json

 {
   "name": "react-observable-mixin",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Container like observable mixin for ReactJS.",
@@ -5,0 +5,0 @@ "main": "lib/index.js",

src/RxObservableMixin.js

@@ -62,2 +62,5 @@ import Rx from 'rx';
   observablesSubscribeOnNext(state) {
+    if (!this.observablesSubscription)
+      return;
+
     if (this.componentWillReceiveStateFromObservables)
@@ -77,2 +80,5 @@ this.componentWillReceiveStateFromObservables(state);
   observablesSubscribeOnError(error) {
+    if (!this.observablesSubscription)
+      return;
+
     if (this.componentWillReceiveErrorFromObservables)
@@ -109,4 +115,7 @@ this.componentWillReceiveErrorFromObservables(error);
   unsubscribeFromObservables() {
-    this.observablesSubscription.dispose();
+    if (this.observablesSubscription) {
+      this.observablesSubscription.dispose();
+      this.observablesSubscription = null;
+    }
   }
 };

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

1197961

increased by0.05%

Lines of code

29648

increased by0.06%

No dependency changes