Let diffusion.readmequery accept a commit

Summary:
Ref T2683. This should probably just be `diffusion.filecontentquery` but keep things as they are for now.

This method uses a commit, so accept one. Soon, this will save a bit of work.

Test Plan: Viewed readmes in main and browse views.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T2683

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

src/applications/diffusion/conduit/ConduitAPI_diffusion_readmequery_Method.php

@@ -1,8 +1,5 @@
 <?php
 
-/**
- * @group conduit
- */
 final class ConduitAPI_diffusion_readmequery_Method
   extends ConduitAPI_diffusion_abstractquery_Method {
 
@@ -19,6 +16,7 @@ final class ConduitAPI_diffusion_readmequery_Method
   protected function defineCustomParamTypes() {
     return array(
       'paths' => 'required array <string>',
+      'commit' => 'optional string',
     );
   }
 

src/applications/diffusion/controller/DiffusionBrowseDirectoryController.php

@@ -70,6 +70,7 @@ final class DiffusionBrowseDirectoryController
       'diffusion.readmequery',
       array(
         'paths' => $results->getPathDicts(),
+        'commit' => $drequest->getStableCommit(),
       ));
     if ($readme) {
       $box = new PHUIBoxView();

src/applications/diffusion/controller/DiffusionRepositoryController.php

@@ -83,7 +83,8 @@ final class DiffusionRepositoryController extends DiffusionController {
       $readme = $this->callConduitWithDiffusionRequest(
         'diffusion.readmequery',
         array(
-         'paths' => $browse_results->getPathDicts()
+         'paths' => $browse_results->getPathDicts(),
+         'commit' => $drequest->getStableCommit(),
         ));
     } else {
       $readme = null;