Merge pull request #26 from DannyDelott/master

Redirect user to login page

client/app/ideaDetail/ideaDetail.js

@@ -4,7 +4,7 @@
 
 // The pattern we're using here is the pattern we're using across all our controllers: the controllerAs syntax. This syntax is for Angular versions 1.2 and up, and means you don't have to use `$scope` anymore. Instead, inside of your HTML, you declare your controller with `ng-controller="IdeasCtrl as ictrl"` and reference your variables within that controlled scope as `ictrl.<varname>`. Additionally, instead of setting your properties within your controller to `$scope`, assign your controller's `this` to a variable called self and set your properties to that.
 angular.module( 'glint.ideaDetail', [] )
-  .controller( 'IdeaCollaboratorsCtrl', function( $stateParams, IdeaDetail, UserInfo, Ideas, $filter, $route ) {
+  .controller( 'IdeaCollaboratorsCtrl', function( $stateParams, IdeaDetail, UserInfo, Ideas, $filter, $route, $location ) {
     var self = this;
 
     self.newCollaborator = {};
@@ -51,6 +51,9 @@ angular.module( 'glint.ideaDetail', [] )
       // Escape user input.
 
       self.newCollaborator.username = UserInfo.getUsername();
+      if ( self.newCollaborator.username === 'not available' ) {
+        $location.path( '/login' );
+      }
       self.newCollaborator.idea_id = self._id;
       self.newCollaborator.role = _.escape( self.newCollaborator.role );
       var collab = JSON.stringify( self.newCollaborator );
@@ -73,6 +76,9 @@ angular.module( 'glint.ideaDetail', [] )
 
       // Escape user input.
       self.newComment.created_by = UserInfo.getUsername();
+      if ( self.newComment.created_by === 'not available' ) {
+        $location.path( '/login' );
+      }
       self.newComment.idea_id = self._id;
       self.newComment.text = _.escape( self.newComment.text );
       var comm = JSON.stringify( self.newComment );