Merge pull request #377 from nickago/MM-1894

Mm 1894 Add state listening to the sidebar relative to the team store

web/react/components/sidebar.jsx

@@ -128,6 +128,7 @@ module.exports = React.createClass({
         ChannelStore.addChangeListener(this.onChange);
         UserStore.addChangeListener(this.onChange);
         UserStore.addStatusesChangeListener(this.onChange);
+        TeamStore.addChangeListener(this.onChange);
         SocketStore.addChangeListener(this.onSocketChange);
         $('.nav-pills__container').perfectScrollbar();
 
@@ -146,6 +147,7 @@ module.exports = React.createClass({
         ChannelStore.removeChangeListener(this.onChange);
         UserStore.removeChangeListener(this.onChange);
         UserStore.removeStatusesChangeListener(this.onChange);
+        TeamStore.removeChangeListener(this.onChange);
         SocketStore.removeChangeListener(this.onSocketChange);
     },
     onChange: function() {
@@ -348,15 +350,16 @@ module.exports = React.createClass({
 
             // set up click handler to switch channels (or create a new channel for non-existant ones)
             var clickHandler = null;
-            var href;
+            var href = '#';
+            var teamURL = TeamStore.getCurrentTeamUrl();
             if (!channel.fake) {
                 clickHandler = function(e) {
                     e.preventDefault();
                     utils.switchChannel(channel);
                 };
-                href = '#';
+            }
-            } else {
+            if (channel.fake && teamURL){
-                href = TeamStore.getCurrentTeamUrl() + '/channels/' + channel.name;
+                href = teamURL + '/channels/' + channel.name;
             }
 
             return (

web/react/stores/team_store.jsx

@@ -13,7 +13,9 @@ var CHANGE_EVENT = 'change';
 
 var utils;
 function getWindowLocationOrigin() {
-    if (!utils) utils = require('../utils/utils.jsx');
+    if (!utils) {
+        utils = require('../utils/utils.jsx');
+    }
     return utils.getWindowLocationOrigin();
 }
 
@@ -28,15 +30,14 @@ var TeamStore = assign({}, EventEmitter.prototype, {
         this.removeListener(CHANGE_EVENT, callback);
     },
     get: function(id) {
-    var c = this._getTeams();
+        var c = this.pGetTeams();
         return c[id];
     },
     getByName: function(name) {
-    var current = null;
+        var t = this.pGetTeams();
-    var t = this._getTeams();
 
-    for (id in t) {
+        for (var id in t) {
-        if (t[id].name == name) {
+            if (t[id].name === name) {
                 return t[id];
             }
         }
@@ -44,45 +45,49 @@ var TeamStore = assign({}, EventEmitter.prototype, {
         return null;
     },
     getAll: function() {
-    return this._getTeams();
+        return this.pGetTeams();
     },
     setCurrentId: function(id) {
-    if (id == null)
+        if (id === null) {
-      BrowserStore.removeItem("current_team_id");
+            BrowserStore.removeItem('current_team_id');
-    else
+        } else {
-      BrowserStore.setItem("current_team_id", id);
+            BrowserStore.setItem('current_team_id', id);
+        }
     },
     getCurrentId: function() {
-    return BrowserStore.getItem("current_team_id");
+        return BrowserStore.getItem('current_team_id');
     },
     getCurrent: function() {
         var currentId = TeamStore.getCurrentId();
 
-    if (currentId != null)
+        if (currentId !== null) {
             return this.get(currentId);
-    else
+        }
         return null;
     },
     getCurrentTeamUrl: function() {
-      return getWindowLocationOrigin() + "/" + this.getCurrent().name;
+        if (this.getCurrent()) {
+            return getWindowLocationOrigin() + '/' + this.getCurrent().name;
+        }
+        return null;
     },
     storeTeam: function(team) {
-      var teams = this._getTeams();
+        var teams = this.pGetTeams();
         teams[team.id] = team;
-      this._storeTeams(teams);
+        this.pStoreTeams(teams);
     },
-  _storeTeams: function(teams) {
+    pStoreTeams: function(teams) {
-    BrowserStore.setItem("user_teams", teams);
+        BrowserStore.setItem('user_teams', teams);
     },
-  _getTeams: function() {
+    pGetTeams: function() {
-    return BrowserStore.getItem("user_teams", {});
+        return BrowserStore.getItem('user_teams', {});
     }
 });
 
-TeamStore.dispatchToken = AppDispatcher.register(function(payload) {
+TeamStore.dispatchToken = AppDispatcher.register(function registry(payload) {
     var action = payload.action;
 
-  switch(action.type) {
+    switch (action.type) {
 
         case ActionTypes.CLICK_TEAM:
             TeamStore.setCurrentId(action.id);