Move the "Inline List" view to "DiffInlineCommentQuery"

Summary: Ref T13513. Continue removing usage sites for the obsolete "DifferentialInlineCommentQuery".

Test Plan: Viewed the inline list in Differential, saw sensible inlines.

Maniphest Tasks: T13513

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

Author: epriestley

src/applications/differential/controller/DifferentialRevisionInlinesController.php

@@ -24,11 +24,12 @@ public function handleRequest(AphrontRequest $request) {
     $revision_uri = $revision->getURI();
     $revision_title = $revision->getTitle();
 
-    $query = id(new DifferentialInlineCommentQuery())
+    $inlines = id(new DifferentialDiffInlineCommentQuery())
       ->setViewer($viewer)
-      ->needHidden(true)
-      ->withRevisionPHIDs(array($revision->getPHID()));
-    $inlines = $query->execute();
+      ->withRevisionPHIDs(array($revision->getPHID()))
+      ->withPublishedComments(true)
+      ->execute();
+    $inlines = mpull($inlines, 'newInlineCommentObject');
 
     $crumbs = $this->buildApplicationCrumbs();
     $crumbs->addTextCrumb($revision_monogram, $revision_uri);

src/applications/diffusion/controller/DiffusionCommitController.php

@@ -418,7 +418,8 @@ public function handleRequest(AphrontRequest $request) {
         $inlines = id(new DiffusionDiffInlineCommentQuery())
           ->setViewer($viewer)
           ->withCommitPHIDs(array($commit->getPHID()))
-          ->withVisibleComments(true)
+          ->withPublishedComments(true)
+          ->withPublishableComments(true)
           ->execute();
         $inlines = mpull($inlines, 'newInlineCommentObject');
 

src/applications/diffusion/controller/DiffusionDiffController.php

@@ -103,7 +103,8 @@ public function handleRequest(AphrontRequest $request) {
       ->setViewer($viewer)
       ->withCommitPHIDs(array($commit->getPHID()))
       ->withPathIDs(array($path_id))
-      ->withVisibleComments(true)
+      ->withPublishedComments(true)
+      ->withPublishableComments(true)
       ->execute();
     $inlines = mpull($inlines, 'newInlineCommentObject');
 

src/infrastructure/diff/query/PhabricatorDiffInlineCommentQuery.php

@@ -5,7 +5,7 @@ abstract class PhabricatorDiffInlineCommentQuery
 
   private $fixedStates;
   private $needReplyToComments;
-  private $visibleComments;
+  private $publishedComments;
   private $publishableComments;
 
   abstract protected function buildInlineCommentWhereClauseParts(
@@ -22,13 +22,13 @@ public function needReplyToComments($need_reply_to) {
     return $this;
   }
 
-  public function withVisibleComments($with_visible) {
-    $this->visibleComments = $with_visible;
+  public function withPublishableComments($with_publishable) {
+    $this->publishableComments = $with_publishable;
     return $this;
   }
 
-  public function withPublishableComments($with_publishable) {
-    $this->publishableComments = $with_publishable;
+  public function withPublishedComments($with_published) {
+    $this->publishedComments = $with_published;
     return $this;
   }
 
@@ -51,32 +51,30 @@ protected function buildWhereClauseParts(AphrontDatabaseConnection $conn) {
     $show_published = false;
     $show_publishable = false;
 
-    if ($this->visibleComments !== null) {
-      if (!$this->visibleComments) {
+    if ($this->publishableComments !== null) {
+      if (!$this->publishableComments) {
         throw new Exception(
           pht(
-            'Querying for comments that are not visible is '.
+            'Querying for comments that are "not publishable" is '.
             'not supported.'));
       }
-      $show_published = true;
       $show_publishable = true;
     }
 
-    if ($this->publishableComments !== null) {
-      if (!$this->publishableComments) {
+    if ($this->publishedComments !== null) {
+      if (!$this->publishedComments) {
         throw new Exception(
           pht(
-            'Querying for comments that are not publishable is '.
+            'Querying for comments that are "not published" is '.
             'not supported.'));
       }
-      $show_publishable = true;
+      $show_published = true;
     }
 
     if ($show_publishable || $show_published) {
       $clauses = array();
 
       if ($show_published) {
-        // Published comments are always visible.
         $clauses[] = qsprintf(
           $conn,
           '%T.transactionPHID IS NOT NULL',