revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-26 08:42:41 +01:00
Code Issues Releases Wiki Activity

Fix two minor display issues with the Conduit "*.search" API documentation

Browse source 
Summary:
Depends on D20685. Ref T13350. Currently:

  - When a SearchEngine parameter is marked as hidden from Conduit, we may still render a table of possible values. Instead, only render the table if the parameter is actually usable.
  - The table header is hard-coded to say `'statuses'`, which is just a silly mistake. (Most commonly, this table does have `statuses` constants.)

Test Plan: Viewed the Conduit API documentation for the new "slowvote.poll.search" API method, saw more sensible display behavior.

Maniphest Tasks: T13350

Differential Revision: https://secure.phabricator.com/D20686
This commit is contained in:
epriestley 2019-07-31 10:06:34 -07:00
parent 0b0ab1bd7c
commit f92480fb77
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/applications/search/engine/PhabricatorSearchEngineAPIMethod.php
View file

@ -198,6 +198,7 @@ EOTEXT
$label = $field->getLabel();
$constants = $field->newConduitConstants();
$show_table = false;
$type_object = $field->getConduitParameterType();
if ($type_object) {
@ -209,6 +210,7 @@ EOTEXT
' ',
phutil_tag('em', array(), pht('(See table below.)')),
);
$show_table = true;
}
} else {
$type = null;
@ -222,11 +224,11 @@ EOTEXT
$description,
);
if ($constants) {
if ($show_table) {
$constant_lists[] = $this->newRemarkupDocumentationView(
pht(
'Constants supported by the `%s` constraint:',
'statuses'));
$key));
$constants_rows = array();
foreach ($constants as $constant) {