Fixing comments

Author: Nathan219

client/controllers/instance/controllerInstance.js

@@ -13,9 +13,7 @@ function ControllerInstance(
   $rootScope,
   $scope,
   $state,
-  $log,
   $stateParams,
-  exists,
   fetchUser
 ) {
   var dataInstance = $scope.dataInstance = {
@@ -59,7 +57,15 @@ function ControllerInstance(
     async.waterfall([
       fetchUser,
       fetchInstance
-    ]);
+    ], function (err) {
+      if (err) {
+        $state.go('instance.instance', {
+          instanceName: '',
+          userName: $stateParams.userName
+        }, {reload: true});
+      }
+      errs.handler(err);
+    });
   }
   // watch showExplorer (toggle when user clicks file menu)
   // if no running container, return early (user shouldn't be able to even click

client/controllers/setup/controllerNew.js

@@ -10,8 +10,7 @@ function ControllerNew(
   fetchUser,
   $scope,
   $state,
-  uuid,
-  $stateParams
+  uuid
 ) {
 
   var thisUser;

client/directives/instanceList/directiveInstanceList.js

@@ -6,16 +6,10 @@ require('app')
  * the active account
  * @ngInject
  */
-function instanceList (
-  async,
-  determineActiveAccount,
+function instanceList(
   getInstanceClasses,
   getInstanceAltTitle,
-  QueryAssist,
-  fetchUser,
-  $rootScope,
-  $state,
-  user
+  $state
 ) {
   return {
     restrict: 'E',
@@ -26,9 +20,6 @@ function instanceList (
       state: '='
     },
     link: function ($scope, elem, attrs) {
-      // The sidebar should only update if the given instance list changes, which can happen in two
-      // ways.  The activeAccount changed and we fetched them, or the collection changed in the
-      // api-client because an instance was modified/created/removed.  Because of how the api-client
 
       $scope.stateToInstance = function (instance) {
         $state.go('instance.instance', {