From d785da8dcd795ff1252b44511536c0d8fd667203 Mon Sep 17 00:00:00 2001 From: Emily Stark Date: Fri, 1 Aug 2014 17:03:47 -0700 Subject: [PATCH] Skip unknown usernames rather than print "". Not sure what the original motivation for printing was (it shouldn't ever come up in normal operation, only if something weird happens like we manually unset a user's username in the database). But it's now rather inconvenient because we don't yet clean up deleted organizations from a site, so you would see for every authorized organization that has been deleted. While we should eventually clean up deleted organizations from a site's "authorized_orgs" field, for now this will clean up the output of this command. --- tools/deploy.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/tools/deploy.js b/tools/deploy.js index a8b033dcbe..be71b9801e 100644 --- a/tools/deploy.js +++ b/tools/deploy.js @@ -631,9 +631,7 @@ var listAuthorized = function (site) { process.stdout.write((auth.loggedInUsername() || "") + "\n"); _.each(info.authorized, function (username) { - if (! username) - process.stdout.write("" + "\n"); - else + if (username) process.stdout.write(username + "\n"); }); return 0;