Auth module: explicitly emit 'change' events (#211)

- Replace `this.update` with `this.emit` throughout the `auth` module.
zooniverse · Jul 3, 2023 · 906c550 · 906c550
1 parent d4911f8
commit 906c550
Showing 1 changed file with 10 additions and 19 deletions.
diff --git a/lib/auth.js b/lib/auth.js
@@ -147,11 +147,8 @@ const authClient = new Model({
       };
 
       const registrationRequest = this._makeRegistrationRequest(data);
-      this.update({
-        _currentUserPromise: registrationRequest.catch(function() {
-          return null;
-        }),
-      });
+      this._currentUserPromise = registrationRequest.catch(() => null);
+      this.emit('change', this._currentUserPromise);
 
       return registrationRequest;
     }
@@ -171,9 +168,8 @@ const authClient = new Model({
   async checkCurrent() {
     if (!this._currentUserPromise) {
       console.log('Checking current user');
-      this.update({
-        _currentUserPromise: this._getUser()
-      });
+      this._currentUserPromise = this._getUser();
+      this.emit('change', this._currentUserPromise);
     }
 
     return this._currentUserPromise;
@@ -221,11 +217,8 @@ const authClient = new Model({
       };
 
       const signInRequest = this._makeSignInRequest(data);
-      this.update({
-        _currentUserPromise: signInRequest.catch(function() {
-          return null;
-        }),
-      });
+      this._currentUserPromise = signInRequest.catch(() => null);
+      this.emit('change', this._currentUserPromise);
 
       return signInRequest;
     }
@@ -291,9 +284,8 @@ const authClient = new Model({
     if (user) {
       await user.delete();
       this._deleteBearerToken();
-      this.update({
-        _currentUserPromise: Promise.resolve(null),
-      });
+      this._currentUserPromise = Promise.resolve(null);
+      this.emit('change', this._currentUserPromise);
       console.info('Disabled account');
       return null;
     } else {
@@ -318,9 +310,8 @@ const authClient = new Model({
       try {
         makeCredentialHTTPRequest('DELETE', url, null, deleteHeaders);
         this._deleteBearerToken();
-        this.update({
-          _currentUserPromise: Promise.resolve(null),
-        });
+        this._currentUserPromise = Promise.resolve(null);
+        this.emit('change', this._currentUserPromise);
         console.info('Signed out');
         return null;
       } catch (error) {