Differential D16096 Diff 38726 src/applications/diffusion/typeahead/DiffusionTaggedRepositoriesFunctionDatasource.php

Changeset View

Standalone View

src/applications/diffusion/typeahead/DiffusionTaggedRepositoriesFunctionDatasource.php

This file was added.

				<?php

				final class DiffusionTaggedRepositoriesFunctionDatasource
				extends PhabricatorTypeaheadCompositeDatasource {

				public function getBrowseTitle() {
				return pht('Browse Repositories');
				}

				public function getPlaceholderText() {
				return pht('Type tagged(<project>)...');
				}

				public function getDatasourceApplicationClass() {
				return 'PhabricatorProjectApplication';
				}

				public function getComponentDatasources() {
				return array(
				new PhabricatorProjectDatasource(),
				);
				}

				public function getDatasourceFunctions() {
				return array(
				'repositories' => array(
				epriestleyUnsubmitted Done Inline Actions This should be `'tagged'`. epriestley: This should be `'tagged'`.
				'name' => pht('Repositories: ...'),
				'arguments' => pht('project'),
				'summary' => pht('Find results for repositories of a project.'),
				'description' => pht(
				'This function allows you to find results for any of the `.
				`repositories of a project:'.
				"\n\n".
				'> projects'),
				epriestleyUnsubmitted Done Inline Actions This should have a usage example, like: > tagged(engineering) epriestley: This should have a usage example, like: ``` > tagged(engineering) ```
				),
				);
				}

				protected function didLoadResults(array $results) {
				foreach ($results as $result) {
				$result
				->setTokenType(PhabricatorTypeaheadTokenView::TYPE_FUNCTION)
				->setColor(null)
				->setPHID('repositories('.$result->getPHID().')')
				epriestleyUnsubmitted Done Inline Actions This should be `tagged(...)`. epriestley: This should be `tagged(...)`.
				->setDisplayName(pht('tagged: %s', $result->getDisplayName()))
				epriestleyUnsubmitted Done Inline Actions Prefer "Tagged" (with capital letter) over "tagged" (no capital letter) for consistency with other function datasources. epriestley: Prefer "Tagged" (with capital letter) over "tagged" (no capital letter) for consistency with…
				->setName($result->getName().' repositories');
				epriestleyUnsubmitted Done Inline Actions This is very minor, but prefer putting the function name (`'tagged '.`) first instead of last. I just changed this for some of the other functions recently in D16094. This improves some sorting behaviors, described in slightly more detail in T8510. epriestley: This is very minor, but prefer putting the function name (`'tagged '.`) first instead of last.
				}

				return $results;
				}

				protected function evaluateFunction($function, array $argv_list) {
				$phids = array();
				foreach ($argv_list as $argv) {
				$phids[] = head($argv);
				}

				$repositories = id(new PhabricatorRepositoryQuery())
				->setViewer($this->getViewer())
				->withEdgeLogicPHIDs(
				PhabricatorProjectObjectHasProjectEdgeType::EDGECONST,
				PhabricatorQueryConstraint::OPERATOR_OR,
				$phids)
				->execute();
				epriestleyUnsubmitted Done Inline Actions This block should only be indented one level. (We should ideally get lint to fix this automatically, but it's tricky.) epriestley: This block should only be indented one level. (We should ideally get lint to fix this…

				$results = array();

				foreach ($repositories as $repository) {
				$results[] = $repository->getPHID();
				}

				return $results;
				}

				public function renderFunctionTokens($function, array $argv_list) {
				$phids = array();
				foreach ($argv_list as $argv) {
				$phids[] = head($argv);
				}

				$tokens = $this->renderTokens($phids);
				foreach ($tokens as $token) {
				// Remove any project color on this token.
				$token->setColor(null);

				if ($token->isInvalid()) {
				$token
				->setValue(pht('Repositories: Invalid Project'));
				} else {
				$token
				->setTokenType(PhabricatorTypeaheadTokenView::TYPE_FUNCTION)
				->setKey('repositories('.$token->getKey().')')
				->setValue(pht('tagged: %s', $token->getValue()));
				epriestleyUnsubmitted Done Inline Actions Should be `tagged(` and `"Tagged"` for consistency with above. epriestley: Should be `tagged(` and `"Tagged"` for consistency with above.
				}
				}

				return $tokens;
				}

				}