Important ! Listener is not removed

jr-k · web-flow · commit 9a36a78eb8b8 · 2018-03-19T09:56:36.000+01:00
There is no ID for the new listener so actually when the component is updated it keeps creating new listeners.
diff --git a/src/TriggeringView.js b/src/TriggeringView.js
@@ -74,7 +74,7 @@ class TriggeringView extends Component<Props, State> {
       return;
     }
     this.context.scrollY.removeListener(this.listenerId);
-    nextContext.scrollY.addListener(this.onScroll);
+    this.listenerId = nextContext.scrollY.addListener(this.onScroll);
   }
 
   onRef = (ref: any) => {

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ class TriggeringView extends Component<Props, State> {`
`74`	`74`	`return;`
`75`	`75`	`}`
`76`	`76`	`this.context.scrollY.removeListener(this.listenerId);`
`77`		`- nextContext.scrollY.addListener(this.onScroll);`
	`77`	`+ this.listenerId = nextContext.scrollY.addListener(this.onScroll);`
`78`	`78`	`}`
`79`	`79`
`80`	`80`	`onRef = (ref: any) => {`