Prevent use of the "quality" constraint in the Badge search API

Summary:
Ref T12270. This just drops the constraint for now, rather than dealing with all the typecasting stuff and putting us in a position which will almost certainly require backward compatibility breaks in the future.

Also renames "badges.*" to "badge.*" for consistency (all other methods are singular: token.*, project.*, differential.revision.*, etc).

Test Plan:
Saw "qualities" now "Not Supported", while other constraints continue to work:

{F3887194}

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12270

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

Author: epriestley

src/applications/badges/conduit/PhabricatorBadgesEditConduitAPIMethod.php

@@ -4,7 +4,7 @@ final class PhabricatorBadgesEditConduitAPIMethod
   extends PhabricatorEditEngineAPIMethod {
 
   public function getAPIMethodName() {
-    return 'badges.edit';
+    return 'badge.edit';
   }
 
   public function newEditEngine() {

src/applications/badges/conduit/PhabricatorBadgesSearchConduitAPIMethod.php

@@ -4,7 +4,7 @@ final class PhabricatorBadgesSearchConduitAPIMethod
   extends PhabricatorSearchEngineAPIMethod {
 
   public function getAPIMethodName() {
-    return 'badges.search';
+    return 'badge.search';
   }
 
   public function newSearchEngine() {

src/applications/badges/lipsum/PhabricatorBadgesAwardTestDataGenerator.php

@@ -26,7 +26,7 @@ public function generateObject() {
       'objectIdentifier' => $badge_phid,
     );
 
-    $result = id(new ConduitCall('badges.edit', $params))
+    $result = id(new ConduitCall('badge.edit', $params))
       ->setUser($author)
       ->execute();
 

src/applications/badges/lipsum/PhabricatorBadgesBadgeTestDataGenerator.php

@@ -40,7 +40,7 @@ public function generateObject() {
       'transactions' => $xactions,
     );
 
-    $result = id(new ConduitCall('badges.edit', $params))
+    $result = id(new ConduitCall('badge.edit', $params))
       ->setUser($author)
       ->execute();
 

src/applications/badges/query/PhabricatorBadgesSearchEngine.php

@@ -24,6 +24,7 @@ protected function buildCustomSearchFields() {
       id(new PhabricatorSearchCheckboxesField())
         ->setKey('qualities')
         ->setLabel(pht('Quality'))
+        ->setEnableForConduit(false)
         ->setOptions(PhabricatorBadgesQuality::getDropdownQualityMap()),
       id(new PhabricatorSearchCheckboxesField())
         ->setKey('statuses')

src/applications/search/field/PhabricatorSearchField.php

@@ -19,6 +19,8 @@ abstract class PhabricatorSearchField extends Phobject {
   private $description;
   private $isHidden;
 
+  private $enableForConduit = true;
+
 
 /* -(  Configuring Fields  )------------------------------------------------- */
 
@@ -333,6 +335,10 @@ public function appendToForm(AphrontFormView $form) {
    * @task conduit
    */
   final public function getConduitParameterType() {
+    if (!$this->getEnableForConduit()) {
+      return false;
+    }
+
     $type = $this->newConduitParameterType();
 
     if ($type) {
@@ -367,6 +373,15 @@ public function getValidConstraintKeys() {
       $this->getConduitKey());
   }
 
+  final public function setEnableForConduit($enable) {
+    $this->enableForConduit = $enable;
+    return $this;
+  }
+
+  final public function getEnableForConduit() {
+    return $this->enableForConduit;
+  }
+
 
 /* -(  Utility Methods )----------------------------------------------------- */