Don't display 0 in launch view

Summary: This includes assigned tasks in the Maniphest number.

Test Plan: Looked at it.

Reviewers: chad, epriestley

Reviewed By: epriestley

CC: aran, epriestley

Differential Revision: https://secure.phabricator.com/D5067

src/applications/audit/application/PhabricatorApplicationAudit.php

@@ -49,20 +49,6 @@ final class PhabricatorApplicationAudit extends PhabricatorApplication {
 
     $phids = PhabricatorAuditCommentEditor::loadAuditPHIDsForUser($user);
 
-    $audits = id(new PhabricatorAuditQuery())
-      ->withAuditorPHIDs($phids)
-      ->withStatus(PhabricatorAuditQuery::STATUS_OPEN)
-      ->withAwaitingUser($user)
-      ->execute();
-
-    $count = count($audits);
-    $type = PhabricatorApplicationStatusView::TYPE_INFO;
-    $status[] = id(new PhabricatorApplicationStatusView())
-      ->setType($type)
-      ->setText(pht('%d Commit(s) Awaiting Audit', $count))
-      ->setCount($count);
-
-
     $commits = id(new PhabricatorAuditCommitQuery())
       ->withAuthorPHIDs($phids)
       ->withStatus(PhabricatorAuditQuery::STATUS_OPEN)
@@ -75,6 +61,19 @@ final class PhabricatorApplicationAudit extends PhabricatorApplication {
       ->setText(pht('%d Problem Commit(s)', $count))
       ->setCount($count);
 
+    $audits = id(new PhabricatorAuditQuery())
+      ->withAuditorPHIDs($phids)
+      ->withStatus(PhabricatorAuditQuery::STATUS_OPEN)
+      ->withAwaitingUser($user)
+      ->execute();
+
+    $count = count($audits);
+    $type = PhabricatorApplicationStatusView::TYPE_WARNING;
+    $status[] = id(new PhabricatorApplicationStatusView())
+      ->setType($type)
+      ->setText(pht('%d Commit(s) Awaiting Audit', $count))
+      ->setCount($count);
+
     return $status;
   }
 

src/applications/differential/application/PhabricatorApplicationDifferential.php

@@ -95,7 +95,7 @@ final class PhabricatorApplicationDifferential extends PhabricatorApplication {
       ->setCount($blocking);
 
     $active = count($active);
-    $type = PhabricatorApplicationStatusView::TYPE_NEEDS_ATTENTION;
+    $type = PhabricatorApplicationStatusView::TYPE_WARNING;
     $status[] = id(new PhabricatorApplicationStatusView())
       ->setType($type)
       ->setText(pht('%d Review(s) Need Attention', $active))

src/applications/flag/application/PhabricatorApplicationFlags.php

@@ -32,7 +32,7 @@ final class PhabricatorApplicationFlags extends PhabricatorApplication {
       ->execute();
 
     $count = count($flags);
-    $type = PhabricatorApplicationStatusView::TYPE_NEEDS_ATTENTION;
+    $type = PhabricatorApplicationStatusView::TYPE_WARNING;
     $status[] = id(new PhabricatorApplicationStatusView())
       ->setType($type)
       ->setText(pht('%d Flagged Object(s)', $count))

src/applications/maniphest/application/PhabricatorApplicationManiphest.php

@@ -99,7 +99,7 @@ final class PhabricatorApplicationManiphest extends PhabricatorApplication {
     $query->execute();
 
     $count = $query->getRowCount();
-    $type = PhabricatorApplicationStatusView::TYPE_INFO;
+    $type = PhabricatorApplicationStatusView::TYPE_WARNING;
     $status[] = id(new PhabricatorApplicationStatusView())
       ->setType($type)
       ->setText(pht('%d Assigned Task(s)', $count))

src/applications/meta/view/PhabricatorApplicationLaunchView.php

@@ -59,9 +59,9 @@ final class PhabricatorApplicationLaunchView extends AphrontView {
       $count = 0;
       $text = array();
       if ($this->status) {
-        $attention = PhabricatorApplicationStatusView::TYPE_NEEDS_ATTENTION;
+        $info = PhabricatorApplicationStatusView::TYPE_INFO;
         foreach ($this->status as $status) {
-          if ($status->getType() == $attention) {
+          if ($status->getType() != $info) {
             $count += $status->getCount();
           }
           if ($status->getCount()) {
@@ -70,7 +70,7 @@ final class PhabricatorApplicationLaunchView extends AphrontView {
         }
       }
 
-      if ($text) {
+      if ($count) {
         Javelin::initBehavior('phabricator-tooltips');
         $content[] = javelin_tag(
           'span',