fix(TopBarProfileMenu): use window.location.reload on logout

opencollective · May 15, 2018 · 8df0e07 · 8df0e07
1 parent 3eb5017
commit 8df0e07
Showing 1 changed file with 10 additions and 12 deletions.
diff --git a/src/components/TopBarProfileMenu.js b/src/components/TopBarProfileMenu.js
@@ -17,24 +17,15 @@ class TopBarProfileMenu extends React.Component {
     super(props);
     this.state = { showProfileMenu: false, loading: true };
     this.toggleProfileMenu = this.toggleProfileMenu.bind(this);
-    this.logout = this.logout.bind(this);
     this.messages = defineMessages({
       'tooltip.balance': { id: 'profilemenu.memberships.tooltip.balance', defaultMessage: 'Balance {balance}' },
       'tooltip.pendingExpenses': { id: 'profilemenu.memberships.tooltip.pendingExpenses', defaultMessage: '{n} pending expenses' },
       'menu.transactions': { id: 'menu.transactions', defaultMessage: 'transactions' }
     });
   }
 
-  logout() {
-    this.setState({ showProfileMenu: false, status: 'loggingout' })
-    window.localStorage.removeItem('accessToken');
-    window.localStorage.removeItem('LoggedInUser');
-    window.location.replace(window.location.href);
-  }
-
   componentDidMount() {
-    this.onClickOutsideRef = this.onClickOutside.bind(this);
-    document.addEventListener('click', this.onClickOutsideRef);
+    document.addEventListener('click', this.onClickOutside);
     if (typeof window !== 'undefined') {
       this.redirectAfterSignin = window.location.href.replace(/^https?:\/\/[^/]+/,'');
       if (!window.localStorage.accessToken) {
@@ -44,10 +35,17 @@ class TopBarProfileMenu extends React.Component {
   }
 
   componentWillUnmount() {
-    document.removeEventListener('click', this.onClickOutsideRef);
+    document.removeEventListener('click', this.onClickOutside);
+  }
+
+  logout = () => {
+    this.setState({ showProfileMenu: false, status: 'loggingout' })
+    window.localStorage.removeItem('accessToken');
+    window.localStorage.removeItem('LoggedInUser');
+    window.location.reload();
   }
 
-  onClickOutside() {
+  onClickOutside = () => {
     this.setState({ showProfileMenu: false });
   }