From 2ed56f04c9b7a99f3593fcbb09fbdaa812aec847 Mon Sep 17 00:00:00 2001
From: Jimmy Zelinskie <jimmy.zelinskie@coreos.com>
Date: Mon, 12 Jan 2015 12:05:58 -0500
Subject: [PATCH] gruntfile: recursively include *.js in js dir

This CL also begins placing controllers into their own directory and
individual files.
---
 grunt/Gruntfile.js                |   2 +-
 static/js/controllers.js          | 143 ------------------------------
 static/js/controllers/repolist.js | 142 +++++++++++++++++++++++++++++
 3 files changed, 143 insertions(+), 144 deletions(-)
 create mode 100644 static/js/controllers/repolist.js

diff --git a/grunt/Gruntfile.js b/grunt/Gruntfile.js
index 526252bd1..5dd381e8d 100644
--- a/grunt/Gruntfile.js
+++ b/grunt/Gruntfile.js
@@ -25,7 +25,7 @@ module.exports = function(grunt) {
         },
       },
       build: {
-        src: ['../static/lib/**/*.js', '../static/js/*.js', '../static/dist/template-cache.js'],
+        src: ['../static/lib/**/*.js', '../static/js/**/*.js', '../static/dist/template-cache.js'],
         dest: '../static/dist/<%= pkg.name %>.js'
       }
     },
diff --git a/static/js/controllers.js b/static/js/controllers.js
index 53de635f0..df183a844 100644
--- a/static/js/controllers.js
+++ b/static/js/controllers.js
@@ -231,149 +231,6 @@ function TutorialCtrl($scope, AngularTour, AngularTourSignals, UserService, Conf
   };
 }
 
-function RepoListCtrl($scope, $sanitize, Restangular, UserService, ApiService) {
-  $scope.namespace = null;
-  $scope.page = 1;
-  $scope.publicPageCount = null;
-
-  // When loading the UserService, if the user is logged in, create a list of
-  // relevant namespaces for later collecting relevant repositories.
-  UserService.load(function() {
-    var user = UserService.currentUser();
-    if (!user.anonymous) {
-      $scope.namespaces = [user];
-      for (var i = 0; i < user.organizations.length; i++) {
-        $scope.namespaces.push(user.organizations[i]);
-      }
-      //loadStarredRepos();
-      //loadRepos();
-    }
-  });
-
-  // If someone signs in on this page, we have to watch the user and re-load
-  // their repositories after they've signed in to actually have any content
-  // on the page.
-  $scope.$watch(function(scope) { return scope.user },
-                function(user) {
-                  if (!user.anonymous) {
-                    $scope.namespaces = [user];
-                    for (var i = 0; i < user.organizations.length; i++) {
-                      $scope.namespaces.push(user.organizations[i]);
-                    }
-                    loadStarredRepos();
-                    loadRepos();
-                  }
-                });
-
-  $scope.toggleStar = function(repo) {
-    if (repo.is_starred) {
-      unstarRepo(repo);
-    } else {
-      starRepo(repo);
-    }
-  }
-
-  var starRepo = function(repo) {
-    var data = {
-      'namespace': repo.namespace,
-      'repository': repo.name
-    };
-    ApiService.createStar(data).then(function(result) {
-      updateReposAfterStar(repo);
-    }, function(result) {
-      // TODO(jzelinskie): have some kind of pop-up for star failure
-    });
-  };
-
-  var unstarRepo = function(repo) {
-    var data = {
-      'repository': repo.namespace + '/' + repo.name
-    };
-    ApiService.deleteStar(null, data).then(function(result) {
-      updateReposAfterUnstar(repo);
-    }, function(result) {
-      // TODO(jzelinskie): have some kind of pop-up for star failure
-    });
-  };
-
-  // Finds a repository within the list of namespaces attached to $scope.
-  var findRepoInList = function(repoNamespace, repoName) {
-    var namespaceIndex = $scope.namespaces.map(function (n) {
-      return n.username || n.name;
-    }).indexOf(repoNamespace);
-
-    var namespace = $scope.namespaces[namespaceIndex]
-
-    var repoIndex = namespace.repositories.value.map(function (r) {
-      return r.namespace + '/' + r.name;
-    }).indexOf(repoNamespace + '/' + repoName);
-
-    return repoIndex != -1 ? namespace.repositories.value[repoIndex] : null;
-  }
-
-  // Add a starred repository to the list starred repository list and make
-  // sure it appears starred elsewhere on the page.
-  var updateReposAfterStar = function(repository) {
-    $scope.starred_repositories.value.push(repository);
-
-    var repo = findRepoInList(repository.namespace, repository.name);
-    if (repo != null) {
-      repo.is_starred = true;
-    }
-  }
-
-  // Remove a repository from the starred repository list and make sure that
-  // it doesn't appear starred elsewhere on the page.
-  var updateReposAfterUnstar = function(repository) {
-    // Remove from the starred listings
-    var index = $scope.starred_repositories.value.map(function(r) {
-      return r.namespace + '/' + r.name;
-    }).indexOf(repository.namespace + '/' + repository.name);
-    $scope.starred_repositories.value.splice(index, 1);
-
-    // Set repo from the normal listings to unstarred.
-    var repo = findRepoInList(repository.namespace, repository.name);
-    if (repo != null) {
-      repo.is_starred = false;
-    }
-  };
-
-  var loadStarredRepos = function() {
-    if (!$scope.user || $scope.user.anonymous) {
-      return;
-    }
-
-    $scope.starred_repositories = ApiService.listStarredReposAsResource().get(function(resp) {
-      return resp.repositories.map(function(repo) {
-        repo.is_starred = true;
-        return repo;
-      });
-    });
-  };
-
-  // Iterate over all of the $scope.namespaces and collect their respective
-  // repositories.
-  var loadRepos = function() {
-    if ($scope.namespaces.length == 0 || $scope.user.anonymous) {
-      return;
-    }
-    console.log('load repos called');
-
-    for (var i = 0; i < $scope.namespaces.length; i++) {
-      var namespace = $scope.namespaces[i];
-      var namespaceName = namespace.username || namespace.name;
-      var options = {
-        'public': false,
-        'sort': true,
-        'namespace': namespaceName,
-      };
-      namespace.repositories = ApiService.listReposAsResource().withOptions(options).get(function(resp) {
-        return resp.repositories;
-      });
-    }
-  };
-}
-
 function LandingCtrl($scope, UserService, ApiService, Features, Config) {
   $scope.namespace = null;
   $scope.currentScreenshot = 'repo-view';
diff --git a/static/js/controllers/repolist.js b/static/js/controllers/repolist.js
new file mode 100644
index 000000000..f1d196570
--- /dev/null
+++ b/static/js/controllers/repolist.js
@@ -0,0 +1,142 @@
+function RepoListCtrl($scope, $sanitize, Restangular, UserService, ApiService) {
+  $scope.namespace = null;
+  $scope.page = 1;
+  $scope.publicPageCount = null;
+
+  // When loading the UserService, if the user is logged in, create a list of
+  // relevant namespaces for later collecting relevant repositories.
+  UserService.load(function() {
+    var user = UserService.currentUser();
+    if (!user.anonymous) {
+      $scope.namespaces = [user];
+      for (var i = 0; i < user.organizations.length; i++) {
+        $scope.namespaces.push(user.organizations[i]);
+      }
+      //loadStarredRepos();
+      //loadRepos();
+    }
+  });
+
+  // If someone signs in on this page, we have to watch the user and re-load
+  // their repositories after they've signed in to actually have any content
+  // on the page.
+  $scope.$watch(function(scope) { return scope.user },
+                function(user) {
+                  if (!user.anonymous) {
+                    $scope.namespaces = [user];
+                    for (var i = 0; i < user.organizations.length; i++) {
+                      $scope.namespaces.push(user.organizations[i]);
+                    }
+                    loadStarredRepos();
+                    loadRepos();
+                  }
+                });
+
+  $scope.toggleStar = function(repo) {
+    if (repo.is_starred) {
+      unstarRepo(repo);
+    } else {
+      starRepo(repo);
+    }
+  }
+
+  var starRepo = function(repo) {
+    var data = {
+      'namespace': repo.namespace,
+      'repository': repo.name
+    };
+    ApiService.createStar(data).then(function(result) {
+      updateReposAfterStar(repo);
+    }, function(result) {
+      // TODO(jzelinskie): have some kind of pop-up for star failure
+    });
+  };
+
+  var unstarRepo = function(repo) {
+    var data = {
+      'repository': repo.namespace + '/' + repo.name
+    };
+    ApiService.deleteStar(null, data).then(function(result) {
+      updateReposAfterUnstar(repo);
+    }, function(result) {
+      // TODO(jzelinskie): have some kind of pop-up for star failure
+    });
+  };
+
+  // Finds a repository within the list of namespaces attached to $scope.
+  var findRepoInList = function(repoNamespace, repoName) {
+    var namespaceIndex = $scope.namespaces.map(function (n) {
+      return n.username || n.name;
+    }).indexOf(repoNamespace);
+
+    var namespace = $scope.namespaces[namespaceIndex]
+
+    var repoIndex = namespace.repositories.value.map(function (r) {
+      return r.namespace + '/' + r.name;
+    }).indexOf(repoNamespace + '/' + repoName);
+
+    return repoIndex != -1 ? namespace.repositories.value[repoIndex] : null;
+  }
+
+  // Add a starred repository to the list starred repository list and make
+  // sure it appears starred elsewhere on the page.
+  var updateReposAfterStar = function(repository) {
+    $scope.starred_repositories.value.push(repository);
+
+    var repo = findRepoInList(repository.namespace, repository.name);
+    if (repo != null) {
+      repo.is_starred = true;
+    }
+  }
+
+  // Remove a repository from the starred repository list and make sure that
+  // it doesn't appear starred elsewhere on the page.
+  var updateReposAfterUnstar = function(repository) {
+    // Remove from the starred listings
+    var index = $scope.starred_repositories.value.map(function(r) {
+      return r.namespace + '/' + r.name;
+    }).indexOf(repository.namespace + '/' + repository.name);
+    $scope.starred_repositories.value.splice(index, 1);
+
+    // Set repo from the normal listings to unstarred.
+    var repo = findRepoInList(repository.namespace, repository.name);
+    if (repo != null) {
+      repo.is_starred = false;
+    }
+  };
+
+  var loadStarredRepos = function() {
+    if (!$scope.user || $scope.user.anonymous) {
+      return;
+    }
+
+    $scope.starred_repositories = ApiService.listStarredReposAsResource().get(function(resp) {
+      return resp.repositories.map(function(repo) {
+        repo.is_starred = true;
+        return repo;
+      });
+    });
+  };
+
+  // Iterate over all of the $scope.namespaces and collect their respective
+  // repositories.
+  var loadRepos = function() {
+    if ($scope.namespaces.length == 0 || $scope.user.anonymous) {
+      return;
+    }
+    console.log('load repos called');
+
+    for (var i = 0; i < $scope.namespaces.length; i++) {
+      var namespace = $scope.namespaces[i];
+      var namespaceName = namespace.username || namespace.name;
+      var options = {
+        'public': false,
+        'sort': true,
+        'namespace': namespaceName,
+      };
+      namespace.repositories = ApiService.listReposAsResource().withOptions(options).get(function(resp) {
+        return resp.repositories;
+      });
+    }
+  };
+}