From fe695009ff2ddd5f74ae8f1f98dfd378d4defb2f Mon Sep 17 00:00:00 2001
From: Matthias Kretschmann <m@kretschmann.io>
Date: Fri, 12 Apr 2019 18:19:55 +0200
Subject: [PATCH] cancel AssetsUser component fetching when unmounted

---
 .../src/components/organisms/AssetsUser.tsx   | 28 +++++++++++++------
 1 file changed, 20 insertions(+), 8 deletions(-)

diff --git a/client/src/components/organisms/AssetsUser.tsx b/client/src/components/organisms/AssetsUser.tsx
index 1382f22..e3baf87 100644
--- a/client/src/components/organisms/AssetsUser.tsx
+++ b/client/src/components/organisms/AssetsUser.tsx
@@ -13,32 +13,42 @@ export default class AssetsUser extends PureComponent<
 > {
     public state = { results: [], isLoading: true }
 
+    public _isMounted: boolean = false
+
     public componentDidMount() {
-        this.searchOcean()
+        this._isMounted = true
+        this._isMounted && this.searchOcean()
+    }
+
+    public componentWillUnmount() {
+        this._isMounted = false
     }
 
     private async searchOcean() {
-        if (this.context.account) {
-            this.context.ocean.keeper.didRegistry.contract.getPastEvents(
+        const { account, ocean } = this.context
+
+        if (account) {
+            ocean.keeper.didRegistry.contract.getPastEvents(
                 'DIDAttributeRegistered',
                 {
-                    filter: { _owner: this.context.account },
+                    filter: { _owner: account },
                     fromBlock: 0,
                     toBlock: 'latest'
                 },
                 async (error: any, events: any) => {
                     if (error) {
                         Logger.log('error retrieving', error)
-                        this.setState({ isLoading: false })
+                        this._isMounted && this.setState({ isLoading: false })
                     } else {
                         const results = []
                         for (const event of events) {
-                            const ddo = await this.context.ocean.assets.resolve(
+                            const ddo = await ocean.assets.resolve(
                                 `did:op:${event.returnValues._did.substring(2)}`
                             )
                             results.push(ddo)
                         }
-                        this.setState({ results, isLoading: false })
+                        this._isMounted &&
+                            this.setState({ results, isLoading: false })
                     }
                 }
             )
@@ -48,7 +58,9 @@ export default class AssetsUser extends PureComponent<
     }
 
     public render() {
-        return this.context.isNile && this.context.account ? (
+        const { account, isNile } = this.context
+
+        return isNile && account ? (
             <div className={styles.assetsUser}>
                 {this.props.recent && (
                     <h2 className={styles.subTitle}>