diff --git a/src/aphront/AphrontRequest.php b/src/aphront/AphrontRequest.php
--- a/src/aphront/AphrontRequest.php
+++ b/src/aphront/AphrontRequest.php
@@ -539,7 +539,7 @@
*
* @param string $name Cookie name.
* @param string $value Cookie value.
- * @return this
+ * @return $this
* @task cookie
*/
public function setCookie($name, $value) {
@@ -555,7 +555,7 @@
*
* @param string $name Cookie name.
* @param string $value Cookie value.
- * @return this
+ * @return $this
* @task cookie
*/
public function setTemporaryCookie($name, $value) {
@@ -569,7 +569,7 @@
* @param string $name Cookie name.
* @param string $value Cookie value.
* @param int $expire Epoch timestamp for cookie expiration.
- * @return this
+ * @return $this
* @task cookie
*/
private function setCookieWithExpiration(
diff --git a/src/aphront/httpparametertype/AphrontHTTPParameterType.php b/src/aphront/httpparametertype/AphrontHTTPParameterType.php
--- a/src/aphront/httpparametertype/AphrontHTTPParameterType.php
+++ b/src/aphront/httpparametertype/AphrontHTTPParameterType.php
@@ -32,7 +32,7 @@
* to use the current viewer to execute queries.
*
* @param PhabricatorUser $viewer Current viewer.
- * @return this
+ * @return $this
* @task read
*/
final public function setViewer(PhabricatorUser $viewer) {
diff --git a/src/aphront/writeguard/AphrontWriteGuard.php b/src/aphront/writeguard/AphrontWriteGuard.php
--- a/src/aphront/writeguard/AphrontWriteGuard.php
+++ b/src/aphront/writeguard/AphrontWriteGuard.php
@@ -58,7 +58,7 @@
* valid token isn't present.
*
* @param $callback Callable CSRF callback.
- * @return this
+ * @return $this
* @task manage
*/
public function __construct($callback) {
diff --git a/src/applications/conduit/protocol/exception/ConduitException.php b/src/applications/conduit/protocol/exception/ConduitException.php
--- a/src/applications/conduit/protocol/exception/ConduitException.php
+++ b/src/applications/conduit/protocol/exception/ConduitException.php
@@ -13,7 +13,7 @@
* an exception (e.g., which values were wrong in an invalid request).
*
* @param string $error_description Detailed error description.
- * @return this
+ * @return $this
*/
final public function setErrorDescription($error_description) {
$this->errorDescription = $error_description;
diff --git a/src/applications/config/issue/PhabricatorSetupIssue.php b/src/applications/config/issue/PhabricatorSetupIssue.php
--- a/src/applications/config/issue/PhabricatorSetupIssue.php
+++ b/src/applications/config/issue/PhabricatorSetupIssue.php
@@ -147,7 +147,7 @@
*
* @param string $php_config PHP configuration option to provide a value for.
* @param string $value Explicit value to show in the UI.
- * @return this
+ * @return $this
*/
public function addPHPConfigOriginalValue($php_config, $value) {
$this->originalPHPConfigValues[$php_config] = $value;
diff --git a/src/applications/differential/parser/DifferentialChangesetParser.php b/src/applications/differential/parser/DifferentialChangesetParser.php
--- a/src/applications/differential/parser/DifferentialChangesetParser.php
+++ b/src/applications/differential/parser/DifferentialChangesetParser.php
@@ -246,7 +246,7 @@
* storage changeset. Note that this may be false, if the left
* side of some storage changeset is being shown as the right
* side of a display diff.
- * @return this
+ * @return $this
*/
public function setRightSideCommentMapping($id, $is_new) {
$this->rightSideChangesetID = $id;
@@ -297,7 +297,7 @@
*
* @param string $key Key for identifying this changeset in the render
* cache.
- * @return this
+ * @return $this
*/
public function setRenderCacheKey($key) {
$this->renderCacheKey = $key;
diff --git a/src/applications/differential/query/DifferentialRevisionQuery.php b/src/applications/differential/query/DifferentialRevisionQuery.php
--- a/src/applications/differential/query/DifferentialRevisionQuery.php
+++ b/src/applications/differential/query/DifferentialRevisionQuery.php
@@ -46,7 +46,7 @@
* Find revisions affecting one or more items in a list of paths.
*
* @param list<string> $paths List of file paths.
- * @return this
+ * @return $this
* @task config
*/
public function withPaths(array $paths) {
@@ -60,7 +60,7 @@
* @{method:withAuthors}.
*
* @param array $author_phids List of PHIDs of authors
- * @return this
+ * @return $this
* @task config
*/
public function withAuthors(array $author_phids) {
@@ -73,7 +73,7 @@
* function will clear anything set by previous calls to @{method:withCCs}.
*
* @param array $cc_phids List of PHIDs of subscribers.
- * @return this
+ * @return $this
* @task config
*/
public function withCCs(array $cc_phids) {
@@ -87,7 +87,7 @@
* to @{method:withReviewers}.
*
* @param array $reviewer_phids List of PHIDs of reviewers
- * @return this
+ * @return $this
* @task config
*/
public function withReviewers(array $reviewer_phids) {
@@ -127,7 +127,7 @@
* @param array $commit_hashes List of pairs <Class
* ArcanistDifferentialRevisionHash::HASH_$type constant,
* hash>
- * @return this
+ * @return $this
* @task config
*/
public function withCommitHashes(array $commit_hashes) {
@@ -150,7 +150,7 @@
* Filter results to revisions on given branches.
*
* @param list $branches List of branch names.
- * @return this
+ * @return $this
* @task config
*/
public function withBranches(array $branches) {
@@ -163,7 +163,7 @@
* Filter results to only return revisions whose ids are in the given set.
*
* @param array $ids List of revision ids
- * @return this
+ * @return $this
* @task config
*/
public function withIDs(array $ids) {
@@ -176,7 +176,7 @@
* Filter results to only return revisions whose PHIDs are in the given set.
*
* @param array $phids List of revision PHIDs
- * @return this
+ * @return $this
* @task config
*/
public function withPHIDs(array $phids) {
@@ -190,7 +190,7 @@
* responsible for (i.e., they are either authors or reviewers).
*
* @param array $responsible_phids List of user PHIDs.
- * @return this
+ * @return $this
* @task config
*/
public function withResponsibleUsers(array $responsible_phids) {
@@ -222,7 +222,7 @@
* revision.
*
* @param bool $need_active_diffs True to load and attach diffs.
- * @return this
+ * @return $this
* @task config
*/
public function needActiveDiffs($need_active_diffs) {
@@ -236,7 +236,7 @@
* each revision.
*
* @param bool $need_commit_phids True to load and attach diffs.
- * @return this
+ * @return $this
* @task config
*/
public function needCommitPHIDs($need_commit_phids) {
@@ -250,7 +250,7 @@
* revision.
*
* @param bool $need_diff_ids True to load and attach diff IDs.
- * @return this
+ * @return $this
* @task config
*/
public function needDiffIDs($need_diff_ids) {
@@ -264,7 +264,7 @@
* revision.
*
* @param bool $need_hashes True to load and attach commit hashes.
- * @return this
+ * @return $this
* @task config
*/
public function needHashes($need_hashes) {
@@ -277,7 +277,7 @@
* Set whether or not the query should load associated reviewers.
*
* @param bool $need_reviewers True to load and attach reviewers.
- * @return this
+ * @return $this
* @task config
*/
public function needReviewers($need_reviewers) {
@@ -292,7 +292,7 @@
* they are a member of.
*
* @param bool $need_reviewer_authority True to load and attach authority.
- * @return this
+ * @return $this
* @task config
*/
public function needReviewerAuthority($need_reviewer_authority) {
diff --git a/src/applications/diffusion/request/DiffusionRequest.php b/src/applications/diffusion/request/DiffusionRequest.php
--- a/src/applications/diffusion/request/DiffusionRequest.php
+++ b/src/applications/diffusion/request/DiffusionRequest.php
@@ -296,7 +296,7 @@
* Modify the request to move the symbolic commit elsewhere.
*
* @param string $symbol New symbolic commit.
- * @return this
+ * @return $this
*/
public function updateSymbolicCommit($symbol) {
$this->symbolicCommit = $symbol;
diff --git a/src/applications/diviner/query/DivinerAtomQuery.php b/src/applications/diviner/query/DivinerAtomQuery.php
--- a/src/applications/diviner/query/DivinerAtomQuery.php
+++ b/src/applications/diviner/query/DivinerAtomQuery.php
@@ -94,7 +94,7 @@
* restore the original data.
*
* @param bool $ghosts
- * @return this
+ * @return $this
*/
public function withGhosts($ghosts) {
$this->isGhost = $ghosts;
diff --git a/src/applications/doorkeeper/engine/DoorkeeperFeedStoryPublisher.php b/src/applications/doorkeeper/engine/DoorkeeperFeedStoryPublisher.php
--- a/src/applications/doorkeeper/engine/DoorkeeperFeedStoryPublisher.php
+++ b/src/applications/doorkeeper/engine/DoorkeeperFeedStoryPublisher.php
@@ -33,7 +33,7 @@
*
* @param bool $render_with_implied_context True to assume object context
* when rendering.
- * @return this
+ * @return $this
* @task config
*/
public function setRenderWithImpliedContext($render_with_implied_context) {
diff --git a/src/applications/drydock/storage/DrydockLease.php b/src/applications/drydock/storage/DrydockLease.php
--- a/src/applications/drydock/storage/DrydockLease.php
+++ b/src/applications/drydock/storage/DrydockLease.php
@@ -511,7 +511,7 @@
/**
* Awaken yielded tasks after a state change.
*
- * @return this
+ * @return $this
*/
public function awakenTasks() {
$awaken_ids = $this->getAttribute('internal.awakenTaskIDs');
diff --git a/src/applications/files/query/PhabricatorFileQuery.php b/src/applications/files/query/PhabricatorFileQuery.php
--- a/src/applications/files/query/PhabricatorFileQuery.php
+++ b/src/applications/files/query/PhabricatorFileQuery.php
@@ -98,7 +98,7 @@
*
* @param list<dict> $specs List of transform specifications, described
* above.
- * @return this
+ * @return $this
*/
public function withTransforms(array $specs) {
foreach ($specs as $spec) {
diff --git a/src/applications/files/storage/PhabricatorFile.php b/src/applications/files/storage/PhabricatorFile.php
--- a/src/applications/files/storage/PhabricatorFile.php
+++ b/src/applications/files/storage/PhabricatorFile.php
@@ -1472,7 +1472,7 @@
* This method is successful even if the file is already attached.
*
* @param phid $phid Object PHID to attach to.
- * @return this
+ * @return $this
*/
public function attachToObject($phid) {
self::attachFileToObject($this->getPHID(), $phid);
@@ -1526,7 +1526,7 @@
*
* @param map<string, wild> $params Bag of parameters, see
* @{class:PhabricatorFile} for documentation.
- * @return this
+ * @return $this
*/
private function readPropertiesFromParameters(array $params) {
PhutilTypeSpec::checkMap(
diff --git a/src/applications/herald/adapter/HeraldAdapter.php b/src/applications/herald/adapter/HeraldAdapter.php
--- a/src/applications/herald/adapter/HeraldAdapter.php
+++ b/src/applications/herald/adapter/HeraldAdapter.php
@@ -173,7 +173,7 @@
*
* @param list<PhabricatorApplicationTransaction> $xactions List of
* transactions.
- * @return this
+ * @return $this
*/
final public function setAppliedTransactions(array $xactions) {
assert_instances_of($xactions, 'PhabricatorApplicationTransaction');
diff --git a/src/applications/metamta/storage/PhabricatorMetaMTAMail.php b/src/applications/metamta/storage/PhabricatorMetaMTAMail.php
--- a/src/applications/metamta/storage/PhabricatorMetaMTAMail.php
+++ b/src/applications/metamta/storage/PhabricatorMetaMTAMail.php
@@ -89,7 +89,7 @@
* of mail, like updates when a task's projects change.
*
* @param list<const> $tags
- * @return this
+ * @return $this
*/
public function setMailTags(array $tags) {
$this->setParam('mailtags', array_unique($tags));
@@ -110,7 +110,7 @@
* T251 for more details.
*
* @param string $id The "Message-ID" of the email which precedes this one.
- * @return this
+ * @return $this
*/
public function setParentMessageID($id) {
$this->setParam('parent-message-id', $id);
@@ -420,7 +420,7 @@
* receive things like password resets.
*
* @param bool $force True to force delivery despite user preferences.
- * @return this
+ * @return $this
*/
public function setForceDelivery($force) {
$this->setParam('force', $force);
@@ -438,7 +438,7 @@
* dependent.
*
* @param bool $is_bulk True if the mail is automated bulk mail.
- * @return this
+ * @return $this
*/
public function setIsBulk($is_bulk) {
$this->setParam('is-bulk', $is_bulk);
@@ -458,7 +458,7 @@
* Message-ID, In-Reply-To or References headers.
* @param bool $is_first_message (optional) If true, indicates this is the
* first message in the thread.
- * @return this
+ * @return $this
*/
public function setThreadID($thread_id, $is_first_message = false) {
$this->setParam('thread-id', $thread_id);
@@ -478,14 +478,14 @@
* Save a newly created mail to the database. The mail will eventually be
* delivered by the MetaMTA daemon.
*
- * @return this
+ * @return $this
*/
public function saveAndSend() {
return $this->save();
}
/**
- * @return this
+ * @return $this
*/
public function save() {
if ($this->getID()) {
diff --git a/src/applications/metamta/view/PhabricatorMetaMTAMailBody.php b/src/applications/metamta/view/PhabricatorMetaMTAMailBody.php
--- a/src/applications/metamta/view/PhabricatorMetaMTAMailBody.php
+++ b/src/applications/metamta/view/PhabricatorMetaMTAMailBody.php
@@ -41,7 +41,7 @@
* Add a raw block of text to the email. This will be rendered as-is.
*
* @param string $text Block of text.
- * @return this
+ * @return $this
* @task compose
*/
public function addRawSection($text) {
@@ -102,7 +102,7 @@
*
* @param string $header Header text.
* @param string $section Section text.
- * @return this
+ * @return $this
* @task compose
*/
public function addTextSection($header, $section) {
@@ -156,7 +156,7 @@
* Add an attachment.
*
* @param PhabricatorMailAttachment $attachment Attachment.
- * @return this
+ * @return $this
* @task compose
*/
public function addAttachment(PhabricatorMailAttachment $attachment) {
diff --git a/src/applications/notification/query/PhabricatorNotificationQuery.php b/src/applications/notification/query/PhabricatorNotificationQuery.php
--- a/src/applications/notification/query/PhabricatorNotificationQuery.php
+++ b/src/applications/notification/query/PhabricatorNotificationQuery.php
@@ -33,7 +33,7 @@
*
* @param mixed $unread True or false to filter results by read status. Null
* to remove the filter.
- * @return this
+ * @return $this
* @task config
*/
public function withUnread($unread) {
diff --git a/src/applications/people/storage/PhabricatorUser.php b/src/applications/people/storage/PhabricatorUser.php
--- a/src/applications/people/storage/PhabricatorUser.php
+++ b/src/applications/people/storage/PhabricatorUser.php
@@ -478,7 +478,7 @@
* This is primarily useful for unit tests.
*
* @param string $identifier New timezone identifier.
- * @return this
+ * @return $this
* @task settings
*/
public function overrideTimezoneIdentifier($identifier) {
@@ -772,7 +772,7 @@
*
* @param wild $availability Availability cache data.
* @param int|null $ttl Cache TTL.
- * @return this
+ * @return $this
* @task availability
*/
public function writeAvailabilityCache(array $availability, $ttl) {
diff --git a/src/applications/people/storage/PhabricatorUserEmail.php b/src/applications/people/storage/PhabricatorUserEmail.php
--- a/src/applications/people/storage/PhabricatorUserEmail.php
+++ b/src/applications/people/storage/PhabricatorUserEmail.php
@@ -181,7 +181,7 @@
* Send a verification email from $user to this address.
*
* @param PhabricatorUser $user The user sending the verification.
- * @return this
+ * @return $this
* @task email
*/
public function sendVerificationEmail(PhabricatorUser $user) {
@@ -270,7 +270,7 @@
* recipient that this is now their account's new primary email address.
*
* @param PhabricatorUser $user The user sending the verification.
- * @return this
+ * @return $this
* @task email
*/
public function sendNewPrimaryEmail(PhabricatorUser $user) {
diff --git a/src/applications/phid/PhabricatorObjectHandle.php b/src/applications/phid/PhabricatorObjectHandle.php
--- a/src/applications/phid/PhabricatorObjectHandle.php
+++ b/src/applications/phid/PhabricatorObjectHandle.php
@@ -276,7 +276,7 @@
* @{method:isComplete} for an explanation of what it means to be complete.
*
* @param bool $complete True if the handle represents a complete object.
- * @return this
+ * @return $this
*/
public function setComplete($complete) {
$this->complete = $complete;
diff --git a/src/applications/phortune/currency/PhortuneCurrency.php b/src/applications/phortune/currency/PhortuneCurrency.php
--- a/src/applications/phortune/currency/PhortuneCurrency.php
+++ b/src/applications/phortune/currency/PhortuneCurrency.php
@@ -197,7 +197,7 @@
*
* @param string|null Currency string, or null to skip check.
* @param string|null Currency string, or null to skip check.
- * @return this
+ * @return $this
*/
public function assertInRange($minimum, $maximum) {
if ($minimum !== null && $maximum !== null) {
diff --git a/src/applications/phortune/query/PhortuneCartQuery.php b/src/applications/phortune/query/PhortuneCartQuery.php
--- a/src/applications/phortune/query/PhortuneCartQuery.php
+++ b/src/applications/phortune/query/PhortuneCartQuery.php
@@ -48,7 +48,7 @@
* Include or exclude carts which represent invoices with payments due.
*
* @param bool `true` to select invoices; `false` to exclude invoices.
- * @return this
+ * @return $this
*/
public function withInvoices($invoices) {
$this->invoices = $invoices;
diff --git a/src/applications/search/field/PhabricatorSearchField.php b/src/applications/search/field/PhabricatorSearchField.php
--- a/src/applications/search/field/PhabricatorSearchField.php
+++ b/src/applications/search/field/PhabricatorSearchField.php
@@ -34,7 +34,7 @@
* does not contain any special characters.
*
* @param string $key Unique key which identifies the field.
- * @return this
+ * @return $this
* @task config
*/
public function setKey($key) {
@@ -60,7 +60,7 @@
* This should be a short text string, like "Reviewers" or "Colors".
*
* @param string $label Short, human-readable field label.
- * @return this
+ * @return $this
* task config
*/
public function setLabel($label) {
@@ -87,7 +87,7 @@
* behalf by the caller.
*
* @param PhabricatorUser $viewer Viewer.
- * @return this
+ * @return $this
* @task config
*/
public function setViewer(PhabricatorUser $viewer) {
@@ -116,7 +116,7 @@
* `&authorPHIDs=alincoln`. This is a little easier to use.
*
* @param list<string> $aliases List of aliases for this field.
- * @return this
+ * @return $this
* @task config
*/
public function setAliases(array $aliases) {
@@ -143,7 +143,7 @@
* If no key is provided, the main key is used.
*
* @param string $conduit_key Alternate key for Conduit.
- * @return this
+ * @return $this
* @task config
*/
public function setConduitKey($conduit_key) {
@@ -171,7 +171,7 @@
* Set a human-readable description for this field.
*
* @param string $description Human-readable description.
- * @return this
+ * @return $this
* @task config
*/
public function setDescription($description) {
@@ -195,7 +195,7 @@
* Hide this field from the web UI.
*
* @param bool $is_hidden True to hide the field from the web UI.
- * @return this
+ * @return $this
* @task config
*/
public function setIsHidden($is_hidden) {
diff --git a/src/applications/subscriptions/editor/PhabricatorSubscriptionsEditor.php b/src/applications/subscriptions/editor/PhabricatorSubscriptionsEditor.php
--- a/src/applications/subscriptions/editor/PhabricatorSubscriptionsEditor.php
+++ b/src/applications/subscriptions/editor/PhabricatorSubscriptionsEditor.php
@@ -19,7 +19,7 @@
* had previously unsubscribed.
*
* @param list<phid> $phids List of PHIDs to explicitly subscribe.
- * @return this
+ * @return $this
*/
public function subscribeExplicit(array $phids) {
$this->explicitSubscribePHIDs += array_fill_keys($phids, true);
@@ -33,7 +33,7 @@
* suppressed if they've previously unsubscribed from the object.
*
* @param list<phid> $phids List of PHIDs to implicitly subscribe.
- * @return this
+ * @return $this
*/
public function subscribeImplicit(array $phids) {
$this->implicitSubscribePHIDs += array_fill_keys($phids, true);
@@ -46,7 +46,7 @@
* will not resubscribe them.
*
* @param list<phid> $phids List of PHIDs to unsubscribe.
- * @return this
+ * @return $this
*/
public function unsubscribe(array $phids) {
$this->unsubscribePHIDs += array_fill_keys($phids, true);
diff --git a/src/applications/transactions/editengine/PhabricatorEditEngine.php b/src/applications/transactions/editengine/PhabricatorEditEngine.php
--- a/src/applications/transactions/editengine/PhabricatorEditEngine.php
+++ b/src/applications/transactions/editengine/PhabricatorEditEngine.php
@@ -699,7 +699,7 @@
* Flag this workflow as a create or edit.
*
* @param bool $is_create True if this is a create workflow.
- * @return this
+ * @return $this
* @task load
*/
private function setIsCreate($is_create) {
diff --git a/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php b/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
--- a/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
+++ b/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
@@ -142,7 +142,7 @@
*
* @param bool $continue True to drop transactions without effect and
* continue.
- * @return this
+ * @return $this
*/
public function setContinueOnNoEffect($continue) {
$this->continueOnNoEffect = $continue;
@@ -171,7 +171,7 @@
*
* @param bool $continue_on_missing_fields True to continue when transactions
* don't completely satisfy all required fields.
- * @return this
+ * @return $this
*/
public function setContinueOnMissingFields($continue_on_missing_fields) {
$this->continueOnMissingFields = $continue_on_missing_fields;
@@ -4603,7 +4603,7 @@
*
* @param dict<string, wild> $state Editor state, from
@{method:getWorkerState}.
- * @return this
+ * @return $this
* @task workers
*/
final public function loadWorkerState(array $state) {
@@ -4630,7 +4630,7 @@
*
* @param dict<string, wild> $state Custom state,
* from @{method:getCustomWorkerState}.
- * @return this
+ * @return $this
* @task workers
*/
protected function loadCustomWorkerState(array $state) {
diff --git a/src/infrastructure/cache/PhutilInRequestKeyValueCache.php b/src/infrastructure/cache/PhutilInRequestKeyValueCache.php
--- a/src/infrastructure/cache/PhutilInRequestKeyValueCache.php
+++ b/src/infrastructure/cache/PhutilInRequestKeyValueCache.php
@@ -27,7 +27,7 @@
* cache. Setting a limit of `0` disables the cache.
*
* @param int $limit Maximum number of items to store in the cache.
- * @return this
+ * @return $this
*/
public function setLimit($limit) {
$this->limit = $limit;
diff --git a/src/infrastructure/cache/PhutilKeyValueCache.php b/src/infrastructure/cache/PhutilKeyValueCache.php
--- a/src/infrastructure/cache/PhutilKeyValueCache.php
+++ b/src/infrastructure/cache/PhutilKeyValueCache.php
@@ -52,7 +52,7 @@
* @param string $key Key to set.
* @param wild $value Value to set.
* @param int|null $ttl (optional) TTL.
- * @return this
+ * @return $this
* @task kvimpl
*/
final public function setKey($key, $value, $ttl = null) {
@@ -65,7 +65,7 @@
* keys at once.
*
* @param string $key Key to delete.
- * @return this
+ * @return $this
* @task kvimpl
*/
final public function deleteKey($key) {
@@ -94,7 +94,7 @@
*
* @param dict<string, wild> $keys Map of cache keys to values.
* @param int|null $ttl (optional) TTL for cache keys, in seconds.
- * @return this
+ * @return $this
* @task kvimpl
*/
abstract public function setKeys(array $keys, $ttl = null);
@@ -104,7 +104,7 @@
* Delete a list of keys from the cache.
*
* @param list<string> $keys List of keys to delete.
- * @return this
+ * @return $this
* @task kvimpl
*/
abstract public function deleteKeys(array $keys);
@@ -113,7 +113,7 @@
/**
* Completely destroy all data in the cache.
*
- * @return this
+ * @return $this
* @task kvimpl
*/
abstract public function destroyCache();
diff --git a/src/infrastructure/cache/PhutilKeyValueCacheProfiler.php b/src/infrastructure/cache/PhutilKeyValueCacheProfiler.php
--- a/src/infrastructure/cache/PhutilKeyValueCacheProfiler.php
+++ b/src/infrastructure/cache/PhutilKeyValueCacheProfiler.php
@@ -18,7 +18,7 @@
* Set a profiler for cache operations.
*
* @param PhutilServiceProfiler $profiler Service profiler.
- * @return this
+ * @return $this
* @task kvimpl
*/
public function setProfiler(PhutilServiceProfiler $profiler) {
diff --git a/src/infrastructure/cache/PhutilKeyValueCacheStack.php b/src/infrastructure/cache/PhutilKeyValueCacheStack.php
--- a/src/infrastructure/cache/PhutilKeyValueCacheStack.php
+++ b/src/infrastructure/cache/PhutilKeyValueCacheStack.php
@@ -40,7 +40,7 @@
*
* @param list<PhutilKeyValueCache> $caches Ordered list of key-value
* caches.
- * @return this
+ * @return $this
* @task config
*/
public function setCaches(array $caches) {
@@ -64,7 +64,7 @@
* $stack->getKey('hedgehog');
*
* @param int $ttl TTL in seconds.
- * @return this
+ * @return $this
*
* @task config
*/
diff --git a/src/infrastructure/cache/PhutilMemcacheKeyValueCache.php b/src/infrastructure/cache/PhutilMemcacheKeyValueCache.php
--- a/src/infrastructure/cache/PhutilMemcacheKeyValueCache.php
+++ b/src/infrastructure/cache/PhutilMemcacheKeyValueCache.php
@@ -101,7 +101,7 @@
* ));
*
* @param list<dict> $servers List of server specifications.
- * @return this
+ * @return $this
* @task memcache
*/
public function setServers(array $servers) {
diff --git a/src/infrastructure/customfield/field/PhabricatorCustomField.php b/src/infrastructure/customfield/field/PhabricatorCustomField.php
--- a/src/infrastructure/customfield/field/PhabricatorCustomField.php
+++ b/src/infrastructure/customfield/field/PhabricatorCustomField.php
@@ -383,7 +383,7 @@
* discussion of field proxies.
*
* @param PhabricatorCustomField $proxy Field implementation.
- * @return this
+ * @return $this
* @task proxy
*/
final public function setProxy(PhabricatorCustomField $proxy) {
@@ -423,7 +423,7 @@
*
* @param PhabricatorCustomFieldInterface $object The object this field
* belongs to.
- * @return this
+ * @return $this
* @task context
*/
final public function setObject(PhabricatorCustomFieldInterface $object) {
@@ -443,7 +443,7 @@
*
* @param PhabricatorCustomFieldInterface $object The object this field
* belongs to.
- * @return this
+ * @return $this
* @task context
*/
public function readValueFromObject(PhabricatorCustomFieldInterface $object) {
@@ -612,7 +612,7 @@
* @param string|null $value Serialized field representation (from
* @{method:getValueForStorage}) or null if no value has
* ever been stored.
- * @return this
+ * @return $this
* @task storage
*/
public function setValueFromStorage($value) {
diff --git a/src/infrastructure/daemon/PhutilDaemon.php b/src/infrastructure/daemon/PhutilDaemon.php
--- a/src/infrastructure/daemon/PhutilDaemon.php
+++ b/src/infrastructure/daemon/PhutilDaemon.php
@@ -344,7 +344,7 @@
* that are part of an autoscale pool are continuously busy for a prolonged
* period of time, the overseer may scale up the pool.
*
- * @return this
+ * @return $this
* @task autoscale
*/
protected function willBeginWork() {
@@ -365,7 +365,7 @@
* that are part of an autoscale pool are idle for a prolonged period of
* time, they may exit to scale the pool down.
*
- * @return this
+ * @return $this
* @task autoscale
*/
protected function willBeginIdle() {
diff --git a/src/infrastructure/daemon/workers/PhabricatorWorker.php b/src/infrastructure/daemon/workers/PhabricatorWorker.php
--- a/src/infrastructure/daemon/workers/PhabricatorWorker.php
+++ b/src/infrastructure/daemon/workers/PhabricatorWorker.php
@@ -233,7 +233,7 @@
* @param string $class Task class to queue.
* @param array $data Data for the followup task.
* @param array $options (optional) Options for the followup task.
- * @return this
+ * @return $this
*/
final protected function queueTask(
$class,
diff --git a/src/infrastructure/daemon/workers/query/PhabricatorWorkerLeaseQuery.php b/src/infrastructure/daemon/workers/query/PhabricatorWorkerLeaseQuery.php
--- a/src/infrastructure/daemon/workers/query/PhabricatorWorkerLeaseQuery.php
+++ b/src/infrastructure/daemon/workers/query/PhabricatorWorkerLeaseQuery.php
@@ -60,7 +60,7 @@
*
* @param mixed $leased `true` to select only leased tasks, `false` to select
* only unleased tasks (default), or `null` to select both.
- * @return this
+ * @return $this
*/
public function withLeasedTasks($leased) {
$this->leased = $leased;
diff --git a/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php b/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php
--- a/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php
+++ b/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php
@@ -60,7 +60,7 @@
* ordering when querying for specific triggers, e.g. by ID or PHID.
*
* @param const $order Result order.
- * @return this
+ * @return $this
*/
public function setOrder($order) {
$this->order = $order;
diff --git a/src/infrastructure/diff/PhabricatorDifferenceEngine.php b/src/infrastructure/diff/PhabricatorDifferenceEngine.php
--- a/src/infrastructure/diff/PhabricatorDifferenceEngine.php
+++ b/src/infrastructure/diff/PhabricatorDifferenceEngine.php
@@ -22,7 +22,7 @@
* Set the name to identify the old file with. Primarily cosmetic.
*
* @param string $old_name Old file name.
- * @return this
+ * @return $this
* @task config
*/
public function setOldName($old_name) {
@@ -35,7 +35,7 @@
* Set the name to identify the new file with. Primarily cosmetic.
*
* @param string $new_name New file name.
- * @return this
+ * @return $this
* @task config
*/
public function setNewName($new_name) {
diff --git a/src/infrastructure/edges/editor/PhabricatorEdgeEditor.php b/src/infrastructure/edges/editor/PhabricatorEdgeEditor.php
--- a/src/infrastructure/edges/editor/PhabricatorEdgeEditor.php
+++ b/src/infrastructure/edges/editor/PhabricatorEdgeEditor.php
@@ -49,7 +49,7 @@
* @param const $type Edge type constant.
* @param phid $dst Destination object PHID.
* @param map $options (optional) Options map (see documentation).
- * @return this
+ * @return $this
*
* @task edit
*/
@@ -70,7 +70,7 @@
* @param phid $src Source object PHID.
* @param const $type Edge type constant.
* @param phid $dst Destination object PHID.
- * @return this
+ * @return $this
*
* @task edit
*/
diff --git a/src/infrastructure/edges/query/PhabricatorEdgeQuery.php b/src/infrastructure/edges/query/PhabricatorEdgeQuery.php
--- a/src/infrastructure/edges/query/PhabricatorEdgeQuery.php
+++ b/src/infrastructure/edges/query/PhabricatorEdgeQuery.php
@@ -41,7 +41,7 @@
* to execute an edge query.
*
* @param list $source_phids List of source PHIDs.
- * @return this
+ * @return $this
*
* @task config
*/
@@ -62,7 +62,7 @@
* Find edges terminating at one or more destination PHIDs.
*
* @param list $dest_phids List of destination PHIDs.
- * @return this
+ * @return $this
*
*/
public function withDestinationPHIDs(array $dest_phids) {
@@ -75,7 +75,7 @@
* Find edges of specific types.
*
* @param list $types List of PhabricatorEdgeConfig type constants.
- * @return this
+ * @return $this
*
* @task config
*/
@@ -89,7 +89,7 @@
* Configure the order edge results are returned in.
*
* @param const $order Order constant.
- * @return this
+ * @return $this
*
* @task config
*/
@@ -103,7 +103,7 @@
* When loading edges, also load edge data.
*
* @param bool $need True to load edge data.
- * @return this
+ * @return $this
*
* @task config
*/
diff --git a/src/infrastructure/env/PhabricatorScopedEnv.php b/src/infrastructure/env/PhabricatorScopedEnv.php
--- a/src/infrastructure/env/PhabricatorScopedEnv.php
+++ b/src/infrastructure/env/PhabricatorScopedEnv.php
@@ -19,7 +19,7 @@
*
* @param string $key Key to override.
* @param wild $value New value.
- * @return this
+ * @return $this
*
* @task override
*/
diff --git a/src/infrastructure/markup/PhabricatorMarkupEngine.php b/src/infrastructure/markup/PhabricatorMarkupEngine.php
--- a/src/infrastructure/markup/PhabricatorMarkupEngine.php
+++ b/src/infrastructure/markup/PhabricatorMarkupEngine.php
@@ -84,7 +84,7 @@
*
* @param PhabricatorMarkupInterface $object The object to render.
* @param string $field The field to render.
- * @return this
+ * @return $this
* @task markup
*/
public function addObject(PhabricatorMarkupInterface $object, $field) {
@@ -102,7 +102,7 @@
* Process objects queued with @{method:addObject}. You can then retrieve
* the output with @{method:getOutput}.
*
- * @return this
+ * @return $this
* @task markup
*/
public function process() {
@@ -319,7 +319,7 @@
* Set the viewing user. Used to implement object permissions.
*
* @param PhabricatorUser $viewer The viewing user.
- * @return this
+ * @return $this
* @task markup
*/
public function setViewer(PhabricatorUser $viewer) {
@@ -331,7 +331,7 @@
* Set the context object. Used to implement object permissions.
*
* @param $object The object in which context this remarkup is used.
- * @return this
+ * @return $this
* @task markup
*/
public function setContextObject($object) {
diff --git a/src/infrastructure/markup/PhutilMarkupEngine.php b/src/infrastructure/markup/PhutilMarkupEngine.php
--- a/src/infrastructure/markup/PhutilMarkupEngine.php
+++ b/src/infrastructure/markup/PhutilMarkupEngine.php
@@ -10,7 +10,7 @@
*
* @param string $key Key to set in the configuration dictionary.
* @param string $value Value to set.
- * @return this
+ * @return $this
*/
abstract public function setConfig($key, $value);
diff --git a/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php b/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php
--- a/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php
+++ b/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php
@@ -885,7 +885,7 @@
* also more involved.
*
* @param string $order Key of a builtin order supported by this query.
- * @return this
+ * @return $this
* @task order
*/
public function setOrder($order) {
@@ -919,7 +919,7 @@
*
* @param PhabricatorQueryOrderVector|list<string> $vector List of order
* keys.
- * @return this
+ * @return $this
* @task order
*/
public function setGroupVector($vector) {
@@ -1044,7 +1044,7 @@
*
* @param PhabricatorQueryOrderVector|list<string> $vector List of order
* keys.
- * @return this
+ * @return $this
* @task order
*/
public function setOrderVector($vector) {
@@ -1358,7 +1358,7 @@
* @param PhabricatorCustomFieldIndexStorage $index Table where the index is
* stored.
* @param string|list<string> $value One or more values to filter by.
- * @return this
+ * @return $this
* @task appsearch
*/
public function withApplicationSearchContainsConstraint(
@@ -1410,7 +1410,7 @@
* stored.
* @param int|null $min Minimum permissible value, inclusive.
* @param int|null $max Maximum permissible value, inclusive.
- * @return this
+ * @return $this
* @task appsearch
*/
public function withApplicationSearchRangeConstraint(
@@ -2590,7 +2590,7 @@
* @param const $edge_type Edge constant.
* @param const $operator Constraint operator.
* @param list<phid> $phids List of PHIDs.
- * @return this
+ * @return $this
* @task edgelogic
*/
public function withEdgeLogicPHIDs($edge_type, $operator, array $phids) {
@@ -2604,7 +2604,7 @@
/**
- * @return this
+ * @return $this
* @task edgelogic
*/
public function withEdgeLogicConstraints($edge_type, array $constraints) {
@@ -3001,7 +3001,7 @@
/**
* Validate edge logic constraints for the query.
*
- * @return this
+ * @return $this
* @task edgelogic
*/
private function validateEdgeLogicConstraints() {
diff --git a/src/infrastructure/query/policy/PhabricatorPolicyAwareQuery.php b/src/infrastructure/query/policy/PhabricatorPolicyAwareQuery.php
--- a/src/infrastructure/query/policy/PhabricatorPolicyAwareQuery.php
+++ b/src/infrastructure/query/policy/PhabricatorPolicyAwareQuery.php
@@ -58,7 +58,7 @@
* a policy query.
*
* @param PhabricatorUser $viewer The viewing user.
- * @return this
+ * @return $this
* @task config
*/
final public function setViewer(PhabricatorUser $viewer) {
@@ -83,7 +83,7 @@
* that configuration like whether or not to raise policy exceptions is
* seamlessly passed along to child queries.
*
- * @return this
+ * @return $this
* @task config
*/
final public function setParentQuery(PhabricatorPolicyAwareQuery $query) {
@@ -106,7 +106,7 @@
/**
* Hook to configure whether this query should raise policy exceptions.
*
- * @return this
+ * @return $this
* @task config
*/
final public function setRaisePolicyExceptions($bool) {
@@ -475,7 +475,7 @@
*
* @param map<phid, PhabricatorPolicyInterface> $objects Objects to add to
* the query workspace.
- * @return this
+ * @return $this
* @task workspace
*/
public function putObjectsInWorkspace(array $objects) {
@@ -507,7 +507,7 @@
* queries.
*
* @param list<phid> $phids List of PHIDs to retrieve.
- * @return this
+ * @return $this
* @task workspace
*/
public function getObjectsFromWorkspace(array $phids) {
@@ -537,7 +537,7 @@
* list can prevent infinite query loops by aborting queries which cycle.
*
* @param list<phid> $phids List of PHIDs which are now in flight.
- * @return this
+ * @return $this
*/
public function putPHIDsInFlight(array $phids) {
foreach ($phids as $phid) {
diff --git a/src/infrastructure/storage/connection/AphrontDatabaseConnection.php b/src/infrastructure/storage/connection/AphrontDatabaseConnection.php
--- a/src/infrastructure/storage/connection/AphrontDatabaseConnection.php
+++ b/src/infrastructure/storage/connection/AphrontDatabaseConnection.php
@@ -164,7 +164,7 @@
* Begin a transaction, or set a savepoint if the connection is already
* transactional.
*
- * @return this
+ * @return $this
* @task xaction
*/
public function openTransaction() {
@@ -189,7 +189,7 @@
* Commit a transaction, or stage a savepoint for commit once the entire
* transaction completes if inside a transaction stack.
*
- * @return this
+ * @return $this
* @task xaction
*/
public function saveTransaction() {
@@ -208,7 +208,7 @@
* Rollback a transaction, or unstage the last savepoint if inside a
* transaction stack.
*
- * @return this
+ * @return $this
*/
public function killTransaction() {
$state = $this->getTransactionState();
diff --git a/src/infrastructure/storage/lisk/LiskDAO.php b/src/infrastructure/storage/lisk/LiskDAO.php
--- a/src/infrastructure/storage/lisk/LiskDAO.php
+++ b/src/infrastructure/storage/lisk/LiskDAO.php
@@ -265,7 +265,7 @@
* @param mode $mode Connection mode.
* @param AphrontDatabaseConnection $connection Connection to cache.
* @param bool $force_unique (optional)
- * @return this
+ * @return $this
* @task conn
*/
protected function setEstablishedConnection(
@@ -548,7 +548,7 @@
* properties. This is primarily useful after entering a transaction but
* before applying changes to an object.
*
- * @return this
+ * @return $this
*
* @task load
*/
@@ -580,7 +580,7 @@
* @param dict $row Dictionary of properties, which should be equivalent
* to selecting a row from the table or calling
* @{method:getProperties}.
- * @return this
+ * @return $this
*
* @task load
*/
@@ -694,7 +694,7 @@
* method unless with `IDS_MANUAL`.
*
* @param mixed $id Unique ID.
- * @return this
+ * @return $this
* @task save
*/
public function setID($id) {
@@ -918,7 +918,7 @@
* method you need to call to do writes. If the object has not yet been
* inserted this will do an insert; if it has, it will do an update.
*
- * @return this
+ * @return $this
*
* @task save
*/
@@ -935,7 +935,7 @@
* Save this object, forcing the query to use REPLACE regardless of object
* state.
*
- * @return this
+ * @return $this
*
* @task save
*/
@@ -949,7 +949,7 @@
* Save this object, forcing the query to use INSERT regardless of object
* state.
*
- * @return this
+ * @return $this
*
* @task save
*/
@@ -963,7 +963,7 @@
* Save this object, forcing the query to use UPDATE regardless of object
* state.
*
- * @return this
+ * @return $this
*
* @task save
*/
@@ -1019,7 +1019,7 @@
/**
* Delete this object, permanently.
*
- * @return this
+ * @return $this
*
* @task save
*/
@@ -1044,7 +1044,7 @@
*
* @param const $mode Either "INSERT" or "REPLACE", to force the desired
* mode.
- * @return this
+ * @return $this
*
* @task save
*/
@@ -1309,7 +1309,7 @@
/**
* Increase transaction stack depth.
*
- * @return this
+ * @return $this
*/
public function openTransaction() {
$this->establishConnection('w')->openTransaction();
@@ -1320,7 +1320,7 @@
/**
* Decrease transaction stack depth, saving work.
*
- * @return this
+ * @return $this
*/
public function saveTransaction() {
$this->establishConnection('w')->saveTransaction();
@@ -1331,7 +1331,7 @@
/**
* Decrease transaction stack depth, discarding work.
*
- * @return this
+ * @return $this
*/
public function killTransaction() {
$this->establishConnection('w')->killTransaction();
@@ -1355,7 +1355,7 @@
* $beach->endReadLocking();
* $beach->saveTransaction();
*
- * @return this
+ * @return $this
* @task xaction
*/
public function beginReadLocking() {
@@ -1367,7 +1367,7 @@
/**
* Ends read-locking that began at an earlier @{method:beginReadLocking} call.
*
- * @return this
+ * @return $this
* @task xaction
*/
public function endReadLocking() {
@@ -1382,7 +1382,7 @@
* MySQL documentation for details). To end write locking, call
* @{method:endWriteLocking}.
*
- * @return this
+ * @return $this
* @task xaction
*/
public function beginWriteLocking() {
@@ -1395,7 +1395,7 @@
* Ends write-locking that began at an earlier @{method:beginWriteLocking}
* call.
*
- * @return this
+ * @return $this
* @task xaction
*/
public function endWriteLocking() {
diff --git a/src/infrastructure/util/PhabricatorGlobalLock.php b/src/infrastructure/util/PhabricatorGlobalLock.php
--- a/src/infrastructure/util/PhabricatorGlobalLock.php
+++ b/src/infrastructure/util/PhabricatorGlobalLock.php
@@ -89,7 +89,7 @@
* be used to lock on a specific connection.
*
* @param AphrontDatabaseConnection $conn
- * @return this
+ * @return $this
*/
public function setExternalConnection(AphrontDatabaseConnection $conn) {
if ($this->conn) {
diff --git a/src/view/AphrontView.php b/src/view/AphrontView.php
--- a/src/view/AphrontView.php
+++ b/src/view/AphrontView.php
@@ -17,7 +17,7 @@
* Set the user viewing this element.
*
* @param PhabricatorUser $viewer Viewing user.
- * @return this
+ * @return $this
*/
public function setViewer(PhabricatorUser $viewer) {
$this->viewer = $viewer;
@@ -101,7 +101,7 @@
* determined by @{method:canAppendChild}.
*
* @param wild $child Something renderable.
- * @return this
+ * @return $this
*/
final public function appendChild($child) {
if (!$this->canAppendChild()) {
diff --git a/src/view/form/AphrontFormView.php b/src/view/form/AphrontFormView.php
--- a/src/view/form/AphrontFormView.php
+++ b/src/view/form/AphrontFormView.php
@@ -121,7 +121,7 @@
* control to simplify rendering.
*
* @param AphrontFormControl $control Control to append.
- * @return this
+ * @return $this
*/
public function appendControl(AphrontFormControl $control) {
$this->controls[] = $control;
diff --git a/src/view/phui/PHUICrumbView.php b/src/view/phui/PHUICrumbView.php
--- a/src/view/phui/PHUICrumbView.php
+++ b/src/view/phui/PHUICrumbView.php
@@ -24,7 +24,7 @@
* be hidden.
*
* @param bool $always_visible True to make the crumb always visible.
- * @return this
+ * @return $this
*/
public function setAlwaysVisible($always_visible) {
$this->alwaysVisible = $always_visible;
diff --git a/src/view/phui/PHUICrumbsView.php b/src/view/phui/PHUICrumbsView.php
--- a/src/view/phui/PHUICrumbsView.php
+++ b/src/view/phui/PHUICrumbsView.php
@@ -17,7 +17,7 @@
*
* @param string $text Text of the crumb.
* @param string $href (optional) href for the crumb.
- * @return this
+ * @return $this
*/
public function addTextCrumb($text, $href = null) {
return $this->addCrumb(
diff --git a/support/startup/PhabricatorClientLimit.php b/support/startup/PhabricatorClientLimit.php
--- a/support/startup/PhabricatorClientLimit.php
+++ b/support/startup/PhabricatorClientLimit.php
@@ -184,7 +184,7 @@
*
* @param float $score The cost for this request; more points pushes them
* toward the limit faster.
- * @return this
+ * @return $this
*/
private function addScore($score) {
$is_apcu = (bool)function_exists('apcu_fetch');