diff --git a/src/aphront/AphrontRequest.php b/src/aphront/AphrontRequest.php --- a/src/aphront/AphrontRequest.php +++ b/src/aphront/AphrontRequest.php @@ -56,8 +56,8 @@ * Applications like Paste, Diffusion, and Harbormaster use "$12-14" in the * URI to allow users to link to particular lines. * - * @param string URI data key to pull line range information from. - * @param int|null Maximum length of the range. + * @param string $key URI data key to pull line range information from. + * @param int|null $limit Maximum length of the range. * @return null|pair Null, or beginning and end of the range. */ public function getURILineRange($key, $limit) { @@ -537,8 +537,8 @@ * * To set a temporary cookie, see @{method:setTemporaryCookie}. * - * @param string Cookie name. - * @param string Cookie value. + * @param string $name Cookie name. + * @param string $value Cookie value. * @return this * @task cookie */ @@ -553,8 +553,8 @@ * * To set a durable cookie, see @{method:setCookie}. * - * @param string Cookie name. - * @param string Cookie value. + * @param string $name Cookie name. + * @param string $value Cookie value. * @return this * @task cookie */ @@ -566,9 +566,9 @@ /** * Set a cookie with a given expiration policy. * - * @param string Cookie name. - * @param string Cookie value. - * @param int Epoch timestamp for cookie expiration. + * @param string $name Cookie name. + * @param string $value Cookie value. + * @param int $expire Epoch timestamp for cookie expiration. * @return this * @task cookie */ @@ -748,7 +748,7 @@ * into a list of key-value pairs suitable for submitting via HTTP request * (with arrays flattened). * - * @param dict Data to flatten. + * @param dict $data Data to flatten. * @return dict Flat data suitable for inclusion in an HTTP * request. */ @@ -778,9 +778,11 @@ * and looks up the appropriate value in `$_SERVER` (in this case, * `"HTTP_ACCEPT_ENCODING"`). * - * @param string Canonical header name, like `"Accept-Encoding"`. - * @param wild Default value to return if header is not present. - * @param array? Read this instead of `$_SERVER`. + * @param string $name Canonical header name, like + `"Accept-Encoding"`. + * @param wild? $default Default value to return if header is not + present. + * @param array? $data Read this instead of `$_SERVER`. * @return string|wild Header value if present, or `$default` if not. */ public static function getHTTPHeader($name, $default = null, $data = null) { @@ -844,7 +846,7 @@ * This is not a general-purpose proxying method; it is a specialized * method with niche applications and severe security implications. * - * @param string URI identifying the host we are proxying the request to. + * @param string URI $uri identifying the host we are proxying the request to. * @return HTTPSFuture New proxy future. * * @phutil-external-symbol class PhabricatorStartup diff --git a/src/aphront/configuration/AphrontApplicationConfiguration.php b/src/aphront/configuration/AphrontApplicationConfiguration.php --- a/src/aphront/configuration/AphrontApplicationConfiguration.php +++ b/src/aphront/configuration/AphrontApplicationConfiguration.php @@ -510,8 +510,8 @@ * Map a specific path to the corresponding controller. For a description * of routing, see @{method:buildController}. * - * @param list List of routing maps. - * @param string Path to route. + * @param list $maps List of routing maps. + * @param string $path Path to route. * @return pair Controller and dictionary of request * parameters. * @task routing @@ -562,7 +562,7 @@ /** * Tests if a response is of a valid type. * - * @param wild Supposedly valid response. + * @param wild $response Supposedly valid response. * @return bool True if the object is of a valid type. * @task response */ @@ -583,8 +583,9 @@ * Verifies that the return value from an @{class:AphrontController} is * of an allowed type. * - * @param AphrontController Controller which returned the response. - * @param wild Supposedly valid response. + * @param AphrontController $controller Controller which returned the + * response. + * @param wild $response Supposedly valid response. * @return void * @task response */ @@ -612,9 +613,9 @@ * Verifies that the return value from an * @{class:AphrontResponseProducerInterface} is of an allowed type. * - * @param AphrontResponseProducerInterface Object which produced + * @param AphrontResponseProducerInterface $producer Object which produced * this response. - * @param wild Supposedly valid response. + * @param wild $response Supposedly valid response. * @return void * @task response */ @@ -642,9 +643,9 @@ * Verifies that the return value from an * @{class:AphrontRequestExceptionHandler} is of an allowed type. * - * @param AphrontRequestExceptionHandler Object which produced this + * @param AphrontRequestExceptionHandler $handler Object which produced this * response. - * @param wild Supposedly valid response. + * @param wild $response Supposedly valid response. * @return void * @task response */ @@ -678,9 +679,9 @@ * If a controller returns a response producer, invoke it now and produce * the real response. * - * @param AphrontRequest Request being handled. - * @param AphrontResponse|AphrontResponseProducerInterface Response, or - * response producer. + * @param AphrontRequest $request Request being handled. + * @param AphrontResponse|AphrontResponseProducerInterface $response + * Response, or response producer. * @return AphrontResponse Response after any required production. * @task response */ @@ -738,7 +739,7 @@ * This method delegates exception handling to available subclasses of * @{class:AphrontRequestExceptionHandler}. * - * @param Throwable Exception which needs to be handled. + * @param Throwable $throwable Exception which needs to be handled. * @return wild Response or response producer, or null if no available * handler can produce a response. * @task exception 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 @@ -31,7 +31,7 @@ * example, a type might lookup usernames or project names. These types need * to use the current viewer to execute queries. * - * @param PhabricatorUser Current viewer. + * @param PhabricatorUser $viewer Current viewer. * @return this * @task read */ @@ -58,8 +58,8 @@ /** * Test if a value is present in a request. * - * @param AphrontRequest The incoming request. - * @param string The key to examine. + * @param AphrontRequest $request The incoming request. + * @param string $key The key to examine. * @return bool True if a readable value is present in the request. * @task read */ @@ -74,8 +74,8 @@ * If the value is not present, a default value is returned (usually `null`). * Use @{method:getExists} to test if a value is present. * - * @param AphrontRequest The incoming request. - * @param string The key to examine. + * @param AphrontRequest $request The incoming request. + * @param string $key The key to examine. * @return wild Value, or default if value is not present. * @task read */ @@ -165,9 +165,9 @@ * existence check that a simpler "list of strings" type has, and can just * call the simpler type to reuse its behavior. * - * @param AphrontHTTPParameterType The other type. - * @param AphrontRequest Incoming request. - * @param string Key to examine. + * @param AphrontHTTPParameterType $type The other type. + * @param AphrontRequest $request Incoming request. + * @param string $key Key to examine. * @return bool True if the parameter exists. * @task util */ @@ -189,9 +189,9 @@ * type. For example, a "list of users" type may start by running the same * basic parsing that a simpler "list of strings" type does. * - * @param AphrontHTTPParameterType The other type. - * @param AphrontRequest Incoming request. - * @param string Key to examine. + * @param AphrontHTTPParameterType $type The other type. + * @param AphrontRequest $request Incoming request. + * @param string $key Key to examine. * @return wild Parsed value. * @task util */ @@ -233,8 +233,8 @@ * To call another type's behavior in order to perform this check, use * @{method:getExistsWithType}. * - * @param AphrontRequest The incoming request. - * @param string The key to examine. + * @param AphrontRequest $request The incoming request. + * @param string $key The key to examine. * @return bool True if a readable value is present in the request. * @task impl */ @@ -253,8 +253,8 @@ * To call another type's behavior in order to parse a value, use * @{method:getValueWithType}. * - * @param AphrontRequest The incoming request. - * @param string The key to examine. + * @param AphrontRequest $request The incoming request. + * @param string $key The key to examine. * @return wild Parsed value. * @task impl */ diff --git a/src/aphront/response/AphrontFileResponse.php b/src/aphront/response/AphrontFileResponse.php --- a/src/aphront/response/AphrontFileResponse.php +++ b/src/aphront/response/AphrontFileResponse.php @@ -30,7 +30,7 @@ /** * Set a download filename * - * @param $download string + * @param string $download * @return self */ public function setDownload($download) { diff --git a/src/aphront/response/AphrontRedirectResponse.php b/src/aphront/response/AphrontRedirectResponse.php --- a/src/aphront/response/AphrontRedirectResponse.php +++ b/src/aphront/response/AphrontRedirectResponse.php @@ -117,8 +117,9 @@ * ambiguity. For example, Chrome interprets "Location: /\evil.com" to mean * "perform a protocol-relative redirect to evil.com". * - * @param string URI to redirect to. - * @param bool True if this URI identifies a remote resource. + * @param string $uri URI to redirect to. + * @param bool $is_external True if this URI identifies a remote + * resource. * @return string URI for use in a "Location:" header. */ public static function getURIForRedirect($uri, $is_external) { diff --git a/src/aphront/sink/AphrontHTTPSink.php b/src/aphront/sink/AphrontHTTPSink.php --- a/src/aphront/sink/AphrontHTTPSink.php +++ b/src/aphront/sink/AphrontHTTPSink.php @@ -28,7 +28,8 @@ /** * Write an HTTP status code to the output. * - * @param int Numeric HTTP status code. + * @param int $code Numeric HTTP status code. + * @param string? $message * @return void */ final public function writeHTTPStatus($code, $message = '') { @@ -44,7 +45,7 @@ /** * Write HTTP headers to the output. * - * @param list List of pairs. + * @param list $headers List of pairs. * @return void */ final public function writeHeaders(array $headers) { @@ -89,7 +90,7 @@ /** * Write HTTP body data to the output. * - * @param string Body data. + * @param string $data Body data. * @return void */ final public function writeData($data) { @@ -100,7 +101,7 @@ /** * Write an entire @{class:AphrontResponse} to the output. * - * @param AphrontResponse The response object to write. + * @param AphrontResponse $response The response object to write. * @return void */ final public function writeResponse(AphrontResponse $response) { diff --git a/src/aphront/site/AphrontRoutingMap.php b/src/aphront/site/AphrontRoutingMap.php --- a/src/aphront/site/AphrontRoutingMap.php +++ b/src/aphront/site/AphrontRoutingMap.php @@ -50,7 +50,7 @@ /** * Find the route matching a path, if one exists. * - * @param string Path to route. + * @param string $path Path to route. * @return AphrontRoutingResult|null Routing result, if path matches map. * @task routing */ @@ -84,9 +84,9 @@ /** * Test a sub-map to see if any routes match a path. * - * @param string Path to route. - * @param string Pattern from the map. - * @param string Value from the map. + * @param string $route Pattern from the map. + * @param string $value Value from the map. + * @param string $path Path to route. * @return dict|null Match details, if path matches sub-map. * @task routing */ 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 @@ -57,7 +57,7 @@ * the request, or abort the request (e.g., by throwing an exception) if a * valid token isn't present. * - * @param callable CSRF callback. + * @param $callback Callable CSRF callback. * @return this * @task manage */ diff --git a/src/applications/auth/constants/PhabricatorCommonPasswords.php b/src/applications/auth/constants/PhabricatorCommonPasswords.php --- a/src/applications/auth/constants/PhabricatorCommonPasswords.php +++ b/src/applications/auth/constants/PhabricatorCommonPasswords.php @@ -16,7 +16,7 @@ /** * Check if a password is extremely common. * - * @param string Password to test. + * @param string $password Password to test. * @return bool True if the password is pathologically weak. * * @task common diff --git a/src/applications/auth/constants/PhabricatorCookies.php b/src/applications/auth/constants/PhabricatorCookies.php --- a/src/applications/auth/constants/PhabricatorCookies.php +++ b/src/applications/auth/constants/PhabricatorCookies.php @@ -77,7 +77,7 @@ * Set the client ID cookie. This is a random cookie used like a CSRF value * during authentication workflows. * - * @param AphrontRequest Request to modify. + * @param AphrontRequest $request Request to modify. * @return void * @task clientid */ @@ -105,10 +105,10 @@ * written, to avoid writing over a real URI with a bunch of "humans.txt" * stuff. See T3793 for discussion. * - * @param AphrontRequest Request to write to. - * @param string URI to write. - * @param bool Write this cookie even if we have a fresh - * cookie already. + * @param AphrontRequest $request Request to write to. + * @param string $next_uri URI to write. + * @param bool? $force Write this cookie even if we have a + * fresh cookie already. * @return void * * @task next @@ -139,7 +139,7 @@ /** * Read the URI out of the Next URI cookie. * - * @param AphrontRequest Request to examine. + * @param AphrontRequest $request Request to examine. * @return string|null Next URI cookie's URI value. * * @task next @@ -155,7 +155,7 @@ /** * Parse a Next URI cookie into its components. * - * @param string Raw cookie value. + * @param string $cookie Raw cookie value. * @return list List of timestamp and URI. * * @task next diff --git a/src/applications/auth/controller/PhabricatorAuthController.php b/src/applications/auth/controller/PhabricatorAuthController.php --- a/src/applications/auth/controller/PhabricatorAuthController.php +++ b/src/applications/auth/controller/PhabricatorAuthController.php @@ -44,8 +44,9 @@ * the user's cookies are set. However, event listeners can intercept this * event and do something else if they prefer. * - * @param PhabricatorUser User to log the viewer in as. - * @param bool True to issue a full session immediately, bypassing MFA. + * @param PhabricatorUser $user User to log the viewer in as. + * @param bool? $force_full_session True to issue a full session immediately, + * bypassing MFA. * @return AphrontResponse Response which continues the login process. */ protected function loginUser( diff --git a/src/applications/auth/engine/PhabricatorAuthSessionEngine.php b/src/applications/auth/engine/PhabricatorAuthSessionEngine.php --- a/src/applications/auth/engine/PhabricatorAuthSessionEngine.php +++ b/src/applications/auth/engine/PhabricatorAuthSessionEngine.php @@ -75,7 +75,7 @@ * Get the session kind (e.g., anonymous, user, external account) from a * session token. Returns a `KIND_` constant. * - * @param string Session token. + * @param string $session_token Session token. * @return const Session kind constant. */ public static function getSessionKindFromToken($session_token) { @@ -109,8 +109,8 @@ * loading context. This prevents use of a Conduit sesssion as a Web * session, for example. * - * @param const The type of session to load. - * @param string The session token. + * @param const $session_type The type of session to load. + * @param string $session_token The session token. * @return PhabricatorUser|null * @task use */ @@ -250,11 +250,12 @@ * You can configure the maximum number of concurrent sessions for various * session types in the Phabricator configuration. * - * @param const Session type constant (see + * @param const $session_type Session type constant (see * @{class:PhabricatorAuthSession}). - * @param phid|null Identity to establish a session for, usually a user - * PHID. With `null`, generates an anonymous session. - * @param bool True to issue a partial session. + * @param phid|null $identity_phid Identity to establish a session for, + * usually a user PHID. With `null`, generates an + * anonymous session. + * @param bool $partial True to issue a partial session. * @return string Newly generated session key. */ public function establishSession($session_type, $identity_phid, $partial) { @@ -325,9 +326,9 @@ * This is used when users change passwords, linked accounts, or add * multifactor authentication. * - * @param PhabricatorUser User whose sessions should be terminated. - * @param string|null Optionally, one session to keep. Normally, the current - * login session. + * @param PhabricatorUser $user User whose sessions should be terminated. + * @param string|null? $except_session Optionally, one session to keep. + * Normally, the current login session. * * @return void */ @@ -388,9 +389,11 @@ * does not upgrade the user's session as a side effect. This method is * appropriate for one-time checks. * - * @param PhabricatorUser User whose session needs to be in high security. - * @param AphrontRequest Current request. - * @param string URI to return the user to if they cancel. + * @param PhabricatorUser $viewer User whose session needs to be in high + * security. + * @param AphrontRequest $request Current request. + * @param string $cancel_uri URI to return the user to if they + * cancel. * @return PhabricatorAuthHighSecurityToken Security token. * @task hisec */ @@ -420,12 +423,14 @@ * take multiple high security actions. To perform a one-time check instead, * use @{method:requireHighSecurityToken}. * - * @param PhabricatorUser User whose session needs to be in high security. - * @param AphrontRequest Current request. - * @param string URI to return the user to if they cancel. - * @param bool True to jump partial sessions directly into high - * security instead of just upgrading them to full - * sessions. + * @param PhabricatorUser $viewer User whose session needs to be in high + * security. + * @param AphrontRequest $request Current request. + * @param string $cancel_uri URI to return the user to if they + * cancel. + * @param bool? $jump_into_hisec True to jump partial sessions + * directly into high security instead of just + * upgrading them to full sessions. * @return PhabricatorAuthHighSecurityToken Security token. * @task hisec */ @@ -736,8 +741,8 @@ /** * Issue a high security token for a session, if authorized. * - * @param PhabricatorAuthSession Session to issue a token for. - * @param bool Force token issue. + * @param PhabricatorAuthSession $session Session to issue a token for. + * @param bool? $force Force token issue. * @return PhabricatorAuthHighSecurityToken|null Token, if authorized. * @task hisec */ @@ -756,8 +761,10 @@ /** * Render a form for providing relevant multi-factor credentials. * - * @param PhabricatorUser Viewing user. - * @param AphrontRequest Current request. + * @param array $factors + * @param array $validation_results + * @param PhabricatorUser $viewer Viewing user. + * @param AphrontRequest $request Current request. * @return AphrontFormView Renderable form. * @task hisec */ @@ -810,8 +817,9 @@ * * Kicks a session out of high security and logs the exit. * - * @param PhabricatorUser Acting user. - * @param PhabricatorAuthSession Session to return to normal security. + * @param PhabricatorUser $viewer Acting user. + * @param PhabricatorAuthSession $session Session to return to normal + * security. * @return void * @task hisec */ @@ -843,7 +851,7 @@ /** * Upgrade a partial session to a full session. * - * @param PhabricatorAuthSession Session to upgrade. + * @param PhabricatorUser $viewer Viewer whose session should upgrade. * @return void * @task partial */ @@ -885,8 +893,8 @@ /** * Upgrade a session to have all legalpad documents signed. * - * @param PhabricatorUser User whose session should upgrade. - * @param array LegalpadDocument objects + * @param PhabricatorUser $viewer User whose session should upgrade. + * @param array $docs LegalpadDocument objects * @return void * @task partial */ @@ -934,13 +942,14 @@ * These URIs are used for password recovery and to regain access to accounts * which users have been locked out of. * - * @param PhabricatorUser User to generate a URI for. - * @param PhabricatorUserEmail Optionally, email to verify when + * @param PhabricatorUser $user User to generate a URI for. + * @param PhabricatorUserEmail? $email Optionally, email to verify when * link is used. - * @param string Optional context string for the URI. This is purely cosmetic - * and used only to customize workflow and error messages. - * @param bool True to generate a URI which forces an immediate upgrade to - * a full session, bypassing MFA and other login checks. + * @param string? $type Optional context string for the URI. This is purely + * cosmetic and used only to customize workflow and error messages. + * @param bool? $force_full_session True to generate a URI which forces an + * immediate upgrade to a full session, bypassing MFA and other login + * checks. * @return string Login URI. * @task onetime */ @@ -984,10 +993,10 @@ /** * Load the temporary token associated with a given one-time login key. * - * @param PhabricatorUser User to load the token for. - * @param PhabricatorUserEmail Optionally, email to verify when + * @param PhabricatorUser $user User to load the token for. + * @param PhabricatorUserEmail? $email Optionally, email to verify when * link is used. - * @param string Key user is presenting as a valid one-time login key. + * @param string? $key Key user is presenting as a valid one-time login key. * @return PhabricatorAuthTemporaryToken|null Token, if one exists. * @task onetime */ @@ -1012,10 +1021,10 @@ /** * Hash a one-time login key for storage as a temporary token. * - * @param PhabricatorUser User this key is for. - * @param PhabricatorUserEmail Optionally, email to verify when + * @param PhabricatorUser $user User this key is for. + * @param PhabricatorUserEmail? $email Optionally, email to verify when * link is used. - * @param string The one time login key. + * @param string? $key The one time login key. * @return string Hash of the key. * task onetime */ diff --git a/src/applications/auth/password/PhabricatorAuthPasswordHashInterface.php b/src/applications/auth/password/PhabricatorAuthPasswordHashInterface.php --- a/src/applications/auth/password/PhabricatorAuthPasswordHashInterface.php +++ b/src/applications/auth/password/PhabricatorAuthPasswordHashInterface.php @@ -14,8 +14,8 @@ * as their password or picking other passwords which are trivially similar * to an account or object identifier. * - * @param PhabricatorUser The user selecting the password. - * @param PhabricatorAuthPasswordEngine The password engine updating a + * @param PhabricatorUser $viewer The user selecting the password. + * @param PhabricatorAuthPasswordEngine $engine The password engine updating a * password. * @return list Blocklist of nonsecret identifiers which the password * should not be similar to. diff --git a/src/applications/auth/provider/PhabricatorAuthProvider.php b/src/applications/auth/provider/PhabricatorAuthProvider.php --- a/src/applications/auth/provider/PhabricatorAuthProvider.php +++ b/src/applications/auth/provider/PhabricatorAuthProvider.php @@ -460,9 +460,9 @@ * - `uri`: URI the button should take the user to when clicked. * - `method`: Optional HTTP method the button should use, defaults to GET. * - * @param AphrontRequest HTTP request. - * @param string Request mode string. - * @param map Additional parameters, see above. + * @param AphrontRequest $request HTTP request. + * @param string $mode Request mode string. + * @param map? $attributes Additional parameters, see above. * @return wild Log in button. */ protected function renderStandardLoginButton( diff --git a/src/applications/base/PhabricatorApplication.php b/src/applications/base/PhabricatorApplication.php --- a/src/applications/base/PhabricatorApplication.php +++ b/src/applications/base/PhabricatorApplication.php @@ -141,7 +141,7 @@ * * Users who have not yet set preferences see a default list of applications. * - * @param PhabricatorUser User viewing the pinned application list. + * @param PhabricatorUser $viewer User viewing the pinned application list. * @return bool True if this application should be pinned by default. */ public function isPinnedByDefault(PhabricatorUser $viewer) { @@ -330,9 +330,9 @@ /** * Build items for the main menu. * - * @param PhabricatorUser The viewing user. - * @param AphrontController The current controller. May be null for special - * pages like 404, exception handlers, etc. + * @param PhabricatorUser $user The viewing user. + * @param AphrontController? $controller The current controller. May be null + * for special pages like 404, exception handlers, etc. * @return list List of menu items. * @task ui */ @@ -409,7 +409,7 @@ * To check if an application is installed //and// available to a particular * viewer, user @{method:isClassInstalledForViewer}. * - * @param string Application class name. + * @param string $class Application class name. * @return bool True if the class is installed. * @task meta */ @@ -425,8 +425,8 @@ * To check if an application is installed at all, use * @{method:isClassInstalled}. * - * @param string Application class name. - * @param PhabricatorUser Viewing user. + * @param string $class Application class name. + * @param PhabricatorUser $viewer Viewing user. * @return bool True if the class is installed for the viewer. * @task meta */ diff --git a/src/applications/cache/PhabricatorCaches.php b/src/applications/cache/PhabricatorCaches.php --- a/src/applications/cache/PhabricatorCaches.php +++ b/src/applications/cache/PhabricatorCaches.php @@ -416,7 +416,7 @@ * * Data can later be inflated with @{method:inflateData}. * - * @param string String to attempt to deflate. + * @param string $value String to attempt to deflate. * @return string|null Deflated string, or null if it was not deflated. * @task compress */ @@ -447,7 +447,7 @@ /** * Inflate data previously deflated by @{method:maybeDeflateData}. * - * @param string Deflated data, from @{method:maybeDeflateData}. + * @param string $value Deflated data, from @{method:maybeDeflateData}. * @return string Original, uncompressed data. * @task compress */ diff --git a/src/applications/celerity/CelerityResourceMap.php b/src/applications/celerity/CelerityResourceMap.php --- a/src/applications/celerity/CelerityResourceMap.php +++ b/src/applications/celerity/CelerityResourceMap.php @@ -160,7 +160,7 @@ /** * Get the epoch timestamp of the last modification time of a symbol. * - * @param string Resource symbol to lookup. + * @param string $name Resource symbol to lookup. * @return int Epoch timestamp of last resource modification. */ public function getModifiedTimeForName($name) { @@ -186,7 +186,7 @@ * Return the absolute URI for the resource associated with a symbol. This * method is fairly low-level and ignores packaging. * - * @param string Resource symbol to lookup. + * @param string $symbol Resource symbol to lookup. * @return string|null Resource URI, or null if the symbol is unknown. */ public function getURIForSymbol($symbol) { @@ -199,7 +199,7 @@ * Return the absolute URI for the resource associated with a resource name. * This method is fairly low-level and ignores packaging. * - * @param string Resource name to lookup. + * @param string $name Resource name to lookup. * @return string|null Resource URI, or null if the name is unknown. */ public function getURIForName($name) { @@ -217,7 +217,7 @@ * Return the absolute URI for a resource, identified by hash. * This method is fairly low-level and ignores packaging. * - * @param string Resource hash to lookup. + * @param string $hash Resource hash to lookup. * @return string|null Resource URI, or null if the hash is unknown. */ private function getURIForHash($hash) { @@ -231,7 +231,7 @@ /** * Return the resource symbols required by a named resource. * - * @param string Resource name to lookup. + * @param string $name Resource name to lookup. * @return list|null List of required symbols, or null if the name * is unknown. */ @@ -247,7 +247,7 @@ /** * Return the resource name for a given symbol. * - * @param string Resource symbol to lookup. + * @param string $symbol Resource symbol to lookup. * @return string|null Resource name, or null if the symbol is unknown. */ public function getResourceNameForSymbol($symbol) { diff --git a/src/applications/celerity/CelerityResourceMapGenerator.php b/src/applications/celerity/CelerityResourceMapGenerator.php --- a/src/applications/celerity/CelerityResourceMapGenerator.php +++ b/src/applications/celerity/CelerityResourceMapGenerator.php @@ -146,7 +146,8 @@ * Find binary resources (like PNG and SWF) and return information about * them. * - * @param CelerityPhysicalResources Resource map to find binary resources for. + * @param CelerityPhysicalResources $resources Resource map to find binary + * resources for. * @return map> Resource information map. */ private function rebuildBinaryResources( @@ -170,8 +171,10 @@ /** * Find text resources (like JS and CSS) and return information about them. * - * @param CelerityPhysicalResources Resource map to find text resources for. - * @param CelerityResourceTransformer Configured resource transformer. + * @param CelerityPhysicalResources $resources Resource map to find text + * resources for. + * @param CelerityResourceTransformer $xformer Configured resource + * transformer. * @return map> Resource information map. */ private function rebuildTextResources( @@ -211,8 +214,8 @@ * Parse the `@provides` and `@requires` symbols out of a text resource, like * JS or CSS. * - * @param string Resource name. - * @param string Resource data. + * @param string $name Resource name. + * @param string $data Resource data. * @return pair|null> The `@provides` symbol and * the list of `@requires` symbols. If the resource is not part of the * dependency graph, both are null. @@ -254,8 +257,8 @@ * Check for dependency cycles in the resource graph. Raises an exception if * a cycle is detected. * - * @param map> Map of `@provides` symbols to their - * `@requires` symbols. + * @param map> $nodes Map of `@provides` symbols to + * their `@requires` symbols. * @return void */ private function detectGraphCycles(array $nodes) { @@ -278,9 +281,9 @@ /** * Build package specifications for a given resource source. * - * @param CelerityPhysicalResources Resource source to rebuild. - * @param map Map of `@provides` to hashes. - * @param map Map of hashes to resource names. + * @param CelerityPhysicalResources $resources Resource source to rebuild. + * @param map $symbol_map Map of `@provides` to hashes. + * @param map $reverse_map Map of hashes to resource names. * @return map> Package information maps. */ private function rebuildPackages( diff --git a/src/applications/celerity/CelerityResourceTransformer.php b/src/applications/celerity/CelerityResourceTransformer.php --- a/src/applications/celerity/CelerityResourceTransformer.php +++ b/src/applications/celerity/CelerityResourceTransformer.php @@ -228,7 +228,8 @@ * small enough. Otherwise, this method will return `null` and we'll end up * using a normal URI instead. * - * @param string Resource name to attempt to generate a data URI for. + * @param string $resource_name Resource name to attempt to generate a data + * URI for. * @return string|null Data URI, or null if we declined to generate one. */ private function generateDataURI($resource_name) { diff --git a/src/applications/celerity/api.php b/src/applications/celerity/api.php --- a/src/applications/celerity/api.php +++ b/src/applications/celerity/api.php @@ -9,8 +9,9 @@ * * For more information, see @{article:Adding New CSS and JS}. * - * @param string Name of the celerity module to include. This is whatever you - * annotated as "@provides" in the file. + * @param string $symbol Name of the celerity module to include. This is + * whatever you annotated as "@provides" in the file. + * @param string? $source_name * @return void */ function require_celerity_resource($symbol, $source_name = 'phabricator') { @@ -40,7 +41,8 @@ /** * Get the versioned URI for a raw resource, like an image. * - * @param string Path to the raw image. + * @param string $resource Path to the raw image. + * @param string? $source * @return string Versioned path to the image, if one is available. */ function celerity_get_resource_uri($resource, $source = 'phabricator') { diff --git a/src/applications/celerity/controller/CelerityResourceController.php b/src/applications/celerity/controller/CelerityResourceController.php --- a/src/applications/celerity/controller/CelerityResourceController.php +++ b/src/applications/celerity/controller/CelerityResourceController.php @@ -194,7 +194,7 @@ * are cached, while other types of resources (which are large, and cheap * to process) are not. * - * @param string Resource type. + * @param string $type Resource type. * @return bool True to enable caching. */ private function isLocallyCacheableResourceType($type) { diff --git a/src/applications/celerity/management/CelerityManagementMapWorkflow.php b/src/applications/celerity/management/CelerityManagementMapWorkflow.php --- a/src/applications/celerity/management/CelerityManagementMapWorkflow.php +++ b/src/applications/celerity/management/CelerityManagementMapWorkflow.php @@ -32,7 +32,7 @@ /** * Rebuild the resource map for a resource source. * - * @param CelerityPhysicalResources Resource source to rebuild. + * @param $resources CelerityPhysicalResources Resource source to rebuild. * @return void */ private function rebuildResources(CelerityPhysicalResources $resources) { diff --git a/src/applications/conduit/controller/PhabricatorConduitAPIController.php b/src/applications/conduit/controller/PhabricatorConduitAPIController.php --- a/src/applications/conduit/controller/PhabricatorConduitAPIController.php +++ b/src/applications/conduit/controller/PhabricatorConduitAPIController.php @@ -159,8 +159,9 @@ /** * Authenticate the client making the request to a Phabricator user account. * - * @param ConduitAPIRequest Request being executed. - * @param dict Request metadata. + * @param ConduitAPIRequest $api_request Request being executed. + * @param dict $metadata Request metadata. + * @param wild $method * @return null|pair Null to indicate successful authentication, or * an error code and error message pair. */ 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 @@ -12,7 +12,7 @@ * will be used instead. This is useful to provide specific information about * an exception (e.g., which values were wrong in an invalid request). * - * @param string Detailed error description. + * @param string $error_description Detailed error description. * @return this */ final public function setErrorDescription($error_description) { diff --git a/src/applications/config/custom/PhabricatorCustomLogoConfigType.php b/src/applications/config/custom/PhabricatorCustomLogoConfigType.php --- a/src/applications/config/custom/PhabricatorCustomLogoConfigType.php +++ b/src/applications/config/custom/PhabricatorCustomLogoConfigType.php @@ -15,7 +15,7 @@ /** * Return the full URI of the Phorge logo - * @param PhabricatorUser Current viewer + * @param PhabricatorUser $viewer Current viewer * @return string Full URI of the Phorge logo */ public static function getLogoURI(PhabricatorUser $viewer) { 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 @@ -145,8 +145,8 @@ * we alter `memory_limit` during startup, so if the original value is not * provided it will look like it is always set to `-1`. * - * @param string PHP configuration option to provide a value for. - * @param string Explicit value to show in the UI. + * @param string $php_config PHP configuration option to provide a value for. + * @param string $value Explicit value to show in the UI. * @return this */ public function addPHPConfigOriginalValue($php_config, $value) { diff --git a/src/applications/config/json/PhabricatorConfigJSON.php b/src/applications/config/json/PhabricatorConfigJSON.php --- a/src/applications/config/json/PhabricatorConfigJSON.php +++ b/src/applications/config/json/PhabricatorConfigJSON.php @@ -4,7 +4,8 @@ /** * Properly format a JSON value. * - * @param wild Any value, but should be a raw value, not a string of JSON. + * @param wild $value Any value, but should be a raw value, not a string of + * JSON. * @return string */ public static function prettyPrintJSON($value) { diff --git a/src/applications/config/option/PhabricatorApplicationConfigOptions.php b/src/applications/config/option/PhabricatorApplicationConfigOptions.php --- a/src/applications/config/option/PhabricatorApplicationConfigOptions.php +++ b/src/applications/config/option/PhabricatorApplicationConfigOptions.php @@ -65,8 +65,8 @@ * or other context. For example, this is used to show workspace IDs when * configuring `asana.workspace-id`. * - * @param PhabricatorConfigOption Option being rendered. - * @param AphrontRequest Active request. + * @param PhabricatorConfigOption $option Option being rendered. + * @param AphrontRequest $request Active request. * @return wild Additional contextual description * information. */ diff --git a/src/applications/dashboard/engine/PhabricatorDashboardPanelRenderingEngine.php b/src/applications/dashboard/engine/PhabricatorDashboardPanelRenderingEngine.php --- a/src/applications/dashboard/engine/PhabricatorDashboardPanelRenderingEngine.php +++ b/src/applications/dashboard/engine/PhabricatorDashboardPanelRenderingEngine.php @@ -427,7 +427,7 @@ * panel. Generally, all of this stuff is ridiculous and we just want to * shut it down. * - * @param PhabricatorDashboardPanel Panel being rendered. + * @param PhabricatorDashboardPanel $panel Panel being rendered. * @return void */ private function detectRenderingCycle(PhabricatorDashboardPanel $panel) { diff --git a/src/applications/differential/constants/DifferentialReviewerStatus.php b/src/applications/differential/constants/DifferentialReviewerStatus.php --- a/src/applications/differential/constants/DifferentialReviewerStatus.php +++ b/src/applications/differential/constants/DifferentialReviewerStatus.php @@ -19,7 +19,7 @@ * will attempt to update you to both "commented" and "accepted". We want * "accepted" to win, because it's the stronger of the two. * - * @param const Reviewer status constant. + * @param const $constant Reviewer status constant. * @return int Relative strength (higher is stronger). */ public static function getStatusStrength($constant) { 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 @@ -240,12 +240,12 @@ * the left and right halves of the displayed diff to be correctly mapped to * storage changesets. * - * @param id The Differential Changeset ID that comments added to the right - * side of the visible diff should be attached to. - * @param bool If true, attach new comments to the right side of the 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. + * @param id $id The Differential Changeset ID that comments added to the + * right side of the visible diff should be attached to. + * @param bool $is_new If true, attach new comments to the right side of the + * 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 */ public function setRightSideCommentMapping($id, $is_new) { @@ -295,7 +295,8 @@ * By default, there is no render cache key and parsers do not use the cache. * This is appropriate for rarely-viewed changesets. * - * @param string Key for identifying this changeset in the render cache. + * @param string $key Key for identifying this changeset in the render + * cache. * @return this */ public function setRenderCacheKey($key) { @@ -1238,7 +1239,7 @@ * taking into consideration which halves of which changesets will actually * be shown. * - * @param PhabricatorInlineComment Comment to test for visibility. + * @param PhabricatorInlineComment $comment Comment to test for visibility. * @return bool True if the comment is visible on the rendered diff. */ private function isCommentVisibleOnRenderedDiff( @@ -1266,7 +1267,7 @@ * Note that the comment must appear somewhere on the rendered changeset, as * per isCommentVisibleOnRenderedDiff(). * - * @param PhabricatorInlineComment Comment to test for display + * @param PhabricatorInlineComment $comment Comment to test for display * location. * @return bool True for right, false for left. */ @@ -1298,8 +1299,8 @@ * list($start, $end, $mask) = $parsed; * $parser->render($start, $end, $mask); * - * @param string Range specification, indicating the range of the diff that - * should be rendered. + * @param string $spec Range specification, indicating the range of the diff + * that should be rendered. * @return tuple List of suitable for passing to * @{method:render}. */ diff --git a/src/applications/differential/parser/DifferentialLineAdjustmentMap.php b/src/applications/differential/parser/DifferentialLineAdjustmentMap.php --- a/src/applications/differential/parser/DifferentialLineAdjustmentMap.php +++ b/src/applications/differential/parser/DifferentialLineAdjustmentMap.php @@ -58,8 +58,8 @@ /** * Map a line across a change, or a series of changes. * - * @param int Line to map - * @param bool True to map it as the end of a range. + * @param int $line Line to map + * @param bool $is_end True to map it as the end of a range. * @return wild Spooky magic. */ public function mapLine($line, $is_end) { 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 @@ -45,7 +45,7 @@ /** * Find revisions affecting one or more items in a list of paths. * - * @param list List of file paths. + * @param list $paths List of file paths. * @return this * @task config */ @@ -59,7 +59,7 @@ * this function will clear anything set by previous calls to * @{method:withAuthors}. * - * @param array List of PHIDs of authors + * @param array $author_phids List of PHIDs of authors * @return this * @task config */ @@ -72,7 +72,7 @@ * Filter results to revisions which CC one of the listed people. Calling this * function will clear anything set by previous calls to @{method:withCCs}. * - * @param array List of PHIDs of subscribers. + * @param array $cc_phids List of PHIDs of subscribers. * @return this * @task config */ @@ -86,7 +86,7 @@ * reviewers. Calling this function will clear anything set by previous calls * to @{method:withReviewers}. * - * @param array List of PHIDs of reviewers + * @param array $reviewer_phids List of PHIDs of reviewers * @return this * @task config */ @@ -124,7 +124,7 @@ * Calling this function will clear anything set by previous calls to * @{method:withCommitHashes}. * - * @param array List of pairs * @return this @@ -149,7 +149,7 @@ /** * Filter results to revisions on given branches. * - * @param list List of branch names. + * @param list $branches List of branch names. * @return this * @task config */ @@ -162,7 +162,7 @@ /** * Filter results to only return revisions whose ids are in the given set. * - * @param array List of revision ids + * @param array $ids List of revision ids * @return this * @task config */ @@ -175,7 +175,7 @@ /** * Filter results to only return revisions whose PHIDs are in the given set. * - * @param array List of revision PHIDs + * @param array $phids List of revision PHIDs * @return this * @task config */ @@ -189,7 +189,7 @@ * Given a set of users, filter results to return only revisions they are * responsible for (i.e., they are either authors or reviewers). * - * @param array List of user PHIDs. + * @param array $responsible_phids List of user PHIDs. * @return this * @task config */ @@ -221,7 +221,7 @@ * Set whether or not the query should load the active diff for each * revision. * - * @param bool True to load and attach diffs. + * @param bool $need_active_diffs True to load and attach diffs. * @return this * @task config */ @@ -235,7 +235,7 @@ * Set whether or not the query should load the associated commit PHIDs for * each revision. * - * @param bool True to load and attach diffs. + * @param bool $need_commit_phids True to load and attach diffs. * @return this * @task config */ @@ -249,7 +249,7 @@ * Set whether or not the query should load associated diff IDs for each * revision. * - * @param bool True to load and attach diff IDs. + * @param bool $need_diff_ids True to load and attach diff IDs. * @return this * @task config */ @@ -263,7 +263,7 @@ * Set whether or not the query should load associated commit hashes for each * revision. * - * @param bool True to load and attach commit hashes. + * @param bool $need_hashes True to load and attach commit hashes. * @return this * @task config */ @@ -276,7 +276,7 @@ /** * Set whether or not the query should load associated reviewers. * - * @param bool True to load and attach reviewers. + * @param bool $need_reviewers True to load and attach reviewers. * @return this * @task config */ @@ -291,7 +291,7 @@ * reviewer. In particular, they have authority to act on behalf of projects * they are a member of. * - * @param bool True to load and attach authority. + * @param bool $need_reviewer_authority True to load and attach authority. * @return this * @task config */ diff --git a/src/applications/differential/render/DifferentialChangesetHTMLRenderer.php b/src/applications/differential/render/DifferentialChangesetHTMLRenderer.php --- a/src/applications/differential/render/DifferentialChangesetHTMLRenderer.php +++ b/src/applications/differential/render/DifferentialChangesetHTMLRenderer.php @@ -491,9 +491,10 @@ /** * Build links which users can click to show more context in a changeset. * - * @param int Beginning of the line range to build links for. - * @param int Length of the line range to build links for. - * @param int Total number of lines in the changeset. + * @param int $top Beginning of the line range to build links for. + * @param int $len Length of the line range to build links for. + * @param int $changeset_length Total number of lines in the changeset. + * @param bool? $is_blocks * @return markup Rendered links. */ protected function renderShowContextLinks( @@ -580,9 +581,9 @@ * * See @{method:renderShowContextLinks}. * - * @param bool Does this link show all context when clicked? - * @param string Range specification for lines to show. - * @param string Text of the link. + * @param bool $is_all Does this link show all context when clicked? + * @param string $range Range specification for lines to show. + * @param string $text Text of the link. * @return markup Rendered link. */ private function renderShowContextLink($is_all, $range, $text) { diff --git a/src/applications/differential/render/DifferentialChangesetRenderer.php b/src/applications/differential/render/DifferentialChangesetRenderer.php --- a/src/applications/differential/render/DifferentialChangesetRenderer.php +++ b/src/applications/differential/render/DifferentialChangesetRenderer.php @@ -442,8 +442,8 @@ * when a file is not changed. * - `"none"`: Don't show the link (e.g., text not available). * - * @param string Message explaining why the diff is hidden. - * @param string|null Force mode, see above. + * @param string $message Message explaining why the diff is hidden. + * @param string|null $force Force mode, see above. * @return string Shield markup. */ abstract public function renderShield($message, $force = 'default'); diff --git a/src/applications/differential/storage/DifferentialChangeset.php b/src/applications/differential/storage/DifferentialChangeset.php --- a/src/applications/differential/storage/DifferentialChangeset.php +++ b/src/applications/differential/storage/DifferentialChangeset.php @@ -213,7 +213,7 @@ * Test if this changeset and some other changeset put the affected file in * the same state. * - * @param DifferentialChangeset Changeset to compare against. + * @param DifferentialChangeset $other Changeset to compare against. * @return bool True if the two changesets have the same effect. */ public function hasSameEffectAs(DifferentialChangeset $other) { diff --git a/src/applications/diffusion/data/DiffusionBrowseResultSet.php b/src/applications/diffusion/data/DiffusionBrowseResultSet.php --- a/src/applications/diffusion/data/DiffusionBrowseResultSet.php +++ b/src/applications/diffusion/data/DiffusionBrowseResultSet.php @@ -123,7 +123,7 @@ * When a directory contains several README files, this function scores them * so the caller can select a preferred file. See @{method:getReadmePath}. * - * @param string Local README path, like "README.txt". + * @param string $path Local README path, like "README.txt". * @return int Priority score, with higher being more preferred. */ public static function getReadmePriority($path) { diff --git a/src/applications/diffusion/data/DiffusionGitBranch.php b/src/applications/diffusion/data/DiffusionGitBranch.php --- a/src/applications/diffusion/data/DiffusionGitBranch.php +++ b/src/applications/diffusion/data/DiffusionGitBranch.php @@ -19,8 +19,9 @@ * 'master' => '99a9c082f9a1b68c7264e26b9e552484a5ae5f25', * ); * - * @param string stdout of git branch command. - * @param string Filter branches to those on a specific remote. + * @param string $stdout stdout of git branch command. + * @param string? $only_this_remote Filter branches to those on a specific + * remote. * @return map Map of 'branch' or 'remote/branch' to hash at HEAD. */ public static function parseRemoteBranchOutput( diff --git a/src/applications/diffusion/protocol/DiffusionMercurialCommandEngine.php b/src/applications/diffusion/protocol/DiffusionMercurialCommandEngine.php --- a/src/applications/diffusion/protocol/DiffusionMercurialCommandEngine.php +++ b/src/applications/diffusion/protocol/DiffusionMercurialCommandEngine.php @@ -58,7 +58,7 @@ * Sanitize output of an `hg` command invoked with the `--debug` flag to make * it usable. * - * @param string Output from `hg --debug ...` + * @param string $stdout Output from `hg --debug ...` * @return string Usable output. */ public static function filterMercurialDebugOutput($stdout) { diff --git a/src/applications/diffusion/query/pathid/DiffusionPathIDQuery.php b/src/applications/diffusion/query/pathid/DiffusionPathIDQuery.php --- a/src/applications/diffusion/query/pathid/DiffusionPathIDQuery.php +++ b/src/applications/diffusion/query/pathid/DiffusionPathIDQuery.php @@ -42,7 +42,7 @@ /** * Convert a path to the canonical, absolute representation used by Diffusion. * - * @param string Some repository path. + * @param string $path Some repository path. * @return string Canonicalized Diffusion path. * @task pathutil */ @@ -62,7 +62,7 @@ * Return the canonical parent directory for a path. Note, returns "/" when * passed "/". * - * @param string Some repository path. + * @param string $path Some repository path. * @return string That path's canonical parent directory. * @task pathutil */ @@ -80,7 +80,7 @@ * Generate a list of parents for a repository path. The path itself is * included. * - * @param string Some repository path. + * @param string $path Some repository path. * @return list List of canonical paths between the path and the root. * @task pathutil */ 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 @@ -56,7 +56,7 @@ * - `commit` Optional, commit identifier. * - `line` Optional, line range. * - * @param map See documentation. + * @param map $data See documentation. * @return DiffusionRequest New request object. * @task new */ @@ -133,8 +133,9 @@ /** * Internal. Use @{method:newFromDictionary}, not this method. * - * @param string Repository identifier. - * @param PhabricatorUser Viewing user. + * @param string $identifier Repository identifier. + * @param PhabricatorUser $viewer Viewing user. + * @param bool? $need_edit * @return DiffusionRequest New request object. * @task new */ @@ -170,7 +171,7 @@ /** * Internal. Use @{method:newFromDictionary}, not this method. * - * @param PhabricatorRepository Repository object. + * @param PhabricatorRepository $repository Repository object. * @return DiffusionRequest New request object. * @task new */ @@ -201,7 +202,7 @@ /** * Internal. Use @{method:newFromDictionary}, not this method. * - * @param map Map of parsed data. + * @param map $data Map of parsed data. * @return void * @task new */ @@ -294,7 +295,7 @@ /** * Modify the request to move the symbolic commit elsewhere. * - * @param string New symbolic commit. + * @param string $symbol New symbolic commit. * @return this */ public function updateSymbolicCommit($symbol) { @@ -464,8 +465,8 @@ * * Parse the request URI into components. * - * @param string URI blob. - * @param bool True if this VCS supports branches. + * @param string $blob URI blob. + * @param bool $supports_branches True if this VCS supports branches. * @return map Parsed URI. * * @task uri @@ -535,7 +536,7 @@ /** * Check that the working copy of the repository is present and readable. * - * @param string Path to the working copy. + * @param string $path Path to the working copy. */ protected function validateWorkingCopy($path) { if (!is_readable(dirname($path))) { diff --git a/src/applications/diffusion/view/DiffusionReadmeView.php b/src/applications/diffusion/view/DiffusionReadmeView.php --- a/src/applications/diffusion/view/DiffusionReadmeView.php +++ b/src/applications/diffusion/view/DiffusionReadmeView.php @@ -26,7 +26,7 @@ /** * Get the markup language a README should be interpreted as. * - * @param string Local README path, like "README.txt". + * @param string $path Local README path, like "README.txt". * @return string Best markup interpreter (like "remarkup") for this file. */ private function getReadmeLanguage($path) { 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 @@ -93,7 +93,7 @@ * have been generated incorrectly by accident. In these cases, we can * restore the original data. * - * @param bool + * @param bool $ghosts * @return this */ public function withGhosts($ghosts) { @@ -437,9 +437,10 @@ * children. When recursing, also walk up the tree and collect children of * atoms they extend. * - * @param list List of symbols to collect child hashes of. - * @param bool True to collect children of extended atoms, - * as well. + * @param list $symbols List of symbols to collect child + * hashes of. + * @param bool $recurse_up True to collect children of + * extended atoms, as well. * @return map Hashes of atoms' children. */ private function getAllChildHashes(array $symbols, $recurse_up) { @@ -469,9 +470,12 @@ * Attach child atoms to existing atoms. In recursive mode, also attach child * atoms to atoms that these atoms extend. * - * @param list List of symbols to attach children to. - * @param map Map of symbols, keyed by node hash. - * @param bool True to attach children to extended atoms, as well. + * @param list $symbols List of symbols to attach children + * to. + * @param map $children Map of symbols, keyed by + * node hash. + * @param bool $recurse_up True to attach children to extended atoms, as + * well. * @return void */ private function attachAllChildren( 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 @@ -31,7 +31,8 @@ * JIRA issues which can have several linked revisions), it's generally * more useful not to assume context. * - * @param bool True to assume object context when rendering. + * @param bool $render_with_implied_context True to assume object context + * when rendering. * @return this * @task config */ diff --git a/src/applications/drydock/blueprint/DrydockBlueprintImplementation.php b/src/applications/drydock/blueprint/DrydockBlueprintImplementation.php --- a/src/applications/drydock/blueprint/DrydockBlueprintImplementation.php +++ b/src/applications/drydock/blueprint/DrydockBlueprintImplementation.php @@ -63,9 +63,10 @@ * This method should not acquire locks or expect anything to be locked. This * is a coarse compatibility check between a lease and a resource. * - * @param DrydockBlueprint Concrete blueprint to allocate for. - * @param DrydockResource Candidate resource to allocate the lease on. - * @param DrydockLease Pending lease that wants to allocate here. + * @param DrydockBlueprint $blueprint Concrete blueprint to allocate for. + * @param DrydockResource $resource Candidate resource to allocate the lease + * on. + * @param DrydockLease $lease Pending lease that wants to allocate here. * @return bool True if the resource and lease are compatible. * @task lease */ @@ -81,9 +82,9 @@ * * If acquisition fails, throw an exception. * - * @param DrydockBlueprint Blueprint which built the resource. - * @param DrydockResource Resource to acquire a lease on. - * @param DrydockLease Requested lease. + * @param DrydockBlueprint $blueprint Blueprint which built the resource. + * @param DrydockResource $resource Resource to acquire a lease on. + * @param DrydockLease $lease Requested lease. * @return void * @task lease */ @@ -111,9 +112,9 @@ * This callback is primarily useful for automatically releasing resources * once all leases are released. * - * @param DrydockBlueprint Blueprint which built the resource. - * @param DrydockResource Resource a lease was released on. - * @param DrydockLease Recently released lease. + * @param DrydockBlueprint $blueprint Blueprint which built the resource. + * @param DrydockResource $resource Resource a lease was released on. + * @param DrydockLease $lease Recently released lease. * @return void * @task lease */ @@ -128,9 +129,9 @@ * * If a lease creates temporary state while held, destroy it here. * - * @param DrydockBlueprint Blueprint which built the resource. - * @param DrydockResource Resource the lease is acquired on. - * @param DrydockLease The lease being destroyed. + * @param DrydockBlueprint $blueprint Blueprint which built the resource. + * @param DrydockResource $resource Resource the lease is acquired on. + * @param DrydockLease $lease The lease being destroyed. * @return void * @task lease */ @@ -156,11 +157,11 @@ * has a flexible pool of expensive resources and you want to pack leases * onto them as tightly as possible. * - * @param DrydockBlueprint The blueprint for an existing resource being - * acquired. - * @param DrydockResource The resource being acquired, which we may want to - * build a supplemental resource for. - * @param DrydockLease The current lease performing acquisition. + * @param DrydockBlueprint $blueprint The blueprint for an existing resource + * being acquired. + * @param DrydockResource $resource The resource being acquired, which we may + * want to build a supplemental resource for. + * @param DrydockLease $lease The current lease performing acquisition. * @return bool True to prefer allocating a supplemental resource. * * @task lease @@ -190,7 +191,7 @@ * really exists, only if some blueprint may conceivably exist which could * plausibly be able to build a suitable resource. * - * @param DrydockLease Requested lease. + * @param DrydockLease $lease Requested lease. * @return bool True if some concrete blueprint of this implementation's * type might ever be able to build a resource for the lease. * @task resource @@ -211,9 +212,9 @@ * if the blueprint as configured may, at some time, be able to build a * suitable resource. * - * @param DrydockBlueprint Blueprint which may be asked to allocate a - * resource. - * @param DrydockLease Requested lease. + * @param DrydockBlueprint $blueprint Blueprint which may be asked to + * allocate a resource. + * @param DrydockLease $lease Requested lease. * @return bool True if this blueprint can eventually build a suitable * resource for the lease, as currently configured. * @task resource @@ -240,9 +241,9 @@ * eaten up free capacity by the time it actually tries to build a resource. * This is normal and the allocator will recover from it. * - * @param DrydockBlueprint The blueprint which may be asked to allocate a - * resource. - * @param DrydockLease Requested lease. + * @param DrydockBlueprint $blueprint The blueprint which may be asked to + * allocate a resource. + * @param DrydockLease $lease Requested lease. * @return bool True if this blueprint appears likely to be able to allocate * a suitable resource. * @task resource @@ -260,8 +261,9 @@ * called. Blueprints are entirely responsible for any lock handling they * need to perform. * - * @param DrydockBlueprint The blueprint which should allocate a resource. - * @param DrydockLease Requested lease. + * @param DrydockBlueprint $blueprint The blueprint which should allocate a + * resource. + * @param DrydockLease $lease Requested lease. * @return DrydockResource Allocated resource. * @task resource */ @@ -287,8 +289,8 @@ * here. For example, you might shut down a virtual host or destroy a working * copy on disk. * - * @param DrydockBlueprint Blueprint which built the resource. - * @param DrydockResource Resource being destroyed. + * @param DrydockBlueprint $blueprint Blueprint which built the resource. + * @param DrydockResource $resource Resource being destroyed. * @return void * @task resource */ @@ -300,8 +302,8 @@ /** * Get a human readable name for a resource. * - * @param DrydockBlueprint Blueprint which built the resource. - * @param DrydockResource Resource to get the name of. + * @param DrydockBlueprint $blueprint Blueprint which built the resource. + * @param DrydockResource $resource Resource to get the name of. * @return string Human-readable resource name. * @task resource */ @@ -338,7 +340,7 @@ * build a resource which can satisfy the lease. They may not be able to * build that resource right now. * - * @param DrydockLease Requested lease. + * @param DrydockLease $lease Requested lease. * @return list List of qualifying blueprint * implementations. */ @@ -429,7 +431,7 @@ /** * Get the effective concurrent resource limit for this blueprint. * - * @param DrydockBlueprint Blueprint to get the limit for. + * @param DrydockBlueprint $blueprint Blueprint to get the limit for. * @return int|null Limit, or `null` for no limit. */ protected function getConcurrentResourceLimit(DrydockBlueprint $blueprint) { @@ -500,7 +502,7 @@ /** * Apply standard limits on resource allocation rate. * - * @param DrydockBlueprint The blueprint requesting an allocation. + * @param DrydockBlueprint $blueprint The blueprint requesting an allocation. * @return bool True if further allocations should be limited. */ protected function shouldLimitAllocatingPoolSize( diff --git a/src/applications/drydock/storage/DrydockAuthorization.php b/src/applications/drydock/storage/DrydockAuthorization.php --- a/src/applications/drydock/storage/DrydockAuthorization.php +++ b/src/applications/drydock/storage/DrydockAuthorization.php @@ -103,10 +103,10 @@ * Apply external authorization effects after a user changes the value of a * blueprint selector control an object. * - * @param PhabricatorUser User applying the change. - * @param phid Object PHID change is being applied to. - * @param list Old blueprint PHIDs. - * @param list New blueprint PHIDs. + * @param PhabricatorUser $viewer User applying the change. + * @param phid $object_phid Object PHID change is being applied to. + * @param list $old Old blueprint PHIDs. + * @param list $new New blueprint PHIDs. * @return void */ public static function applyAuthorizationChanges( diff --git a/src/applications/drydock/storage/DrydockSlotLock.php b/src/applications/drydock/storage/DrydockSlotLock.php --- a/src/applications/drydock/storage/DrydockSlotLock.php +++ b/src/applications/drydock/storage/DrydockSlotLock.php @@ -43,7 +43,7 @@ /** * Load all locks held by a particular owner. * - * @param phid Owner PHID. + * @param phid $owner_phid Owner PHID. * @return list All held locks. * @task info */ @@ -57,7 +57,7 @@ /** * Test if a lock is currently free. * - * @param string Lock key to test. + * @param string $lock Lock key to test. * @return bool True if the lock is currently free. * @task info */ @@ -69,7 +69,7 @@ /** * Test if a list of locks are all currently free. * - * @param list List of lock keys to test. + * @param list $locks List of lock keys to test. * @return bool True if all locks are currently free. * @task info */ @@ -82,7 +82,7 @@ /** * Load named locks. * - * @param list List of lock keys to load. + * @param list $locks List of lock keys to load. * @return list List of held locks. * @task info */ @@ -114,8 +114,8 @@ * This method either acquires all the locks or throws an exception (usually * because one or more locks are held). * - * @param phid Lock owner PHID. - * @param list List of locks to acquire. + * @param phid $owner_phid Lock owner PHID. + * @param list $locks List of locks to acquire. * @return void * @task locks */ @@ -158,7 +158,7 @@ /** * Release all locks held by an owner. * - * @param phid Lock owner PHID. + * @param phid $owner_phid Lock owner PHID. * @return void * @task locks */ diff --git a/src/applications/drydock/worker/DrydockLeaseUpdateWorker.php b/src/applications/drydock/worker/DrydockLeaseUpdateWorker.php --- a/src/applications/drydock/worker/DrydockLeaseUpdateWorker.php +++ b/src/applications/drydock/worker/DrydockLeaseUpdateWorker.php @@ -168,7 +168,7 @@ * Find or build a resource which can satisfy a given lease request, then * acquire the lease. * - * @param DrydockLease Requested lease. + * @param DrydockLease $lease Requested lease. * @return void * @task allocator */ @@ -573,7 +573,7 @@ * Get all the concrete @{class:DrydockBlueprint}s which can possibly * build a resource to satisfy a lease. * - * @param DrydockLease Requested lease. + * @param DrydockLease $lease Requested lease. * @return list List of qualifying blueprints. * @task allocator */ @@ -645,9 +645,9 @@ * Load a list of all resources which a given lease can possibly be * allocated against. * - * @param list Blueprints which may produce suitable - * resources. - * @param DrydockLease Requested lease. + * @param list $blueprints Blueprints which may produce + * suitable resources. + * @param DrydockLease $lease Requested lease. * @return list Resources which may be able to allocate * the lease. * @task allocator @@ -675,8 +675,8 @@ /** * Remove resources which can not be acquired by a given lease from a list. * - * @param list Candidate resources. - * @param DrydockLease Acquiring lease. + * @param list $resources Candidate resources. + * @param DrydockLease $lease Acquiring lease. * @return list Resources which the lease may be able to * acquire. * @task allocator @@ -703,9 +703,9 @@ * Remove blueprints which are too heavily allocated to build a resource for * a lease from a list of blueprints. * - * @param list List of blueprints. - * @return list List with blueprints that can not allocate - * a resource for the lease right now removed. + * @param list $blueprints List of blueprints. + * @return list $lease List with blueprints that can not + * allocate a resource for the lease right now removed. * @task allocator */ private function removeOverallocatedBlueprints( @@ -731,8 +731,8 @@ * Rank blueprints by suitability for building a new resource for a * particular lease. * - * @param list List of blueprints. - * @param DrydockLease Requested lease. + * @param list $blueprints List of blueprints. + * @param DrydockLease $lease Requested lease. * @return list Ranked list of blueprints. * @task allocator */ @@ -750,8 +750,8 @@ /** * Rank resources by suitability for allocating a particular lease. * - * @param list List of resources. - * @param DrydockLease Requested lease. + * @param list $resources List of resources. + * @param DrydockLease $lease Requested lease. * @return list Ranked list of resources. * @task allocator */ @@ -769,8 +769,9 @@ /** * Perform an actual resource allocation with a particular blueprint. * - * @param DrydockBlueprint The blueprint to allocate a resource from. - * @param DrydockLease Requested lease. + * @param DrydockBlueprint $blueprint The blueprint to allocate a resource + * from. + * @param DrydockLease $lease Requested lease. * @return DrydockResource Allocated resource. * @task allocator */ @@ -815,9 +816,10 @@ * Check that the resource a blueprint allocated is roughly the sort of * object we expect. * - * @param DrydockBlueprint Blueprint which built the resource. - * @param wild Thing which the blueprint claims is a valid resource. - * @param DrydockLease Lease the resource was allocated for. + * @param DrydockBlueprint $blueprint Blueprint which built the resource. + * @param wild $resource Thing which the blueprint claims is a valid + * resource. + * @param DrydockLease $lease Lease the resource was allocated for. * @return void * @task allocator */ @@ -899,8 +901,8 @@ /** * Perform an actual lease acquisition on a particular resource. * - * @param DrydockResource Resource to acquire a lease on. - * @param DrydockLease Lease to acquire. + * @param DrydockResource $resource Resource to acquire a lease on. + * @param DrydockLease $lease Lease to acquire. * @return void * @task acquire */ @@ -931,9 +933,9 @@ /** * Make sure that a lease was really acquired properly. * - * @param DrydockBlueprint Blueprint which created the resource. - * @param DrydockResource Resource which was acquired. - * @param DrydockLease The lease which was supposedly acquired. + * @param DrydockBlueprint $blueprint Blueprint which created the resource. + * @param DrydockResource $resource Resource which was acquired. + * @param DrydockLease $lease The lease which was supposedly acquired. * @return void * @task acquire */ diff --git a/src/applications/drydock/worker/DrydockResourceUpdateWorker.php b/src/applications/drydock/worker/DrydockResourceUpdateWorker.php --- a/src/applications/drydock/worker/DrydockResourceUpdateWorker.php +++ b/src/applications/drydock/worker/DrydockResourceUpdateWorker.php @@ -38,7 +38,7 @@ /** * Update a resource, handling exceptions thrown during the update. * - * @param DrydockReosource Resource to update. + * @param DrydockResource $resource Resource to update. * @return void * @task update */ @@ -58,7 +58,7 @@ /** * Update a resource. * - * @param DrydockResource Resource to update. + * @param DrydockResource $resource Resource to update. * @return void * @task update */ @@ -89,8 +89,8 @@ /** * Convert a temporary exception into a yield. * - * @param DrydockResource Resource to yield. - * @param Exception Temporary exception worker encountered. + * @param DrydockResource $resource Resource to yield. + * @param Exception $ex Temporary exception worker encountered. * @task update */ private function yieldResource(DrydockResource $resource, Exception $ex) { diff --git a/src/applications/feed/PhabricatorFeedStoryPublisher.php b/src/applications/feed/PhabricatorFeedStoryPublisher.php --- a/src/applications/feed/PhabricatorFeedStoryPublisher.php +++ b/src/applications/feed/PhabricatorFeedStoryPublisher.php @@ -214,7 +214,7 @@ /** * Remove PHIDs who should not receive notifications from a subscriber list. * - * @param list List of potential subscribers. + * @param list $phids List of potential subscribers. * @return list List of actual subscribers. */ private function filterSubscribedPHIDs(array $phids) { diff --git a/src/applications/feed/story/PhabricatorFeedStory.php b/src/applications/feed/story/PhabricatorFeedStory.php --- a/src/applications/feed/story/PhabricatorFeedStory.php +++ b/src/applications/feed/story/PhabricatorFeedStory.php @@ -32,8 +32,9 @@ * construct appropriate @{class:PhabricatorFeedStory} wrappers for each * data row. * - * @param list List of @{class:PhabricatorFeedStoryData} rows from the - * database. + * @param list $rows List of @{class:PhabricatorFeedStoryData} rows + * from the database. + * @param PhabricatorUser $viewer * @return list List of @{class:PhabricatorFeedStory} * objects. * @task load diff --git a/src/applications/files/PhabricatorImageTransformer.php b/src/applications/files/PhabricatorImageTransformer.php --- a/src/applications/files/PhabricatorImageTransformer.php +++ b/src/applications/files/PhabricatorImageTransformer.php @@ -20,8 +20,8 @@ * Phabricator can not encode images in the given format (based on available * extensions), but can save images in another format. * - * @param resource GD image resource. - * @param string? Optionally, preferred mime type. + * @param resource $data GD image resource. + * @param string? $preferred_mime Optionally, preferred mime type. * @return string Bytes of an image file. * @task save */ @@ -62,7 +62,7 @@ /** * Save an image in PNG format, returning the file data as a string. * - * @param resource GD image resource. + * @param resource $image GD image resource. * @return string|null PNG file as a string, or null on failure. * @task save */ @@ -90,7 +90,7 @@ /** * Save an image in GIF format, returning the file data as a string. * - * @param resource GD image resource. + * @param resource $image GD image resource. * @return string|null GIF file as a string, or null on failure. * @task save */ @@ -114,7 +114,7 @@ /** * Save an image in JPG format, returning the file data as a string. * - * @param resource GD image resource. + * @param resource $image GD image resource. * @return string|null JPG file as a string, or null on failure. * @task save */ diff --git a/src/applications/files/engine/PhabricatorChunkedFileStorageEngine.php b/src/applications/files/engine/PhabricatorChunkedFileStorageEngine.php --- a/src/applications/files/engine/PhabricatorChunkedFileStorageEngine.php +++ b/src/applications/files/engine/PhabricatorChunkedFileStorageEngine.php @@ -87,8 +87,8 @@ * Ideally, we'd like to be able to verify hashes, but this is complicated * and time consuming and gives us a fairly small benefit. * - * @param PhabricatorUser Viewing user. - * @param string Claimed file hash. + * @param PhabricatorUser $viewer Viewing user. + * @param string $hash Claimed file hash. * @return string Rehashed file hash. */ public static function getChunkedHash(PhabricatorUser $viewer, $hash) { diff --git a/src/applications/files/engine/PhabricatorFileStorageEngine.php b/src/applications/files/engine/PhabricatorFileStorageEngine.php --- a/src/applications/files/engine/PhabricatorFileStorageEngine.php +++ b/src/applications/files/engine/PhabricatorFileStorageEngine.php @@ -160,8 +160,8 @@ * throw an exception. If there are other satisfactory but less-preferred * storage engines available, they will be tried. * - * @param string The file data to write. - * @param array File metadata (name, author), if available. + * @param string $data The file data to write. + * @param array $params File metadata (name, author), if available. * @return string Unique string which identifies the stored file, max length * 255. * @task file @@ -172,8 +172,8 @@ /** * Read the contents of a file previously written by @{method:writeFile}. * - * @param string The handle returned from @{method:writeFile} when the - * file was written. + * @param string $handle The handle returned from @{method:writeFile} + * when the file was written. * @return string File contents. * @task file */ @@ -183,8 +183,8 @@ /** * Delete the data for a file previously written by @{method:writeFile}. * - * @param string The handle returned from @{method:writeFile} when the - * file was written. + * @param string $handle The handle returned from @{method:writeFile} + * when the file was written. * @return void * @task file */ @@ -200,7 +200,7 @@ * select the MySQL and Local Disk storage engines if they are configured * to allow a given file. * - * @param int File size in bytes. + * @param int $length File size in bytes. * @task load */ public static function loadStorageEngines($length) { diff --git a/src/applications/files/engine/PhabricatorLocalDiskFileStorageEngine.php b/src/applications/files/engine/PhabricatorLocalDiskFileStorageEngine.php --- a/src/applications/files/engine/PhabricatorLocalDiskFileStorageEngine.php +++ b/src/applications/files/engine/PhabricatorLocalDiskFileStorageEngine.php @@ -116,7 +116,7 @@ /** * Convert a handle into an absolute local disk path. * - * @param string File data handle. + * @param string $handle File data handle. * @return string Absolute path to the corresponding file. * @task internal */ diff --git a/src/applications/files/engine/PhabricatorMySQLFileStorageEngine.php b/src/applications/files/engine/PhabricatorMySQLFileStorageEngine.php --- a/src/applications/files/engine/PhabricatorMySQLFileStorageEngine.php +++ b/src/applications/files/engine/PhabricatorMySQLFileStorageEngine.php @@ -80,7 +80,7 @@ /** * Load the Lisk object that stores the file data for a handle. * - * @param string File data handle. + * @param string $handle File data handle. * @return PhabricatorFileStorageBlob Data DAO. * @task internal */ 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 @@ -96,7 +96,8 @@ * `PHID-FILE-aaaa` and all transformations of the file with PHID * `PHID-FILE-bbbb`. * - * @param list List of transform specifications, described above. + * @param list $specs List of transform specifications, described + * above. * @return this */ public function withTransforms(array $specs) { 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 @@ -807,8 +807,8 @@ /** * Return an iterable which emits file content bytes. * - * @param int Offset for the start of data. - * @param int Offset for the end of data. + * @param int? $begin Offset for the start of data. + * @param int? $end Offset for the end of data. * @return Iterable Iterable object which emits requested data. */ public function getFileDataIterator($begin = null, $end = null) { @@ -1214,8 +1214,9 @@ * Builtins are located in `resources/builtin/` and identified by their * name. * - * @param PhabricatorUser Viewing user. - * @param list List of builtin file specs. + * @param PhabricatorUser $user Viewing user. + * @param list $builtins List of builtin file + * specs. * @return dict Dictionary of named builtins. */ public static function loadBuiltins(PhabricatorUser $user, array $builtins) { @@ -1281,8 +1282,8 @@ /** * Convenience wrapper for @{method:loadBuiltins}. * - * @param PhabricatorUser Viewing user. - * @param string Single builtin name to load. + * @param PhabricatorUser $user Viewing user. + * @param string $name Single builtin name to load. * @return PhabricatorFile Corresponding builtin file. */ public static function loadBuiltin(PhabricatorUser $user, $name) { @@ -1473,7 +1474,7 @@ * Write the policy edge between this file and some object. * This method is successful even if the file is already attached. * - * @param phid Object PHID to attach to. + * @param phid $phid Object PHID to attach to. * @return this */ public function attachToObject($phid) { @@ -1487,8 +1488,8 @@ * NOTE: Please avoid to use this static method directly. * Instead, use PhabricatorFile#attachToObject(phid). * - * @param phid File PHID to attach from. - * @param phid Object PHID to attach to. + * @param phid $file_phid File PHID to attach from. + * @param phid $object_phid Object PHID to attach to. * @return void */ public static function attachFileToObject($file_phid, $object_phid) { @@ -1526,8 +1527,8 @@ * This method is called both when creating a file from fresh data, and * when creating a new file which reuses existing storage. * - * @param map Bag of parameters, see @{class:PhabricatorFile} - * for documentation. + * @param map $params Bag of parameters, see + * @{class:PhabricatorFile} for documentation. * @return this */ private function readPropertiesFromParameters(array $params) { diff --git a/src/applications/files/transform/PhabricatorFileImageTransform.php b/src/applications/files/transform/PhabricatorFileImageTransform.php --- a/src/applications/files/transform/PhabricatorFileImageTransform.php +++ b/src/applications/files/transform/PhabricatorFileImageTransform.php @@ -11,7 +11,7 @@ /** * Get an estimate of the transformed dimensions of a file. * - * @param PhabricatorFile File to transform. + * @param PhabricatorFile $file File to transform. * @return list|null Width and height, if available. */ public function getTransformedDimensions(PhabricatorFile $file) { @@ -132,7 +132,7 @@ /** * Create a new @{class:PhabricatorFile} from raw data. * - * @param string Raw file data. + * @param string $data Raw file data. */ protected function newFileFromData($data) { if ($this->file) { @@ -159,8 +159,8 @@ /** * Create a new image filled with transparent pixels. * - * @param int Desired image width. - * @param int Desired image height. + * @param int $w Desired image width. + * @param int $h Desired image height. * @return resource New image resource. */ protected function newEmptyImage($w, $h) { diff --git a/src/applications/harbormaster/constants/HarbormasterBuildStatus.php b/src/applications/harbormaster/constants/HarbormasterBuildStatus.php --- a/src/applications/harbormaster/constants/HarbormasterBuildStatus.php +++ b/src/applications/harbormaster/constants/HarbormasterBuildStatus.php @@ -96,7 +96,7 @@ /** * Get a human readable name for a build status constant. * - * @param const Build status constant. + * @param const $status Build status constant. * @return string Human-readable name. */ public static function getBuildStatusName($status) { diff --git a/src/applications/harbormaster/engine/HarbormasterBuildEngine.php b/src/applications/harbormaster/engine/HarbormasterBuildEngine.php --- a/src/applications/harbormaster/engine/HarbormasterBuildEngine.php +++ b/src/applications/harbormaster/engine/HarbormasterBuildEngine.php @@ -322,8 +322,9 @@ * particularly relevant when a build uses multiple hosts since it returns * hosts to the pool more quickly. * - * @param list Targets in the build. - * @param list List of running and waiting steps. + * @param list $targets Targets in the build. + * @param list $steps List of running and waiting + * steps. * @return void */ private function releaseUnusedArtifacts(array $targets, array $steps) { @@ -372,7 +373,7 @@ * Process messages which were sent to these targets, kicking applicable * targets out of "Waiting" and into either "Passed" or "Failed". * - * @param list List of targets to process. + * @param list $targets List of targets to process. * @return void */ private function updateWaitingTargets(array $targets) { @@ -436,7 +437,7 @@ * the overall state of the associated buildable. Compute the new aggregate * state and save it on the buildable. * - * @param HarbormasterBuild The buildable to update. + * @param HarbormasterBuild $buildable The buildable to update. * @return void */ public function updateBuildable(HarbormasterBuildable $buildable) { diff --git a/src/applications/harbormaster/engine/HarbormasterTargetEngine.php b/src/applications/harbormaster/engine/HarbormasterTargetEngine.php --- a/src/applications/harbormaster/engine/HarbormasterTargetEngine.php +++ b/src/applications/harbormaster/engine/HarbormasterTargetEngine.php @@ -56,7 +56,7 @@ * * This method creates the steps if they do not yet exist. * - * @param list Autotarget keys, like `"core.arc.lint"`. + * @param list $autotargets Autotarget keys, like `"core.arc.lint"`. * @return map Map of keys to step objects. */ private function generateBuildStepMap(array $autotargets) { @@ -127,7 +127,7 @@ * Get all of the @{class:HarbormasterBuildStepImplementation} objects for * a list of autotarget keys. * - * @param list Autotarget keys, like `"core.arc.lint"`. + * @param list $autotargets Autotarget keys, like `"core.arc.lint"`. * @return map Map of keys to implementations. */ private function getAutosteps(array $autotargets) { @@ -154,8 +154,8 @@ * * If some targets or builds do not exist, they are created. * - * @param HarbormasterBuildable A buildable. - * @param map Map of keys to steps. + * @param HarbormasterBuildable $buildable A buildable. + * @param map $step_map Map of keys to steps. * @return map Map of keys to targets. */ private function generateBuildTargetMap( diff --git a/src/applications/harbormaster/step/HarbormasterBuildStepImplementation.php b/src/applications/harbormaster/step/HarbormasterBuildStepImplementation.php --- a/src/applications/harbormaster/step/HarbormasterBuildStepImplementation.php +++ b/src/applications/harbormaster/step/HarbormasterBuildStepImplementation.php @@ -187,9 +187,11 @@ * * ls 'dir with spaces' * - * @param string Name of a `vxsprintf` function, like @{function:vcsprintf}. - * @param string User-provided pattern string containing `${variables}`. - * @param dict List of available replacement variables. + * @param string $function Name of a `vxsprintf` function, like + * @{function:vcsprintf}. + * @param string $pattern User-provided pattern string containing + * `${variables}`. + * @param dict $variables List of available replacement variables. * @return string String with variables replaced safely into it. */ protected function mergeVariables($function, $pattern, array $variables) { diff --git a/src/applications/harbormaster/storage/HarbormasterBuildable.php b/src/applications/harbormaster/storage/HarbormasterBuildable.php --- a/src/applications/harbormaster/storage/HarbormasterBuildable.php +++ b/src/applications/harbormaster/storage/HarbormasterBuildable.php @@ -60,9 +60,9 @@ /** * Start builds for a given buildable. * - * @param phid PHID of the object to build. - * @param phid Container PHID for the buildable. - * @param list List of builds to perform. + * @param phid $phid PHID of the object to build. + * @param phid $container_phid Container PHID for the buildable. + * @param list $requests List of builds to perform. * @return void */ public static function applyBuildPlans( 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 @@ -171,7 +171,8 @@ * These transactions are set by @{class:PhabricatorApplicationEditor} * automatically, before it invokes Herald. * - * @param list List of transactions. + * @param list $xactions List of + * transactions. * @return this */ final public function setAppliedTransactions(array $xactions) { diff --git a/src/applications/maniphest/constants/ManiphestTaskPriority.php b/src/applications/maniphest/constants/ManiphestTaskPriority.php --- a/src/applications/maniphest/constants/ManiphestTaskPriority.php +++ b/src/applications/maniphest/constants/ManiphestTaskPriority.php @@ -137,7 +137,7 @@ /** * Retrieve the full name of the priority level provided. * - * @param int A priority level. + * @param int $priority A priority level. * @return string The priority name if the level is a valid one. */ public static function getTaskPriorityName($priority) { @@ -147,7 +147,7 @@ /** * Retrieve the color of the priority level given * - * @param int A priority level. + * @param int $priority A priority level. * @return string The color of the priority if the level is valid, * or black if it is not. */ diff --git a/src/applications/meta/panel/PhabricatorApplicationConfigurationPanel.php b/src/applications/meta/panel/PhabricatorApplicationConfigurationPanel.php --- a/src/applications/meta/panel/PhabricatorApplicationConfigurationPanel.php +++ b/src/applications/meta/panel/PhabricatorApplicationConfigurationPanel.php @@ -27,7 +27,7 @@ /** * Get the URI for this application configuration panel. * - * @param string? Optional path to append. + * @param string? $path Optional path to append. * @return string Relative URI for the panel. */ public function getPanelURI($path = '') { diff --git a/src/applications/metamta/parser/PhabricatorMetaMTAEmailBodyParser.php b/src/applications/metamta/parser/PhabricatorMetaMTAEmailBodyParser.php --- a/src/applications/metamta/parser/PhabricatorMetaMTAEmailBodyParser.php +++ b/src/applications/metamta/parser/PhabricatorMetaMTAEmailBodyParser.php @@ -26,7 +26,7 @@ * ), * ) * - * @param string Raw mail text body. + * @param string $body Raw mail text body. * @return dict Parsed body. */ public function parseBody($body) { diff --git a/src/applications/metamta/receiver/PhabricatorObjectMailReceiver.php b/src/applications/metamta/receiver/PhabricatorObjectMailReceiver.php --- a/src/applications/metamta/receiver/PhabricatorObjectMailReceiver.php +++ b/src/applications/metamta/receiver/PhabricatorObjectMailReceiver.php @@ -19,9 +19,9 @@ * Load the object receiving mail, based on an identifying pattern. Normally * this pattern is some sort of object ID. * - * @param string A string matched by @{method:getObjectPattern} - * fragment. - * @param PhabricatorUser The viewing user. + * @param string $pattern A string matched by + * @{method:getObjectPattern} fragment. + * @param PhabricatorUser $viewer The viewing user. * @return object|null The object to receive mail, or null if no such * object exists. */ diff --git a/src/applications/metamta/replyhandler/PhabricatorMailReplyHandler.php b/src/applications/metamta/replyhandler/PhabricatorMailReplyHandler.php --- a/src/applications/metamta/replyhandler/PhabricatorMailReplyHandler.php +++ b/src/applications/metamta/replyhandler/PhabricatorMailReplyHandler.php @@ -199,8 +199,8 @@ * Each target should be sent a separate email, and contains the information * required to generate it with appropriate permissions and configuration. * - * @param list List of "To" PHIDs. - * @param list List of "CC" PHIDs. + * @param list $raw_to List of "To" PHIDs. + * @param list $raw_cc List of "CC" PHIDs. * @return list List of targets. */ final public function getMailTargets(array $raw_to, array $raw_cc) { @@ -274,8 +274,8 @@ * This takes any compound recipients (like projects) and looks up all their * members. * - * @param list List of To PHIDs. - * @param list List of CC PHIDs. + * @param list $to List of To PHIDs. + * @param list $cc List of CC PHIDs. * @return pair, list> Expanded PHID lists. */ private function expandRecipientPHIDs(array $to, array $cc) { @@ -332,8 +332,8 @@ * * Invalid recipients are dropped from the results. * - * @param list List of To PHIDs. - * @param list List of CC PHIDs. + * @param list $to List of To PHIDs. + * @param list $cc List of CC PHIDs. * @return pair Maps from PHIDs to users. */ private function loadRecipientUsers(array $to, array $cc) { @@ -370,8 +370,8 @@ /** * Remove recipients who do not have permission to view the mail receiver. * - * @param map Map of "To" users. - * @param map Map of "CC" users. + * @param map $to Map of "To" users. + * @param map $cc Map of "CC" users. * @return pair Filtered user maps. */ private function filterRecipientUsers(array $to, array $cc) { 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 @@ -88,7 +88,7 @@ * These tags are used to allow users to opt out of receiving certain types * of mail, like updates when a task's projects change. * - * @param list + * @param list $tags * @return this */ public function setMailTags(array $tags) { @@ -109,7 +109,7 @@ * needs to be set whenever an action is triggered by an email message. See * T251 for more details. * - * @param string The "Message-ID" of the email which precedes this one. + * @param string $id The "Message-ID" of the email which precedes this one. * @return this */ public function setParentMessageID($id) { @@ -419,7 +419,7 @@ * This is primarily intended to let users who don't want any email still * receive things like password resets. * - * @param bool True to force delivery despite user preferences. + * @param bool $force True to force delivery despite user preferences. * @return this */ public function setForceDelivery($force) { @@ -437,7 +437,7 @@ * "Precedence: bulk" or similar, but is implementation and configuration * dependent. * - * @param bool True if the mail is automated bulk mail. + * @param bool $is_bulk True if the mail is automated bulk mail. * @return this */ public function setIsBulk($is_bulk) { @@ -454,9 +454,10 @@ * set appropriate headers (Message-ID, In-Reply-To, References and * Thread-Index) based on the capabilities of the underlying mailer. * - * @param string Unique identifier, appropriate for use in a Message-ID, - * In-Reply-To or References headers. - * @param bool If true, indicates this is the first message in the thread. + * @param string $thread_id Unique identifier, appropriate for use in a + * Message-ID, In-Reply-To or References headers. + * @param bool? $is_first_message If true, indicates this is the first + * message in the thread. * @return this */ public function setThreadID($thread_id, $is_first_message = false) { @@ -858,8 +859,8 @@ * For example, this will expand project PHIDs into a list of the project's * members. * - * @param list List of recipient PHIDs, possibly including aggregate - * recipients. + * @param list $phids List of recipient PHIDs, possibly including + * aggregate recipients. * @return list Deaggregated list of mailable recipients. */ public function expandRecipients(array $phids) { diff --git a/src/applications/metamta/util/PhabricatorMailUtil.php b/src/applications/metamta/util/PhabricatorMailUtil.php --- a/src/applications/metamta/util/PhabricatorMailUtil.php +++ b/src/applications/metamta/util/PhabricatorMailUtil.php @@ -10,7 +10,7 @@ * which can make forwarding rules easier to write. This method strips the * prefix if it is present, and normalizes casing and whitespace. * - * @param PhutilEmailAddress Email address. + * @param PhutilEmailAddress $address Email address. * @return PhutilEmailAddress Normalized address. */ public static function normalizeAddress(PhutilEmailAddress $address) { @@ -47,8 +47,8 @@ * * "Abraham" # With configured prefix. * - * @param PhutilEmailAddress Email address. - * @param PhutilEmailAddress Another email address. + * @param PhutilEmailAddress $u Email address. + * @param PhutilEmailAddress $v Another email address. * @return bool True if addresses are effectively the same address. */ public static function matchAddresses( 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 @@ -40,7 +40,7 @@ /** * Add a raw block of text to the email. This will be rendered as-is. * - * @param string Block of text. + * @param string $text Block of text. * @return this * @task compose */ @@ -100,8 +100,8 @@ * HEADER * Text is indented. * - * @param string Header text. - * @param string Section text. + * @param string $header Header text. + * @param string $section Section text. * @return this * @task compose */ @@ -155,7 +155,7 @@ /** * Add an attachment. * - * @param PhabricatorMailAttachment Attachment. + * @param PhabricatorMailAttachment $attachment Attachment. * @return this * @task compose */ @@ -198,7 +198,7 @@ /** * Indent a block of text for rendering under a section heading. * - * @param string Text to indent. + * @param string $text Text to indent. * @return string Indented text. * @task render */ 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 @@ -31,8 +31,8 @@ * only unread notifications, while `false` means to return only //read// * notifications. The default is `null`, which returns both. * - * @param mixed True or false to filter results by read status. Null to remove - * the filter. + * @param mixed $unread True or false to filter results by read status. Null + * to remove the filter. * @return this * @task config */ diff --git a/src/applications/passphrase/view/PassphraseCredentialControl.php b/src/applications/passphrase/view/PassphraseCredentialControl.php --- a/src/applications/passphrase/view/PassphraseCredentialControl.php +++ b/src/applications/passphrase/view/PassphraseCredentialControl.php @@ -157,9 +157,9 @@ * - If you do change the credential, the new credential must be one you * can use. * - * @param PhabricatorUser The acting user. - * @param list List of credential altering - * transactions. + * @param PhabricatorUser $actor The acting user. + * @param list $xactions List of + * credential altering transactions. * @return bool True if the transactions are valid. */ public static function validateTransactions( 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 @@ -429,8 +429,8 @@ /** * Test if a given setting is set to a particular value. * - * @param const Setting key. - * @param wild Value to compare. + * @param const $key Setting key. + * @param wild $value Value to compare. * @return bool True if the setting has the specified value. * @task settings */ @@ -477,7 +477,7 @@ * * This is primarily useful for unit tests. * - * @param string New timezone identifier. + * @param string $identifier New timezone identifier. * @return this * @task settings */ @@ -770,8 +770,8 @@ /** * Write to the availability cache. * - * @param wild Availability cache data. - * @param int|null Cache TTL. + * @param wild $availability Availability cache data. + * @param int|null $ttl Cache TTL. * @return this * @task availability */ @@ -916,7 +916,7 @@ * Get a @{class:PhabricatorHandleList} which benefits from this viewer's * internal handle pool. * - * @param list List of PHIDs to load. + * @param list $phids List of PHIDs to load. * @return PhabricatorHandleList Handle list object. * @task handle */ @@ -935,7 +935,7 @@ * * This benefits from the viewer's internal handle pool. * - * @param phid PHID to render a handle for. + * @param phid $phid PHID to render a handle for. * @return PHUIHandleView View of the handle. * @task handle */ @@ -949,7 +949,7 @@ * * This benefits from the viewer's internal handle pool. * - * @param list List of PHIDs to render. + * @param list $phids List of PHIDs to render. * @return PHUIHandleListView View of the handles. * @task handle */ 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 @@ -180,7 +180,7 @@ /** * Send a verification email from $user to this address. * - * @param PhabricatorUser The user sending the verification. + * @param PhabricatorUser $user The user sending the verification. * @return this * @task email */ @@ -229,8 +229,8 @@ * Send a notification email from $user to this address, informing the * recipient that this is no longer their account's primary address. * - * @param PhabricatorUser The user sending the notification. - * @param PhabricatorUserEmail New primary email address. + * @param PhabricatorUser $user The user sending the notification. + * @param PhabricatorUserEmail $new New primary email address. * @task email */ public function sendOldPrimaryEmail( @@ -269,7 +269,7 @@ * Send a notification email from $user to this address, informing the * recipient that this is now their account's new primary email address. * - * @param PhabricatorUser The user sending the verification. + * @param PhabricatorUser $user The user sending the verification. * @return this * @task email */ 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 @@ -275,7 +275,7 @@ * Set whether or not the underlying object is complete. See * @{method:isComplete} for an explanation of what it means to be complete. * - * @param bool True if the handle represents a complete object. + * @param bool $complete True if the handle represents a complete object. * @return this */ public function setComplete($complete) { diff --git a/src/applications/phid/query/PhabricatorObjectQuery.php b/src/applications/phid/query/PhabricatorObjectQuery.php --- a/src/applications/phid/query/PhabricatorObjectQuery.php +++ b/src/applications/phid/query/PhabricatorObjectQuery.php @@ -196,8 +196,8 @@ * viewer. This method is generally used to validate that PHIDs affected by * a transaction are valid. * - * @param PhabricatorUser Viewer. - * @param list List of ostensibly valid PHIDs. + * @param PhabricatorUser $viewer Viewer. + * @param list $phids List of ostensibly valid PHIDs. * @return list List of invalid or restricted PHIDs. */ public static function loadInvalidPHIDsForViewer( diff --git a/src/applications/phid/type/PhabricatorPHIDType.php b/src/applications/phid/type/PhabricatorPHIDType.php --- a/src/applications/phid/type/PhabricatorPHIDType.php +++ b/src/applications/phid/type/PhabricatorPHIDType.php @@ -52,8 +52,8 @@ * can provide a dummy implementation for this method and overload * @{method:loadObjects} instead. * - * @param PhabricatorObjectQuery Query being executed. - * @param list PHIDs to load. + * @param PhabricatorObjectQuery $query Query being executed. + * @param list $phids PHIDs to load. * @return PhabricatorPolicyAwareQuery Query object which loads the * specified PHIDs when executed. */ @@ -67,8 +67,8 @@ * necessary to implement @{method:buildQueryForObjects} to get object * loading to work. * - * @param PhabricatorObjectQuery Query being executed. - * @param list PHIDs to load. + * @param PhabricatorObjectQuery $query Query being executed. + * @param list $phids PHIDs to load. * @return list Corresponding objects. */ public function loadObjects( @@ -113,10 +113,11 @@ * each handle at a minimum. See @{class:PhabricatorObjectHandle} for other * handle properties. * - * @param PhabricatorHandleQuery Issuing query object. - * @param list Handles to populate with data. - * @param list Objects for these PHIDs loaded by - * @{method:buildQueryForObjects()}. + * @param PhabricatorHandleQuery $query Issuing query object. + * @param list $handles Handles to populate with + * data. + * @param list $objects Objects for these PHIDs + * loaded by @{method:buildQueryForObjects()}. * @return void */ abstract public function loadHandles( @@ -165,7 +166,7 @@ /** * Get all PHID types of applications installed for a given viewer. * - * @param PhabricatorUser Viewing user. + * @param PhabricatorUser $viewer Viewing user. * @return dict Map of constants to installed * types. */ @@ -209,7 +210,7 @@ /** * Get all PHID types of an application. * - * @param string Class name of an application + * @param string $application Class name of an application * @return dict Map of constants of application */ public static function getAllTypesForApplication( diff --git a/src/applications/phid/utils.php b/src/applications/phid/utils.php --- a/src/applications/phid/utils.php +++ b/src/applications/phid/utils.php @@ -4,7 +4,7 @@ * Look up the type of a PHID. Returns * PhabricatorPHIDConstants::PHID_TYPE_UNKNOWN if it fails to look up the type * - * @param phid Anything. + * @param phid $phid Anything. * @return string A value from PhabricatorPHIDConstants (ideally) */ function phid_get_type($phid) { @@ -18,7 +18,7 @@ /** * Group a list of phids by type. * - * @param phids array of phids + * @param $phids Array of phids * @return map of phid type => list of phids */ function phid_group_by_type($phids) { diff --git a/src/applications/phrequent/storage/PhrequentTimeBlock.php b/src/applications/phrequent/storage/PhrequentTimeBlock.php --- a/src/applications/phrequent/storage/PhrequentTimeBlock.php +++ b/src/applications/phrequent/storage/PhrequentTimeBlock.php @@ -254,7 +254,8 @@ * This is used to avoid double-counting time on objects which had timers * started multiple times. * - * @param list> List of possibly overlapping time ranges. + * @param list> $ranges List of possibly overlapping time + * ranges. * @return list> Nonoverlapping time ranges. */ public static function mergeTimeRanges(array $ranges) { diff --git a/src/applications/policy/filter/PhabricatorPolicyFilter.php b/src/applications/policy/filter/PhabricatorPolicyFilter.php --- a/src/applications/policy/filter/PhabricatorPolicyFilter.php +++ b/src/applications/policy/filter/PhabricatorPolicyFilter.php @@ -52,11 +52,13 @@ * ...will throw a @{class:PhabricatorPolicyException} if the new policy would * remove the user's ability to edit the object. * - * @param PhabricatorUser The viewer to perform a policy check for. - * @param PhabricatorPolicyInterface The object to perform a policy check on. - * @param string Capability to test. - * @param string Perform the test as though the object has this - * policy instead of the policy it actually has. + * @param PhabricatorUser $viewer The viewer to perform a policy check for. + * @param PhabricatorPolicyInterface $object The object to perform a policy + * check on. + * @param string $capability Capability to test. + * @param string $forced_policy Perform the test as though the + * object has this policy instead of the policy it + * actually has. * @return void */ public static function requireCapabilityWithForcedPolicy( diff --git a/src/applications/policy/interface/PhabricatorExtendedPolicyInterface.php b/src/applications/policy/interface/PhabricatorExtendedPolicyInterface.php --- a/src/applications/policy/interface/PhabricatorExtendedPolicyInterface.php +++ b/src/applications/policy/interface/PhabricatorExtendedPolicyInterface.php @@ -62,8 +62,8 @@ * // ... * ); * - * @param const Capability being tested. - * @param PhabricatorUser Viewer whose capabilities are being tested. + * @param const $capability Capability being tested. + * @param PhabricatorUser $viewer Viewer whose capabilities are being tested. * @return list> List of extended policies. */ public function getExtendedPolicy($capability, PhabricatorUser $viewer); diff --git a/src/applications/policy/rule/PhabricatorPolicyRule.php b/src/applications/policy/rule/PhabricatorPolicyRule.php --- a/src/applications/policy/rule/PhabricatorPolicyRule.php +++ b/src/applications/policy/rule/PhabricatorPolicyRule.php @@ -120,9 +120,9 @@ * rendering a verdict about whether the user will be able to see the object * or not after applying the policy change. * - * @param PhabricatorPolicyInterface Object to pass a hint about. - * @param PhabricatorPolicyRule Rule to pass hint to. - * @param wild Hint. + * @param PhabricatorPolicyInterface $object Object to pass a hint about. + * @param PhabricatorPolicyRule $rule Rule to pass hint to. + * @param wild $hint Hint. * @return void */ public static function passTransactionHintToRule( diff --git a/src/applications/policy/storage/PhabricatorPolicy.php b/src/applications/policy/storage/PhabricatorPolicy.php --- a/src/applications/policy/storage/PhabricatorPolicy.php +++ b/src/applications/policy/storage/PhabricatorPolicy.php @@ -348,7 +348,7 @@ * policy. This is used to bulk load data (like project memberships) in order * to apply policy filters efficiently. * - * @param string Policy rule classname. + * @param string $rule_class Policy rule classname. * @return list List of values used in this policy. */ public function getCustomRuleValues($rule_class) { @@ -402,7 +402,7 @@ * set of unique users. In this case, neither is strictly stronger than * the other. * - * @param PhabricatorPolicy Other policy. + * @param PhabricatorPolicy $other Other policy. * @return bool `true` if this policy is more restrictive than the other * policy. */ diff --git a/src/applications/repository/daemon/PhabricatorRepositoryPullLocalDaemon.php b/src/applications/repository/daemon/PhabricatorRepositoryPullLocalDaemon.php --- a/src/applications/repository/daemon/PhabricatorRepositoryPullLocalDaemon.php +++ b/src/applications/repository/daemon/PhabricatorRepositoryPullLocalDaemon.php @@ -341,8 +341,8 @@ * With the `$consume` flag, an internal cursor will also be incremented so * that these messages are not returned by subsequent calls. * - * @param bool Pass `true` to consume these messages, so the process will - * not see them again. + * @param bool? $consume Pass `true` to consume these messages, so the + * process will not see them again. * @return list Pending update messages. * * @task pull diff --git a/src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php b/src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php --- a/src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php +++ b/src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php @@ -538,7 +538,7 @@ * * @task internal * - * @param list List of refs. + * @param list $refs List of refs. * @return list Sorted list of refs. */ private function sortRefs(array $refs) { diff --git a/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php b/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php --- a/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php +++ b/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php @@ -740,7 +740,7 @@ * error message. To prevent this, censor response bodies out of error * messages. * - * @param string Uncensored Mercurial command output. + * @param string $message Uncensored Mercurial command output. * @return string Censored Mercurial command output. */ private function censorMercurialErrorMessage($message) { diff --git a/src/applications/repository/engine/PhabricatorRepositoryRefEngine.php b/src/applications/repository/engine/PhabricatorRepositoryRefEngine.php --- a/src/applications/repository/engine/PhabricatorRepositoryRefEngine.php +++ b/src/applications/repository/engine/PhabricatorRepositoryRefEngine.php @@ -233,7 +233,7 @@ * point at commits which no longer exist. This can make commands issued later * fail. See T5839 for discussion. * - * @param list List of commit identifiers. + * @param list $identifiers List of commit identifiers. * @return list List with nonexistent identifiers removed. */ private function removeMissingCommits(array $identifiers) { diff --git a/src/applications/repository/graphcache/PhabricatorRepositoryGraphCache.php b/src/applications/repository/graphcache/PhabricatorRepositoryGraphCache.php --- a/src/applications/repository/graphcache/PhabricatorRepositoryGraphCache.php +++ b/src/applications/repository/graphcache/PhabricatorRepositoryGraphCache.php @@ -64,10 +64,10 @@ /** * Search the graph cache for the most modification to a path. * - * @param int The commit ID to search ancestors of. - * @param int The path ID to search for changes to. - * @param float Maximum number of seconds to spend trying to satisfy this - * query using the graph cache. By default, `0.5` (500ms). + * @param int $commit_id The commit ID to search ancestors of. + * @param int $path_id The path ID to search for changes to. + * @param float $time Maximum number of seconds to spend trying to satisfy + * this query using the graph cache. By default `0.5` (500ms). * @return mixed Commit ID, or `null` if no ancestors exist, or `false` if * the graph cache was unable to determine the answer. * @task query @@ -189,7 +189,7 @@ /** * Get the bucket key for a given commit ID. * - * @param int Commit ID. + * @param int $commit_id Commit ID. * @return int Bucket key. * @task cache */ @@ -201,7 +201,7 @@ /** * Get the cache key for a given bucket key (from @{method:getBucketKey}). * - * @param int Bucket key. + * @param int $bucket_key Bucket key. * @return string Cache key. * @task cache */ @@ -235,9 +235,10 @@ * Normally, this operates as a readthrough cache call. It can also be used * to force a cache update by passing the existing data to `$rebuild_data`. * - * @param int Bucket key, from @{method:getBucketKey}. - * @param mixed Current data, to force a cache rebuild of this bucket. - * @return array Data from the cache. + * @param int $bucket_key Bucket key, from @{method:getBucketKey}. + * @param mixed? $rebuild_data Current data, to force a cache rebuild of + * this bucket. + * @return array Data from the cache. * @task cache */ private function getBucketData($bucket_key, $rebuild_data = null) { @@ -287,9 +288,9 @@ /** * Rebuild a cache bucket, amending existing data if available. * - * @param int Bucket key, from @{method:getBucketKey}. - * @param array Existing bucket data. - * @return array Rebuilt bucket data. + * @param int $bucket_key Bucket key, from @{method:getBucketKey}. + * @param array $current_data Existing bucket data. + * @return array Rebuilt bucket data. * @task cache */ private function rebuildBucket($bucket_key, array $current_data) { diff --git a/src/applications/repository/storage/PhabricatorRepository.php b/src/applications/repository/storage/PhabricatorRepository.php --- a/src/applications/repository/storage/PhabricatorRepository.php +++ b/src/applications/repository/storage/PhabricatorRepository.php @@ -1288,7 +1288,7 @@ /** * Determine if a protocol is SSH or SSH-like. * - * @param string A protocol string, like "http" or "ssh". + * @param string $protocol A protocol string, like "http" or "ssh". * @return bool True if the protocol is SSH-like. * @task uri */ @@ -1701,7 +1701,7 @@ * 2037). We adjust the pull frequency based on when the most recent commit * occurred. * - * @param int The minimum update interval to use, in seconds. + * @param int? $minimum The minimum update interval to use, in seconds. * @return int Repository update interval, in seconds. */ public function loadUpdateInterval($minimum = 15) { @@ -1836,8 +1836,8 @@ * with repository services. This method provides lower-level resolution of * services, returning raw URIs. * - * @param PhabricatorUser Viewing user. - * @param map Constraints on selectable services. + * @param PhabricatorUser $viewer Viewing user. + * @param map $options Constraints on selectable services. * @return string|null URI, or `null` for local repositories. */ public function getAlmanacServiceURI( @@ -2168,8 +2168,8 @@ * * For lower-level service resolution, see @{method:getAlmanacServiceURI}. * - * @param PhabricatorUser Viewing user. - * @param bool `true` to throw if a client would be returned. + * @param PhabricatorUser $viewer Viewing user. + * @param bool? $never_proxy `true` to throw if a client would be returned. * @return ConduitClient|null Client, or `null` for local repositories. */ public function newConduitClient( diff --git a/src/applications/search/engine/PhabricatorApplicationSearchEngine.php b/src/applications/search/engine/PhabricatorApplicationSearchEngine.php --- a/src/applications/search/engine/PhabricatorApplicationSearchEngine.php +++ b/src/applications/search/engine/PhabricatorApplicationSearchEngine.php @@ -126,7 +126,7 @@ /** * Create a saved query object from the request. * - * @param AphrontRequest The search request. + * @param AphrontRequest $request The search request. * @return PhabricatorSavedQuery */ public function buildSavedQueryFromRequest(AphrontRequest $request) { @@ -147,7 +147,7 @@ /** * Executes the saved query. * - * @param PhabricatorSavedQuery The saved query to operate on. + * @param PhabricatorSavedQuery $original The saved query to operate on. * @return PhabricatorQuery The result of the query. */ public function buildQueryFromSavedQuery(PhabricatorSavedQuery $original) { @@ -201,7 +201,7 @@ * hook to keep old queries working the way users expect, by reading, * adjusting, and overwriting parameters. * - * @param PhabricatorSavedQuery Saved query which will be executed. + * @param PhabricatorSavedQuery $saved Saved query which will be executed. * @return void */ protected function willUseSavedQuery(PhabricatorSavedQuery $saved) { @@ -215,8 +215,8 @@ /** * Builds the search form using the request. * - * @param AphrontFormView Form to populate. - * @param PhabricatorSavedQuery The query from which to build the form. + * @param AphrontFormView $form Form to populate. + * @param PhabricatorSavedQuery $saved Query from which to build the form. * @return void */ public function buildSearchForm( @@ -400,7 +400,7 @@ * Return an application URI corresponding to the results page of a query. * Normally, this is something like `/application/query/QUERYKEY/`. * - * @param string The query key to build a URI for. + * @param string $query_key The query key to build a URI for. * @return string URI where the query can be executed. * @task uri */ @@ -730,9 +730,9 @@ * links to pages (like "alincoln's open revisions") without needing to make * API calls. * - * @param AphrontRequest Request to read user PHIDs from. - * @param string Key to read in the request. - * @param list Other permitted PHID types. + * @param AphrontRequest $request Request to read user PHIDs from. + * @param string $key Key to read in the request. + * @param list? $allow_types Other permitted PHID types. * @return list List of user PHIDs and selector functions. * @task read */ @@ -782,8 +782,8 @@ /** * Read a list of subscribers from a request in a flexible way. * - * @param AphrontRequest Request to read PHIDs from. - * @param string Key to read in the request. + * @param AphrontRequest $request Request to read PHIDs from. + * @param string $key Key to read in the request. * @return list List of object PHIDs. * @task read */ @@ -805,9 +805,10 @@ * comma-delimited forms. Objects can be specified either by PHID or by * object name. * - * @param AphrontRequest Request to read PHIDs from. - * @param string Key to read in the request. - * @param list Optional, list of permitted PHID types. + * @param AphrontRequest $request Request to read PHIDs from. + * @param string $key Key to read in the request. + * @param list? $allow_types Optional, list of permitted PHID + * types. * @return list List of object PHIDs. * * @task read @@ -853,8 +854,8 @@ * This provides flexibility when constructing URIs, especially from external * sources. * - * @param AphrontRequest Request to read strings from. - * @param string Key to read in the request. + * @param AphrontRequest $request Request to read strings from. + * @param string $key Key to read in the request. * @return list List of values. */ protected function readListFromRequest( 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 @@ -33,7 +33,7 @@ * The key should be a short, unique (within a search engine) string which * does not contain any special characters. * - * @param string Unique key which identifies the field. + * @param string $key Unique key which identifies the field. * @return this * @task config */ @@ -59,7 +59,7 @@ * * This should be a short text string, like "Reviewers" or "Colors". * - * @param string Short, human-readable field label. + * @param string $label Short, human-readable field label. * @return this * task config */ @@ -86,7 +86,7 @@ * Engines do not need to do this explicitly; it will be done on their * behalf by the caller. * - * @param PhabricatorUser Viewer. + * @param PhabricatorUser $viewer Viewer. * @return this * @task config */ @@ -115,7 +115,7 @@ * an alias like `authors` to let users write `&authors=alincoln` instead of * `&authorPHIDs=alincoln`. This is a little easier to use. * - * @param list List of aliases for this field. + * @param list $aliases List of aliases for this field. * @return this * @task config */ @@ -142,7 +142,7 @@ * This can allow you to choose a more usable key for API endpoints. * If no key is provided, the main key is used. * - * @param string Alternate key for Conduit. + * @param string $conduit_key Alternate key for Conduit. * @return this * @task config */ @@ -170,7 +170,7 @@ /** * Set a human-readable description for this field. * - * @param string Human-readable description. + * @param string $description Human-readable description. * @return this * @task config */ @@ -194,7 +194,7 @@ /** * Hide this field from the web UI. * - * @param bool True to hide the field from the web UI. + * @param bool $is_hidden True to hide the field from the web UI. * @return this * @task config */ @@ -400,8 +400,8 @@ * This provides flexibility when constructing URIs, especially from external * sources. * - * @param AphrontRequest Request to read strings from. - * @param string Key to read in the request. + * @param AphrontRequest $request Request to read strings from. + * @param string $key Key to read in the request. * @return list List of values. * @task utility */ diff --git a/src/applications/search/fulltextstorage/PhabricatorFulltextStorageEngine.php b/src/applications/search/fulltextstorage/PhabricatorFulltextStorageEngine.php --- a/src/applications/search/fulltextstorage/PhabricatorFulltextStorageEngine.php +++ b/src/applications/search/fulltextstorage/PhabricatorFulltextStorageEngine.php @@ -47,7 +47,7 @@ /** * Update the index for an abstract document. * - * @param PhabricatorSearchAbstractDocument Document to update. + * @param PhabricatorSearchAbstractDocument $document Document to update. * @return void */ abstract public function reindexAbstractDocument( @@ -56,7 +56,7 @@ /** * Execute a search query. * - * @param PhabricatorSavedQuery A query to execute. + * @param PhabricatorSavedQuery $query A query to execute. * @return list A list of matching PHIDs. */ abstract public function executeSearch(PhabricatorSavedQuery $query); diff --git a/src/applications/settings/panel/PhabricatorSettingsPanel.php b/src/applications/settings/panel/PhabricatorSettingsPanel.php --- a/src/applications/settings/panel/PhabricatorSettingsPanel.php +++ b/src/applications/settings/panel/PhabricatorSettingsPanel.php @@ -222,7 +222,7 @@ * Generally, render your settings panel by returning a form, then return * a redirect when the user saves settings. * - * @param AphrontRequest Incoming request. + * @param AphrontRequest $request Incoming request. * @return wild Response to request, either as an * @{class:AphrontResponse} or something which can * be composed into a @{class:AphrontView}. @@ -234,7 +234,7 @@ /** * Get the URI for this panel. * - * @param string? Optional path to append. + * @param string? $path Optional path to append. * @return string Relative URI for the panel. * @task panel */ diff --git a/src/applications/settings/query/PhabricatorUserPreferencesQuery.php b/src/applications/settings/query/PhabricatorUserPreferencesQuery.php --- a/src/applications/settings/query/PhabricatorUserPreferencesQuery.php +++ b/src/applications/settings/query/PhabricatorUserPreferencesQuery.php @@ -49,7 +49,8 @@ * If no settings exist for a user, a new empty settings object with * appropriate defaults is returned. * - * @param bool True to generate synthetic preferences for missing users. + * @param bool $synthetic True to generate synthetic preferences for missing + * users. */ public function needSyntheticPreferences($synthetic) { $this->synthetic = $synthetic; diff --git a/src/applications/settings/storage/PhabricatorUserPreferences.php b/src/applications/settings/storage/PhabricatorUserPreferences.php --- a/src/applications/settings/storage/PhabricatorUserPreferences.php +++ b/src/applications/settings/storage/PhabricatorUserPreferences.php @@ -119,7 +119,7 @@ /** * Load or create a preferences object for the given user. * - * @param PhabricatorUser User to load or create preferences for. + * @param PhabricatorUser $user User to load or create preferences for. */ public static function loadUserPreferences(PhabricatorUser $user) { return id(new PhabricatorUserPreferencesQuery()) @@ -134,7 +134,7 @@ * * If no global preferences exist, an empty preferences object is returned. * - * @param PhabricatorUser Viewing user. + * @param PhabricatorUser $viewer Viewing user. */ public static function loadGlobalPreferences(PhabricatorUser $viewer) { $global = id(new PhabricatorUserPreferencesQuery()) diff --git a/src/applications/spaces/query/PhabricatorSpacesNamespaceQuery.php b/src/applications/spaces/query/PhabricatorSpacesNamespaceQuery.php --- a/src/applications/spaces/query/PhabricatorSpacesNamespaceQuery.php +++ b/src/applications/spaces/query/PhabricatorSpacesNamespaceQuery.php @@ -212,7 +212,7 @@ * This is intended to simplify performing a bunch of redundant checks; you * can intentionally pass any value in (including `null`). * - * @param wild + * @param wild $object * @return phid|null */ public static function getObjectSpacePHID($object) { 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 @@ -18,7 +18,7 @@ * (or been subscribed) to the object, and will be added even if they * had previously unsubscribed. * - * @param list List of PHIDs to explicitly subscribe. + * @param list $phids List of PHIDs to explicitly subscribe. * @return this */ public function subscribeExplicit(array $phids) { @@ -32,7 +32,7 @@ * implicitly subscribes them (e.g., adding a comment) but it will be * suppressed if they've previously unsubscribed from the object. * - * @param list List of PHIDs to implicitly subscribe. + * @param list $phids List of PHIDs to implicitly subscribe. * @return this */ public function subscribeImplicit(array $phids) { @@ -45,7 +45,7 @@ * Unsubscribe PHIDs and mark them as unsubscribed, so implicit subscriptions * will not resubscribe them. * - * @param list List of PHIDs to unsubscribe. + * @param list $phids List of PHIDs to unsubscribe. * @return this */ public function unsubscribe(array $phids) { diff --git a/src/applications/subscriptions/interface/PhabricatorSubscribableInterface.php b/src/applications/subscriptions/interface/PhabricatorSubscribableInterface.php --- a/src/applications/subscriptions/interface/PhabricatorSubscribableInterface.php +++ b/src/applications/subscriptions/interface/PhabricatorSubscribableInterface.php @@ -8,7 +8,8 @@ * irrevocably a subscriber). This will, e.g., cause the UI to render * "Automatically Subscribed" instead of "Subscribe". * - * @param PHID PHID (presumably a user) to test for automatic subscription. + * @param PHID $phid PHID (presumably a user) to test for automatic + * subscription. * @return bool True if the object/user is automatically subscribed. */ public function isAutomaticallySubscribed($phid); diff --git a/src/applications/system/engine/PhabricatorSystemActionEngine.php b/src/applications/system/engine/PhabricatorSystemActionEngine.php --- a/src/applications/system/engine/PhabricatorSystemActionEngine.php +++ b/src/applications/system/engine/PhabricatorSystemActionEngine.php @@ -33,9 +33,9 @@ * If any actor is exceeding their rate limit, this method throws a * @{class:PhabricatorSystemActionRateLimitException}. * - * @param list List of actors. - * @param PhabricatorSystemAction Action being taken. - * @param float Score or credit, see above. + * @param list $actors List of actors. + * @param PhabricatorSystemAction $action Action being taken. + * @param float $score Score or credit, see above. * @return void */ public static function willTakeAction( @@ -174,7 +174,7 @@ * Reset all action counts for actions taken by some set of actors in the * previous action window. * - * @param list Actors to reset counts for. + * @param list $actors Actors to reset counts for. * @return int Number of actions cleared. */ public static function resetActions(array $actors) { 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 @@ -343,7 +343,7 @@ * Set default placeholder plain text in the comment textarea of the engine. * To be overwritten by conditions defined in the child EditEngine class. * - * @param object Object in which the comment textarea is displayed. + * @param object $object Object in which the comment textarea is displayed. * @return string Placeholder text to display in the comment textarea. * @task text */ @@ -678,7 +678,7 @@ * Initialize a new object for object creation via Conduit. * * @return object Newly initialized object. - * @param list Raw transactions. + * @param list $raw_xactions Raw transactions. * @task load */ protected function newEditableObjectFromConduit(array $raw_xactions) { @@ -698,7 +698,7 @@ /** * Flag this workflow as a create or edit. * - * @param bool True if this is a create workflow. + * @param bool $is_create True if this is a create workflow. * @return this * @task load */ @@ -712,9 +712,9 @@ * Try to load an object by ID, PHID, or monogram. This is done primarily * to make Conduit a little easier to use. * - * @param wild ID, PHID, or monogram. - * @param list List of required capability constants, or omit for - * defaults. + * @param wild $identifier ID, PHID, or monogram. + * @param list? $capabilities List of required capability constants, + * or omit for defaults. * @return object Corresponding editable object. * @task load */ @@ -792,9 +792,9 @@ /** * Load an object by ID. * - * @param int Object ID. - * @param list List of required capability constants, or omit for - * defaults. + * @param int $id Object ID. + * @param list? $capabilities List of required capability constants, + * or omit for defaults. * @return object|null Object, or null if no such object exists. * @task load */ @@ -809,9 +809,9 @@ /** * Load an object by PHID. * - * @param phid Object PHID. - * @param list List of required capability constants, or omit for - * defaults. + * @param phid $phid Object PHID. + * @param list? $capabilities List of required capability constants, + * or omit for defaults. * @return object|null Object, or null if no such object exists. * @task load */ @@ -826,9 +826,9 @@ /** * Load an object given a configured query. * - * @param PhabricatorPolicyAwareQuery Configured query. - * @param list List of required capability constants, or omit for - * defaults. + * @param PhabricatorPolicyAwareQuery $query Configured query. + * @param list? $capabilities List of required capability constants, + * or omit for defaults. * @return object|null Object, or null if no such object exists. * @task load */ @@ -860,7 +860,7 @@ /** * Verify that an object is appropriate for editing. * - * @param wild Loaded value. + * @param wild $object Loaded value. * @return void * @task load */ @@ -1767,7 +1767,7 @@ /** * Respond to a request for documentation on HTTP parameters. * - * @param object Editable object. + * @param object $object Editable object. * @return AphrontResponse Response object. * @task http */ @@ -2244,10 +2244,10 @@ * Generate transactions which can be applied from edit actions in a Conduit * request. * - * @param ConduitAPIRequest The request. - * @param list Raw conduit transactions. - * @param list Supported edit types. - * @param PhabricatorApplicationTransaction Template transaction. + * @param ConduitAPIRequest $request The request. + * @param list $xactions Raw conduit transactions. + * @param list $types Supported edit types. + * @param PhabricatorApplicationTransaction $template Template transaction. * @return list Generated transactions. * @task conduit */ diff --git a/src/applications/transactions/editfield/PhabricatorEditField.php b/src/applications/transactions/editfield/PhabricatorEditField.php --- a/src/applications/transactions/editfield/PhabricatorEditField.php +++ b/src/applications/transactions/editfield/PhabricatorEditField.php @@ -573,8 +573,8 @@ * Most fields do not need to store these values or deal with initial value * handling. * - * @param AphrontRequest Request to read from. - * @param string Key to read. + * @param AphrontRequest $request Request to read from. + * @param string $key Key to read. * @return wild Value read from request. */ protected function getInitialValueFromSubmit(AphrontRequest $request, $key) { 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 @@ -140,7 +140,8 @@ * nothing (e.g., empty comment with a status change that has already been * performed by another user). * - * @param bool True to drop transactions without effect and continue. + * @param bool $continue True to drop transactions without effect and + * continue. * @return this */ public function setContinueOnNoEffect($continue) { @@ -168,8 +169,8 @@ * (like the priority, batch, and merge editors in Maniphest), these * operations can continue to function even if an object is outdated. * - * @param bool True to continue when transactions don't completely satisfy - * all required fields. + * @param bool $continue_on_missing_fields True to continue when transactions + * don't completely satisfy all required fields. * @return this */ public function setContinueOnMissingFields($continue_on_missing_fields) { @@ -2853,11 +2854,11 @@ * missing, by detecting that the object has no field value and there is no * transaction which sets one. * - * @param PhabricatorLiskDAO Object being edited. - * @param string Transaction type to validate. - * @param list Transactions of given type, - * which may be empty if the edit does not apply any transactions of the - * given type. + * @param PhabricatorLiskDAO $object Object being edited. + * @param string $type Transaction type to validate. + * @param list $xactions Transactions of + * given type, which may be empty if the edit does not apply any + * transactions of the given type. * @return list List of * validation errors. */ @@ -3277,9 +3278,9 @@ * This will return `true` if the net effect of the object and transactions * is an empty field. * - * @param wild Current field value. - * @param list Transactions editing the - * field. + * @param wild $field_value Current field value. + * @param list $xactions Transactions + * editing the field. * @return bool True if the field will be an empty text field after edits. */ protected function validateIsEmptyTextField($field_value, array $xactions) { @@ -4600,7 +4601,8 @@ * * This method is used to load state when running worker operations. * - * @param dict Editor state, from @{method:getWorkerState}. + * @param dict $state Editor state, from + @{method:getWorkerState}. * @return this * @task workers */ @@ -4626,7 +4628,7 @@ * Hook; set custom properties on the editor from data emitted by * @{method:getCustomWorkerState}. * - * @param dict Custom state, + * @param dict $state Custom state, * from @{method:getCustomWorkerState}. * @return this * @task workers @@ -4674,8 +4676,8 @@ * * See @{method:getCustomWorkerStateEncoding}. * - * @param map Map of values to encode. - * @param map Map of encodings to apply. + * @param map $state Map of values to encode. + * @param map $encodings Map of encodings to apply. * @return map Map of encoded values. * @task workers */ @@ -4720,8 +4722,8 @@ * * See @{method:getCustomWorkerStateEncoding}. * - * @param map Map of encoded values. - * @param map Map of encodings. + * @param map $state Map of encoded values. + * @param map $encodings Map of encodings. * @return map Map of decoded values. * @task workers */ @@ -4759,7 +4761,7 @@ * If the list of PHIDs include mutually exclusive projects, remove the * conflicting projects. * - * @param list List of project PHIDs. + * @param list $phids List of project PHIDs. * @return list List with conflicts removed. */ private function applyProjectConflictRules(array $phids) { diff --git a/src/applications/transactions/storage/PhabricatorApplicationTransaction.php b/src/applications/transactions/storage/PhabricatorApplicationTransaction.php --- a/src/applications/transactions/storage/PhabricatorApplicationTransaction.php +++ b/src/applications/transactions/storage/PhabricatorApplicationTransaction.php @@ -1696,7 +1696,7 @@ * Should this transaction be visually grouped with an existing transaction * group? * - * @param list List of transactions. + * @param list $group List of transactions. * @return bool True to display in a group with the other transactions. */ public function shouldDisplayGroupWith(array $group) { diff --git a/src/applications/transactions/storage/PhabricatorModularTransactionType.php b/src/applications/transactions/storage/PhabricatorModularTransactionType.php --- a/src/applications/transactions/storage/PhabricatorModularTransactionType.php +++ b/src/applications/transactions/storage/PhabricatorModularTransactionType.php @@ -428,8 +428,9 @@ * additional capability or policy requirement above and beyond CAN_EDIT, it * is usually better implemented as a validation check. * - * @param object Object being edited. - * @param PhabricatorApplicationTransaction Transaction being applied. + * @param object $object Object being edited. + * @param PhabricatorApplicationTransaction $xaction Transaction being + * applied. * @return null|const|list A capability constant (or list of * capability constants) which the actor must have on the object. You can * return `null` as a shorthand for "no capabilities are required". diff --git a/src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php b/src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php --- a/src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php +++ b/src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php @@ -212,7 +212,7 @@ * For datasources backed by database objects, this is often much less * efficient than filtering at the query level. * - * @param list List of typeahead results. + * @param list $results List of typeahead results. * @return list Filtered results. */ protected function filterResultsAgainstTokens(array $results) { 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 @@ -26,7 +26,7 @@ * When too many keys are inserted, the oldest keys are removed from the * cache. Setting a limit of `0` disables the cache. * - * @param int Maximum number of items to store in the cache. + * @param int $limit Maximum number of items to store in the cache. * @return this */ public function setLimit($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 @@ -30,9 +30,9 @@ * Get a single key from cache. See @{method:getKeys} to get multiple keys at * once. * - * @param string Key to retrieve. - * @param wild Optional value to return if the key is not found. By - * default, returns null. + * @param string $key Key to retrieve. + * @param wild? $default Optional value to return if the key is not + * found. By default, returns null. * @return wild Cache value (on cache hit) or default value (on cache * miss). * @task kvimpl @@ -49,9 +49,9 @@ * * See @{method:setKeys} for a description of TTLs. * - * @param string Key to set. - * @param wild Value to set. - * @param int|null Optional TTL. + * @param string $key Key to set. + * @param wild $value Value to set. + * @param int|null? $ttl Optional TTL. * @return this * @task kvimpl */ @@ -64,7 +64,7 @@ * Delete a key from the cache. See @{method:deleteKeys} to delete multiple * keys at once. * - * @param string Key to delete. + * @param string $key Key to delete. * @return this * @task kvimpl */ @@ -76,7 +76,7 @@ /** * Get data from the cache. * - * @param list List of cache keys to retrieve. + * @param list $keys List of cache keys to retrieve. * @return dict Dictionary of keys that were found in the * cache. Keys not present in the cache are * omitted, so you can detect a cache miss. @@ -92,8 +92,8 @@ * after a specified number of seconds. By default, there is no expiration * policy and data will persist in cache indefinitely. * - * @param dict Map of cache keys to values. - * @param int|null TTL for cache keys, in seconds. + * @param dict $keys Map of cache keys to values. + * @param int|null? $ttl TTL for cache keys, in seconds. * @return this * @task kvimpl */ @@ -103,7 +103,7 @@ /** * Delete a list of keys from the cache. * - * @param list List of keys to delete. + * @param list $keys List of keys to delete. * @return this * @task kvimpl */ 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 @@ -17,7 +17,7 @@ /** * Set a profiler for cache operations. * - * @param PhutilServiceProfiler Service profiler. + * @param PhutilServiceProfiler $profiler Service profiler. * @return this * @task kvimpl */ 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 @@ -38,7 +38,8 @@ /** * Set the caches which comprise this stack. * - * @param list Ordered list of key-value caches. + * @param list $caches Ordered list of key-value + * caches. * @return this * @task config */ @@ -62,7 +63,7 @@ * // TTL does not persist; this will use no TTL. * $stack->getKey('hedgehog'); * - * @param int TTL in seconds. + * @param int $ttl TTL in seconds. * @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 @@ -100,7 +100,7 @@ * ), * )); * - * @param list List of server specifications. + * @param list $servers List of server specifications. * @return this * @task memcache */ diff --git a/src/infrastructure/contentsource/PhabricatorContentSource.php b/src/infrastructure/contentsource/PhabricatorContentSource.php --- a/src/infrastructure/contentsource/PhabricatorContentSource.php +++ b/src/infrastructure/contentsource/PhabricatorContentSource.php @@ -22,10 +22,10 @@ /** * Construct a new content source object. * - * @param const The source type constant to build a source for. - * @param array Source parameters. - * @param bool True to suppress errors and force construction of a source - * even if the source type is not valid. + * @param const $source The source type constant to build a source for. + * @param array? $params Source parameters. + * @param bool? $force True to suppress errors and force construction of a + * source even if the source type is not valid. * @return PhabricatorContentSource New source object. */ final public static function newForSource( 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 @@ -243,7 +243,7 @@ * For general implementations, the general field implementation can return * multiple field instances here. * - * @param object The object to create fields for. + * @param object $object The object to create fields for. * @return list List of fields. * @task core */ @@ -382,7 +382,7 @@ * Set the proxy implementation for this field. See @{method:canSetProxy} for * discussion of field proxies. * - * @param PhabricatorCustomField Field implementation. + * @param PhabricatorCustomField $proxy Field implementation. * @return this * @task proxy */ @@ -421,7 +421,8 @@ /** * Sets the object this field belongs to. * - * @param PhabricatorCustomFieldInterface The object this field belongs to. + * @param PhabricatorCustomFieldInterface $object The object this field + * belongs to. * @return this * @task context */ @@ -440,7 +441,8 @@ /** * Read object data into local field storage, if applicable. * - * @param PhabricatorCustomFieldInterface The object this field belongs to. + * @param PhabricatorCustomFieldInterface $object The object this field + * belongs to. * @return this * @task context */ @@ -607,7 +609,7 @@ * need to undo whatever serialization you applied in * @{method:getValueForStorage}. * - * @param string|null Serialized field representation (from + * @param string|null $value Serialized field representation (from * @{method:getValueForStorage}) or null if no value has * ever been stored. * @return this @@ -729,7 +731,7 @@ /** * Build and populate storage for a string index. * - * @param string String to index. + * @param string $value String to index. * @return PhabricatorCustomFieldStringIndexStorage Populated storage. * @task appsearch */ @@ -748,7 +750,7 @@ /** * Build and populate storage for a numeric index. * - * @param string Numeric value to index. + * @param string $value Numeric value to index. * @return PhabricatorCustomFieldNumericIndexStorage Populated storage. * @task appsearch */ @@ -767,8 +769,9 @@ * Read a query value from a request, for storage in a saved query. Normally, * this method should, e.g., read a string out of the request. * - * @param PhabricatorApplicationSearchEngine Engine building the query. - * @param AphrontRequest Request to read from. + * @param PhabricatorApplicationSearchEngine $engine Engine building the + * query. + * @param AphrontRequest $request Request to read from. * @return wild * @task appsearch */ @@ -789,9 +792,10 @@ * use `with...()` methods to apply filters or other constraints to the * query. * - * @param PhabricatorApplicationSearchEngine Engine executing the query. - * @param PhabricatorCursorPagedPolicyAwareQuery Query to constrain. - * @param wild Constraint provided by the user. + * @param PhabricatorApplicationSearchEngine $engine Engine executing the + * query. + * @param PhabricatorCursorPagedPolicyAwareQuery $query Query to constrain. + * @param wild $value Constraint provided by the user. * @return void * @task appsearch */ @@ -812,9 +816,10 @@ /** * Append search controls to the interface. * - * @param PhabricatorApplicationSearchEngine Engine constructing the form. - * @param AphrontFormView The form to update. - * @param wild Value from the saved query. + * @param PhabricatorApplicationSearchEngine $engine Engine constructing the + * form. + * @param AphrontFormView $form The form to update. + * @param wild $value Value from the saved query. * @return void * @task appsearch */ @@ -1000,12 +1005,12 @@ * when a transaction would set a field to an invalid value, or when a field * is required but no transactions provide value. * - * @param PhabricatorLiskDAO Editor applying the transactions. - * @param string Transaction type. This type is always + * @param PhabricatorLiskDAO $editor Editor applying the transactions. + * @param string $type Transaction type. This type is always * `PhabricatorTransactions::TYPE_CUSTOMFIELD`, it is provided for * convenience when constructing exceptions. - * @param list Transactions being applied, - * which may be empty if this field is not being edited. + * @param list $xactions Transactions + * being applied, which may be empty if this field is not being edited. * @return list Validation * errors. * @@ -1566,7 +1571,7 @@ /** * Get the Herald value type for the given condition. * - * @param const Herald condition constant. + * @param const $condition Herald condition constant. * @return const|null Herald value type, or null to use the default. * @task herald */ diff --git a/src/infrastructure/customfield/field/PhabricatorCustomFieldList.php b/src/infrastructure/customfield/field/PhabricatorCustomFieldList.php --- a/src/infrastructure/customfield/field/PhabricatorCustomFieldList.php +++ b/src/infrastructure/customfield/field/PhabricatorCustomFieldList.php @@ -46,7 +46,8 @@ /** * Read stored values for all fields which support storage. * - * @param PhabricatorCustomFieldInterface Object to read field values for. + * @param PhabricatorCustomFieldInterface $object Object to read field values + * for. * @return void */ public function readFieldsFromStorage( diff --git a/src/infrastructure/customfield/storage/PhabricatorCustomFieldStorage.php b/src/infrastructure/customfield/storage/PhabricatorCustomFieldStorage.php --- a/src/infrastructure/customfield/storage/PhabricatorCustomFieldStorage.php +++ b/src/infrastructure/customfield/storage/PhabricatorCustomFieldStorage.php @@ -44,7 +44,7 @@ * The keys in the result should correspond to the keys in the input. The * fields in the list may belong to different objects. * - * @param map Map of fields. + * @param map $fields Map of fields. * @return map Map of available field data. */ final public function loadStorageSourceData(array $fields) { diff --git a/src/infrastructure/daemon/PhabricatorDaemon.php b/src/infrastructure/daemon/PhabricatorDaemon.php --- a/src/infrastructure/daemon/PhabricatorDaemon.php +++ b/src/infrastructure/daemon/PhabricatorDaemon.php @@ -24,7 +24,7 @@ * Format a command so it executes as the daemon user, if a daemon user is * defined. This wraps the provided command in `sudo -u ...`, roughly. * - * @param PhutilCommandString Command to execute. + * @param PhutilCommandString $command Command to execute. * @return PhutilCommandString `sudo` version of the command. */ public static function sudoCommandAsDaemonUser($command) { diff --git a/src/infrastructure/daemon/PhutilDaemonHandle.php b/src/infrastructure/daemon/PhutilDaemonHandle.php --- a/src/infrastructure/daemon/PhutilDaemonHandle.php +++ b/src/infrastructure/daemon/PhutilDaemonHandle.php @@ -328,8 +328,8 @@ /** * Dispatch an event to event listeners. * - * @param string Event type. - * @param dict Event parameters. + * @param string $type Event type. + * @param dict? $params Event parameters. * @return void */ private function dispatchEvent($type, array $params = array()) { diff --git a/src/infrastructure/daemon/PhutilDaemonOverseerModule.php b/src/infrastructure/daemon/PhutilDaemonOverseerModule.php --- a/src/infrastructure/daemon/PhutilDaemonOverseerModule.php +++ b/src/infrastructure/daemon/PhutilDaemonOverseerModule.php @@ -47,8 +47,8 @@ * return; * } * - * @param string Throttle key. - * @param float Duration in seconds. + * @param string $name Throttle key. + * @param float $duration Duration in seconds. * @return bool True to throttle the check. */ protected function shouldThrottle($name, $duration) { diff --git a/src/infrastructure/daemon/workers/PhabricatorTriggerDaemon.php b/src/infrastructure/daemon/workers/PhabricatorTriggerDaemon.php --- a/src/infrastructure/daemon/workers/PhabricatorTriggerDaemon.php +++ b/src/infrastructure/daemon/workers/PhabricatorTriggerDaemon.php @@ -122,7 +122,7 @@ * Process all of the triggers which have been updated since the last time * the daemon ran, scheduling them into the event table. * - * @param int Cursor for the next version update to process. + * @param int $cursor Cursor for the next version update to process. * @return void */ private function scheduleTriggers($cursor) { @@ -326,7 +326,7 @@ /** * Run the garbage collector for up to a specified number of seconds. * - * @param int Number of seconds the GC may run for. + * @param int $duration Number of seconds the GC may run for. * @return int Number of seconds remaining in the time budget. * @task garbage */ 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 @@ -68,12 +68,12 @@ * retrying. For most tasks you can leave this at `null`, which will give you * a short default retry period (currently 60 seconds). * - * @param PhabricatorWorkerTask The task itself. This object is probably - * useful mostly to examine the failure count - * if you want to implement staggered retries, - * or to examine the execution exception if - * you want to react to different failures in - * different ways. + * @param PhabricatorWorkerTask $task The task itself. This object is + * probably useful mostly to examine the + * failure count if you want to implement + * staggered retries, or to examine the + * execution exception if you want to react to + * different failures in different ways. * @return int|null Number of seconds to wait between retries, * or null for a default retry period * (currently 60 seconds). @@ -230,9 +230,9 @@ * * The followup task will be queued only if this task completes cleanly. * - * @param string Task class to queue. - * @param array Data for the followup task. - * @param array Options for the followup task. + * @param string $class Task class to queue. + * @param array $data Data for the followup task. + * @param array? $options Options for the followup task. * @return this */ final protected function queueTask( @@ -261,7 +261,7 @@ * this method to force the queue to flush before failing (for example, if * you are using queues to improve locking behavior). * - * @param map Optional default options. + * @param map? $defaults Optional default options. */ final public function flushTaskQueue($defaults = array()) { foreach ($this->getQueuedTasks() as $task) { @@ -286,7 +286,7 @@ * This method does not provide any assurances about when these tasks will * execute, or even guarantee that it will have any effect at all. * - * @param list List of task IDs to try to awaken. + * @param list $ids List of task IDs to try to awaken. * @return void */ final public static function awakenTaskIDs(array $ids) { diff --git a/src/infrastructure/daemon/workers/action/PhabricatorTriggerAction.php b/src/infrastructure/daemon/workers/action/PhabricatorTriggerAction.php --- a/src/infrastructure/daemon/workers/action/PhabricatorTriggerAction.php +++ b/src/infrastructure/daemon/workers/action/PhabricatorTriggerAction.php @@ -28,7 +28,7 @@ /** * Validate action configuration. * - * @param map Map of action properties. + * @param map $properties Map of action properties. * @return void */ abstract public function validateProperties(array $properties); @@ -62,10 +62,10 @@ * may be simplest to pass the trigger time to the task and then make the * decision to discard the action there. * - * @param int|null Last time the event occurred, or null if it has never - * triggered before. - * @param int The scheduled time for the current action. This may be - * significantly different from the current time. + * @param int|null $last_epoch Last time the event occurred, or null if it + * has never triggered before. + * @param int $this_epoch The scheduled time for the current action. This + * may be significantly different from the current time. * @return void */ abstract public function execute($last_epoch, $this_epoch); diff --git a/src/infrastructure/daemon/workers/clock/PhabricatorTriggerClock.php b/src/infrastructure/daemon/workers/clock/PhabricatorTriggerClock.php --- a/src/infrastructure/daemon/workers/clock/PhabricatorTriggerClock.php +++ b/src/infrastructure/daemon/workers/clock/PhabricatorTriggerClock.php @@ -32,7 +32,7 @@ /** * Validate clock configuration. * - * @param map Map of clock properties. + * @param map $properties Map of clock properties. * @return void */ abstract public function validateProperties(array $properties); @@ -64,9 +64,10 @@ * week to 3 minutes from now, the clock may reschedule the notification to * occur 12 minutes ago. This will cause it to execute immediately. * - * @param int|null Last time the event occurred, or null if it has never - * triggered before. - * @param bool True if this is a reschedule after a successful trigger. + * @param int|null $last_epoch Last time the event occurred, or null if it + * has never triggered before. + * @param bool $is_reschedule True if this is a reschedule after a successful + * trigger. * @return int|null Next event, or null to decline to reschedule. */ abstract public function getNextEventEpoch($last_epoch, $is_reschedule); 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 @@ -30,7 +30,7 @@ * This can be used to show which tasks are coming up next without altering * the queue's behavior. * - * @param bool True to skip the lease acquisition step. + * @param bool $skip True to skip the lease acquisition step. */ public function setSkipLease($skip) { $this->skipLease = $skip; @@ -58,8 +58,8 @@ * leasing using @{method:setSkipLease}. These options are intended for use * when displaying task status information. * - * @param mixed `true` to select only leased tasks, `false` to select only - * unleased tasks (default), or `null` to select both. + * @param mixed $leased `true` to select only leased tasks, `false` to select + * only unleased tasks (default), or `null` to select both. * @return this */ public function withLeasedTasks($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 @@ -59,7 +59,7 @@ * triggers which have been scheduled to execute. You should not use this * ordering when querying for specific triggers, e.g. by ID or PHID. * - * @param const Result order. + * @param const $order Result order. * @return this */ public function setOrder($order) { diff --git a/src/infrastructure/daemon/workers/storage/PhabricatorWorkerTrigger.php b/src/infrastructure/daemon/workers/storage/PhabricatorWorkerTrigger.php --- a/src/infrastructure/daemon/workers/storage/PhabricatorWorkerTrigger.php +++ b/src/infrastructure/daemon/workers/storage/PhabricatorWorkerTrigger.php @@ -67,9 +67,10 @@ * is changed (usually because of an application edit). The `$is_reschedule` * parameter distinguishes between these cases. * - * @param int|null Epoch of the most recent successful event execution. - * @param bool `true` if we're trying to reschedule the event after - * execution; `false` if this is in response to a trigger update. + * @param int|null $last_epoch Epoch of the most recent successful event + * execution. + * @param bool $is_reschedule `true` if we're trying to reschedule the event + * after execution; `false` if this is in response to a trigger update. * @return int|null Return an epoch to schedule the next event execution, * or `null` to stop the event from executing again. */ @@ -81,10 +82,10 @@ /** * Execute the event. * - * @param int|null Epoch of previous execution, or null if this is the first - * execution. - * @param int Scheduled epoch of this execution. This may not be the same - * as the current time. + * @param int|null $last_event Epoch of previous execution, or null if this + * is the first execution. + * @param int $this_event Scheduled epoch of this execution. This may not be + * the same as the current time. * @return void */ public function executeTrigger($last_event, $this_event) { 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 @@ -21,7 +21,7 @@ /** * Set the name to identify the old file with. Primarily cosmetic. * - * @param string Old file name. + * @param string $old_name Old file name. * @return this * @task config */ @@ -34,7 +34,7 @@ /** * Set the name to identify the new file with. Primarily cosmetic. * - * @param string New file name. + * @param string $new_name New file name. * @return this * @task config */ @@ -62,8 +62,8 @@ * @{method:generateChangesetFromFileContent}, but may be useful if you need * to use a custom parser configuration, as with Diffusion. * - * @param string Entire previous file content. - * @param string Entire current file content. + * @param string $old Entire previous file content. + * @param string $new Entire current file content. * @return string Raw diff between the two files. * @task diff */ @@ -132,8 +132,8 @@ * principally useful because you can feed the output to * @{class:DifferentialChangesetParser} in order to render it. * - * @param string Entire previous file content. - * @param string Entire current file content. + * @param string $old Entire previous file content. + * @param string $new Entire current file content. * @return @{class:DifferentialChangeset} Synthetic changeset. * @task diff */ 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 @@ -45,10 +45,10 @@ * - `inverse_data` Optional, data to write on the inverse edge. If not * provided, `data` will be written. * - * @param phid Source object PHID. - * @param const Edge type constant. - * @param phid Destination object PHID. - * @param map Options map (see documentation). + * @param phid $src Source object PHID. + * @param const $type Edge type constant. + * @param phid $dst Destination object PHID. + * @param map? $options Options map (see documentation). * @return this * * @task edit @@ -67,9 +67,9 @@ * will be ignored. Edges are added after edges are removed, so the effect of * a remove plus an add is to overwrite. * - * @param phid Source object PHID. - * @param const Edge type constant. - * @param phid Destination object PHID. + * @param phid $src Source object PHID. + * @param const $type Edge type constant. + * @param phid $dst Destination object PHID. * @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 @@ -40,7 +40,7 @@ * Find edges originating at one or more source PHIDs. You MUST provide this * to execute an edge query. * - * @param list List of source PHIDs. + * @param list $source_phids List of source PHIDs. * @return this * * @task config @@ -61,7 +61,7 @@ /** * Find edges terminating at one or more destination PHIDs. * - * @param list List of destination PHIDs. + * @param list $dest_phids List of destination PHIDs. * @return this * */ @@ -74,7 +74,7 @@ /** * Find edges of specific types. * - * @param list List of PhabricatorEdgeConfig type constants. + * @param list $types List of PhabricatorEdgeConfig type constants. * @return this * * @task config @@ -88,7 +88,7 @@ /** * Configure the order edge results are returned in. * - * @param const Order constant. + * @param const $order Order constant. * @return this * * @task config @@ -102,7 +102,7 @@ /** * When loading edges, also load edge data. * - * @param bool True to load edge data. + * @param bool $need True to load edge data. * @return this * * @task config @@ -121,8 +121,8 @@ * edge type. Equivalent to building a full query, but simplifies a common * use case. * - * @param phid Source PHID. - * @param const Edge type. + * @param phid $src_phid Source PHID. + * @param const $edge_type Edge type. * @return list List of destination PHIDs. */ public static function loadDestinationPHIDs($src_phid, $edge_type) { @@ -139,9 +139,9 @@ * if the edge does not exist or does not have metadata. Builds * and immediately executes a full query. * - * @param phid Source PHID. - * @param const Edge type. - * @param phid Destination PHID. + * @param phid $src_phid Source PHID. + * @param const $edge_type Edge type. + * @param phid $dest_phid Destination PHID. * @return wild Edge annotation (or null). */ public static function loadSingleEdgeData($src_phid, $edge_type, $dest_phid) { @@ -256,8 +256,8 @@ * $object->attachHandles(array_select_keys($handles, $dst_phids)); * } * - * @param list? List of PHIDs to select, or empty to select all. - * @param list? List of edge types to select, or empty to select all. + * @param list? $src_phids List of PHIDs to select, or empty to select all. + * @param list? $types List of edge types to select, or empty to select all. * @return list List of matching destination PHIDs. */ public function getDestinationPHIDs( diff --git a/src/infrastructure/env/PhabricatorEnv.php b/src/infrastructure/env/PhabricatorEnv.php --- a/src/infrastructure/env/PhabricatorEnv.php +++ b/src/infrastructure/env/PhabricatorEnv.php @@ -638,7 +638,7 @@ * NOTE: This method is generally intended to reject URIs which it may be * unsafe to put in an "href" link attribute. * - * @param string URI to test. + * @param string $uri URI to test. * @return bool True if the URI identifies a web resource. * @task uri */ @@ -654,7 +654,7 @@ * NOTE: This method is generally intended to reject URIs which it may be * unsafe to issue a "Location:" redirect to. * - * @param string URI to test. + * @param string $uri URI to test. * @return bool True if the URI identifies a local page. * @task uri */ @@ -696,7 +696,7 @@ /** * Detect if a URI identifies some valid linkable remote resource. * - * @param string URI to test. + * @param string $uri URI to test. * @return bool True if a URI identifies a remote resource with an allowed * protocol. * @task uri @@ -718,7 +718,7 @@ * A valid linkable remote resource can be safely linked or redirected to. * This is primarily a protocol whitelist check. * - * @param string URI to test. + * @param string $raw_uri URI to test. * @return void * @task uri */ @@ -758,8 +758,8 @@ /** * Detect if a URI identifies a valid fetchable remote resource. * - * @param string URI to test. - * @param list Allowed protocols. + * @param string $uri URI to test. + * @param list $protocols Allowed protocols. * @return bool True if the URI is a valid fetchable remote resource. * @task uri */ @@ -781,8 +781,8 @@ * originating on this server. This is a primarily an address check against * the outbound address blacklist. * - * @param string URI to test. - * @param list Allowed protocols. + * @param string $raw_uri URI to test. + * @param list $protocols Allowed protocols. * @return pair Pre-resolved URI and domain. * @task uri */ @@ -853,7 +853,7 @@ /** * Determine if an IP address is in the outbound address blacklist. * - * @param string IP address. + * @param string $address IP address. * @return bool True if the address is blacklisted. */ public static function isBlacklistedOutboundAddress($address) { 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 @@ -17,8 +17,8 @@ /** * Override a configuration key in this scope, setting it to a new value. * - * @param string Key to override. - * @param wild New value. + * @param string $key Key to override. + * @param wild $value New value. * @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 @@ -56,10 +56,11 @@ * Convenience method for pushing a single object through the markup * pipeline. * - * @param PhabricatorMarkupInterface The object to render. - * @param string The field to render. - * @param PhabricatorUser User viewing the markup. - * @param object A context object for policy checks + * @param PhabricatorMarkupInterface $object The object to render. + * @param string $field The field to render. + * @param PhabricatorUser $viewer User viewing the markup. + * @param object? $context_object A context object for + * policy checks. * @return string Marked up output. * @task markup */ @@ -81,8 +82,8 @@ * Queue an object for markup generation when @{method:process} is * called. You can retrieve the output later with @{method:getOutput}. * - * @param PhabricatorMarkupInterface The object to render. - * @param string The field to render. + * @param PhabricatorMarkupInterface $object The object to render. + * @param string $field The field to render. * @return this * @task markup */ @@ -175,8 +176,8 @@ * @{method:addObject}. Before you can call this method, you must call * @{method:process}. * - * @param PhabricatorMarkupInterface The object to retrieve. - * @param string The field to retrieve. + * @param PhabricatorMarkupInterface $object The object to retrieve. + * @param string $field The field to retrieve. * @return string Processed output. * @task markup */ @@ -191,10 +192,11 @@ /** * Retrieve engine metadata for a given field. * - * @param PhabricatorMarkupInterface The object to retrieve. - * @param string The field to retrieve. - * @param string The engine metadata field to retrieve. - * @param wild Optional default value. + * @param PhabricatorMarkupInterface $object The object to retrieve. + * @param string $field The field to retrieve. + * @param string $metadata_key The engine metadata field + * to retrieve. + * @param wild? $default Optional default value. * @task markup */ public function getEngineMetadata( @@ -316,7 +318,7 @@ /** * Set the viewing user. Used to implement object permissions. * - * @param PhabricatorUser The viewing user. + * @param PhabricatorUser $viewer The viewing user. * @return this * @task markup */ @@ -328,7 +330,7 @@ /** * Set the context object. Used to implement object permissions. * - * @param The object in which context this remarkup is used. + * @param $object The object in which context this remarkup is used. * @return this * @task markup */ @@ -670,7 +672,7 @@ * * TODO: We could do a better job of this. * - * @param string Remarkup corpus to summarize. + * @param string $corpus Remarkup corpus to summarize. * @return string Summarized corpus. */ public static function summarize($corpus) { diff --git a/src/infrastructure/markup/PhabricatorMarkupInterface.php b/src/infrastructure/markup/PhabricatorMarkupInterface.php --- a/src/infrastructure/markup/PhabricatorMarkupInterface.php +++ b/src/infrastructure/markup/PhabricatorMarkupInterface.php @@ -28,7 +28,7 @@ * * "{$phid}:{$field}" * - * @param string Field name. + * @param string $field Field name. * @return string Cache key up to 125 characters. * * @task markup @@ -39,7 +39,7 @@ /** * Build the engine the field should use. * - * @param string Field name. + * @param string $field Field name. * @return PhutilRemarkupEngine Markup engine to use. * @task markup */ @@ -49,7 +49,7 @@ /** * Return the contents of the specified field. * - * @param string Field name. + * @param string $field Field name. * @return string The raw markup contained in the field. * @task markup */ @@ -60,9 +60,9 @@ * Callback for final postprocessing of output. Normally, you can return * the output unmodified. * - * @param string Field name. - * @param string The finalized output of the engine. - * @param string The engine which generated the output. + * @param string $field Field name. + * @param string $output The finalized output of the engine. + * @param string $engine The engine which generated the output. * @return string Final output. * @task markup */ 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 @@ -8,8 +8,8 @@ * documentation for specific rules and blocks for what options are available * for configuration. * - * @param string Key to set in the configuration dictionary. - * @param string Value to set. + * @param string $key Key to set in the configuration dictionary. + * @param string $value Value to set. * @return this */ abstract public function setConfig($key, $value); @@ -21,8 +21,9 @@ * text; consult the documentation for specific rules and blocks to see what * metadata may be available in your configuration. * - * @param string Key to retrieve from metadata. - * @param mixed Default value to return if the key is not available. + * @param string $key Key to retrieve from metadata. + * @param mixed? $default Default value to return if the key is not + * available. * @return mixed Metadata property, or default value. */ abstract public function getTextMetadata($key, $default = null); diff --git a/src/infrastructure/markup/blockrule/PhutilRemarkupCodeBlockRule.php b/src/infrastructure/markup/blockrule/PhutilRemarkupCodeBlockRule.php --- a/src/infrastructure/markup/blockrule/PhutilRemarkupCodeBlockRule.php +++ b/src/infrastructure/markup/blockrule/PhutilRemarkupCodeBlockRule.php @@ -350,7 +350,7 @@ /** * Get the extension from a filename. - * @param string "/path/to/something.name" + * @param string $name "/path/to/something.name" * @return null|string ".name" */ private function guessFilenameExtension($name) { diff --git a/src/infrastructure/markup/markuprule/PhutilRemarkupRule.php b/src/infrastructure/markup/markuprule/PhutilRemarkupRule.php --- a/src/infrastructure/markup/markuprule/PhutilRemarkupRule.php +++ b/src/infrastructure/markup/markuprule/PhutilRemarkupRule.php @@ -20,9 +20,10 @@ /** * Check input whether to apply RemarkupRule. If true, apply formatting. - * @param string|PhutilSafeHTML String to check and potentially format. + * @param string|PhutilSafeHTML $text String to check and potentially + * format. * @return string|PhutilSafeHTML Unchanged input if no match, or input after - * matching the formatting rule and applying the formatting. + * matching the formatting rule and applying the formatting. */ abstract public function apply($text); @@ -59,9 +60,9 @@ * This method acts as @{function:phutil_tag}, but checks attributes before * using them. * - * @param string Tag name. - * @param dict Tag attributes. - * @param wild Tag content. + * @param string $name Tag name. + * @param dict $attrs Tag attributes. + * @param wild? $content Tag content. * @return PhutilSafeHTML Tag object. */ protected function newTag($name, array $attrs, $content = null) { @@ -85,7 +86,7 @@ * Normally, you can call @{method:newTag} rather than calling this method * directly. @{method:newTag} will check attributes for you. * - * @param wild Ostensibly flat text. + * @param wild $text Ostensibly flat text. * @return string Flat text. */ protected function assertFlatText($text) { @@ -104,7 +105,7 @@ /** * Check whether text is flat (contains no replacement tokens) or not. * - * @param wild Ostensibly flat text. + * @param wild $text Ostensibly flat text. * @return bool True if the text is flat. */ protected function isFlatText($text) { diff --git a/src/infrastructure/markup/render.php b/src/infrastructure/markup/render.php --- a/src/infrastructure/markup/render.php +++ b/src/infrastructure/markup/render.php @@ -16,9 +16,9 @@ * trusted blindly, and not escaped. You should not pass user data in these * parameters. * - * @param string The name of the tag, like `a` or `div`. - * @param map A map of tag attributes. - * @param wild Content to put in the tag. + * @param string $tag The name of the tag, like `a` or `div`. + * @param map? $attributes A map of tag attributes. + * @param wild? $content Content to put in the tag. * @return PhutilSafeHTML Tag object. */ function phutil_tag($tag, array $attributes = array(), $content = null) { diff --git a/src/infrastructure/markup/rule/PhabricatorObjectRemarkupRule.php b/src/infrastructure/markup/rule/PhabricatorObjectRemarkupRule.php --- a/src/infrastructure/markup/rule/PhabricatorObjectRemarkupRule.php +++ b/src/infrastructure/markup/rule/PhabricatorObjectRemarkupRule.php @@ -258,7 +258,7 @@ * This is intended to make it easy to write unit tests for object remarkup * rules. Production code is not normally expected to call this method. * - * @param string Text to match rules against. + * @param string $text Text to match rules against. * @return wild Matches, suitable for writing unit tests against. */ public function extractReferences($text) { diff --git a/src/infrastructure/parser/PhutilURIHelper.php b/src/infrastructure/parser/PhutilURIHelper.php --- a/src/infrastructure/parser/PhutilURIHelper.php +++ b/src/infrastructure/parser/PhutilURIHelper.php @@ -19,7 +19,7 @@ private $phutilUri; /** - * @param string|PhutilURI + * @param string|PhutilURI $uri */ public function __construct($uri) { 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 @@ -718,9 +718,9 @@ * * This method will then return a composable clause for inclusion in WHERE. * - * @param AphrontDatabaseConnection Connection query will execute on. - * @param list Column description dictionaries. - * @param map Additional construction options. + * @param AphrontDatabaseConnection $conn Connection query will execute on. + * @param list $columns Column description dictionaries. + * @param map $options Additional construction options. * @return string Query clause. * @task paging */ @@ -884,7 +884,7 @@ * across individual orderable columns. This offers greater control but is * also more involved. * - * @param string Key of a builtin order supported by this query. + * @param string $order Key of a builtin order supported by this query. * @return this * @task order */ @@ -917,7 +917,8 @@ * This is a high-level method which works alongside @{method:setOrder}. For * lower-level control over order vectors, use @{method:setOrderVector}. * - * @param PhabricatorQueryOrderVector|list List of order keys. + * @param PhabricatorQueryOrderVector|list $vector List of order + * keys. * @return this * @task order */ @@ -1041,7 +1042,8 @@ * To set an order vector, specify a list of order keys as provided by * @{method:getOrderableColumns}. * - * @param PhabricatorQueryOrderVector|list List of order keys. + * @param PhabricatorQueryOrderVector|list $vector List of order + * keys. * @return this * @task order */ @@ -1353,8 +1355,9 @@ * - Find users with shirt sizes "X" or "XL". * - Find shoes with size "13". * - * @param PhabricatorCustomFieldIndexStorage Table where the index is stored. - * @param string|list One or more values to filter by. + * @param PhabricatorCustomFieldIndexStorage $index Table where the index is + * stored. + * @param string|list $value One or more values to filter by. * @return this * @task appsearch */ @@ -1403,9 +1406,10 @@ * `5` will match fields with values `3`, `4`, or `5`. Providing `null` for * either end of the range will leave that end of the constraint open. * - * @param PhabricatorCustomFieldIndexStorage Table where the index is stored. - * @param int|null Minimum permissible value, inclusive. - * @param int|null Maximum permissible value, inclusive. + * @param PhabricatorCustomFieldIndexStorage $index Table where the index is + * stored. + * @param int|null $min Minimum permissible value, inclusive. + * @param int|null $max Maximum permissible value, inclusive. * @return this * @task appsearch */ @@ -1449,7 +1453,7 @@ * See @{method:getPrimaryTableAlias} if the column needs to be qualified with * a table alias. * - * @param AphrontDatabaseConnection Connection executing queries. + * @param AphrontDatabaseConnection $conn Connection executing queries. * @return PhutilQueryString Column name. * @task appsearch */ @@ -1509,7 +1513,7 @@ /** * Construct a GROUP BY clause appropriate for ApplicationSearch constraints. * - * @param AphrontDatabaseConnection Connection executing the query. + * @param AphrontDatabaseConnection $conn Connection executing the query. * @return string Group clause. * @task appsearch */ @@ -1531,7 +1535,7 @@ * Construct a JOIN clause appropriate for applying ApplicationSearch * constraints. * - * @param AphrontDatabaseConnection Connection executing the query. + * @param AphrontDatabaseConnection $conn Connection executing the query. * @return string Join clause. * @task appsearch */ @@ -1653,7 +1657,7 @@ * Construct a WHERE clause appropriate for applying ApplicationSearch * constraints. * - * @param AphrontDatabaseConnection Connection executing the query. + * @param AphrontDatabaseConnection $conn Connection executing the query. * @return list Where clause parts. * @task appsearch */ @@ -2583,9 +2587,9 @@ * Convenience method for specifying edge logic constraints with a list of * PHIDs. * - * @param const Edge constant. - * @param const Constraint operator. - * @param list List of PHIDs. + * @param const $edge_type Edge constant. + * @param const $operator Constraint operator. + * @param list $phids List of PHIDs. * @return this * @task edgelogic */ @@ -3091,7 +3095,7 @@ * Queries are always constrained to include only results from spaces the * viewer has access to. * - * @param list + * @param list $space_phids * @task spaces */ public function withSpacePHIDs(array $space_phids) { @@ -3135,7 +3139,7 @@ * viewer has access to see with any explicit constraint on spaces added by * @{method:withSpacePHIDs}. * - * @param AphrontDatabaseConnection Database connection. + * @param AphrontDatabaseConnection $conn Database connection. * @return string Part of a WHERE clause. * @task spaces */ 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 @@ -57,7 +57,7 @@ * according to the viewer's capabilities. You must set a viewer to execute * a policy query. * - * @param PhabricatorUser The viewing user. + * @param PhabricatorUser $viewer The viewing user. * @return this * @task config */ @@ -473,8 +473,8 @@ * automatically populated as a side effect of objects surviving policy * filtering. * - * @param map Objects to add to the query - * workspace. + * @param map $objects Objects to add to + * the query workspace. * @return this * @task workspace */ @@ -506,7 +506,7 @@ * searches both the current query's workspace and the workspaces of parent * queries. * - * @param list List of PHIDs to retrieve. + * @param list $phids List of PHIDs to retrieve. * @return this * @task workspace */ @@ -536,7 +536,7 @@ * PHIDs which are "in flight" are actively being queried for. Using this * list can prevent infinite query loops by aborting queries which cycle. * - * @param list List of PHIDs which are now in flight. + * @param list $phids List of PHIDs which are now in flight. * @return this */ public function putPHIDsInFlight(array $phids) { @@ -606,7 +606,7 @@ * return new results. Generally, you should adjust a cursor position based * on the provided result page. * - * @param list The current page of results. + * @param list $page The current page of results. * @return void * @task policyimpl */ @@ -627,7 +627,7 @@ * This method will only be called if data is available. Implementations * do not need to handle the case of no results specially. * - * @param list Results from `loadPage()`. + * @param list $page Results from `loadPage()`. * @return list Objects for policy filtering. * @task policyimpl */ @@ -650,7 +650,7 @@ * This method will only be called if data is available. Implementations do * not need to handle the case of no results specially. * - * @param list Results from @{method:willFilterPage()}. + * @param list $page Results from @{method:willFilterPage()}. * @return list Objects after additional * non-policy processing. */ @@ -665,8 +665,8 @@ * filtered for policy reasons. The query should remove them from any cached * or partial result sets. * - * @param list List of objects that should not be returned by alternate - * result mechanisms. + * @param list $results List of objects that should not be returned by + * alternate result mechanisms. * @return void * @task policyimpl */ @@ -680,7 +680,7 @@ * used by @{class:PhabricatorCursorPagedPolicyAwareQuery} to reverse results * that are queried during reverse paging. * - * @param list Query results. + * @param list $results Query results. * @return list Final results. * @task policyimpl */ 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 @@ -221,7 +221,8 @@ * return a new connection. Lisk handles connection caching and management; * do not perform caching deeper in the stack. * - * @param string Mode, either 'r' (reading) or 'w' (reading and writing). + * @param string $mode Mode, either 'r' (reading) or 'w' (reading and + * writing). * @return AphrontDatabaseConnection New database connection. * @task conn */ @@ -245,7 +246,7 @@ /** * Get an existing, cached connection for this object. * - * @param mode Connection mode. + * @param mode $mode Connection mode. * @return AphrontDatabaseConnection|null Connection, if it exists in cache. * @task conn */ @@ -261,8 +262,9 @@ /** * Store a connection in the connection cache. * - * @param mode Connection mode. - * @param AphrontDatabaseConnection Connection to cache. + * @param mode $mode Connection mode. + * @param AphrontDatabaseConnection $connection Connection to cache. + * @param bool? $force_unique * @return this * @task conn */ @@ -291,7 +293,8 @@ * This overrides all connection management and forces the object to use * a specific connection when interacting with the database. * - * @param AphrontDatabaseConnection Connection to force this object to use. + * @param AphrontDatabaseConnection $connection Connection to force this + * object to use. * @task conn */ public function setForcedConnection(AphrontDatabaseConnection $connection) { @@ -397,12 +400,12 @@ /** - * Determine the setting of a configuration option for this class of objects. + * Determine the setting of a configuration option for this class of objects. * - * @param const Option name, one of the CONFIG_* constants. - * @return mixed Option value, if configured (null if unavailable). + * @param const $option_name Option name, one of the CONFIG_* constants. + * @return mixed Option value, if configured (null if unavailable). * - * @task config + * @task config */ public function getConfigOption($option_name) { $options = $this->getLiskMetadata('config'); @@ -426,7 +429,7 @@ * * $dog = id(new Dog())->load($dog_id); * - * @param int Numeric ID identifying the object to load. + * @param int $id Numeric ID identifying the object to load. * @return obj|null Identified object, or null if it does not exist. * * @task load @@ -468,7 +471,7 @@ * * The pattern and arguments are as per queryfx(). * - * @param string queryfx()-style SQL WHERE clause. + * @param string $pattern queryfx()-style SQL WHERE clause. * @param ... Zero or more conversions. * @return dict Dictionary of matching objects, keyed on ID. * @@ -489,7 +492,7 @@ * query. See loadAllWhere(). This method is similar, but returns a single * result instead of a list. * - * @param string queryfx()-style SQL WHERE clause. + * @param string $pattern queryfx()-style SQL WHERE clause. * @param ... Zero or more conversions. * @return obj|null Matching object, or null if no object matches. * @@ -574,8 +577,8 @@ * convenient to pull data from elsewhere directly (e.g., a complicated * join via @{method:queryData}) and then load from an array representation. * - * @param dict Dictionary of properties, which should be equivalent to - * selecting a row from the table or calling + * @param dict $row Dictionary of properties, which should be equivalent + * to selecting a row from the table or calling * @{method:getProperties}. * @return this * @@ -649,7 +652,7 @@ * * This is a lot messier than @{method:loadAllWhere}, but more flexible. * - * @param list List of property dictionaries. + * @param list $rows List of property dictionaries. * @return dict List of constructed objects, keyed on ID. * * @task load @@ -690,7 +693,7 @@ * Set unique ID identifying this object. You normally don't need to call this * method unless with `IDS_MANUAL`. * - * @param mixed Unique ID. + * @param mixed $id Unique ID. * @return this * @task save */ @@ -723,7 +726,7 @@ /** * Test if a property exists. * - * @param string Property name. + * @param string $property Property name. * @return bool True if the property exists. * @task info */ @@ -798,9 +801,9 @@ /** * Get or build the database connection for this object. * - * @param string 'r' for read, 'w' for read/write. - * @param bool True to force a new connection. The connection will not - * be retrieved from or saved into the connection cache. + * @param string $mode 'r' for read, 'w' for read/write. + * @param bool? $force_new True to force a new connection. The connection + * will not be retrieved from or saved into the connection cache. * @return AphrontDatabaseConnection Lisk connection object. * * @task info @@ -1038,7 +1041,8 @@ /** * Internal implementation of INSERT and REPLACE. * - * @param const Either "INSERT" or "REPLACE", to force the desired mode. + * @param const $mode Either "INSERT" or "REPLACE", to force the desired + * mode. * @return this * * @task save @@ -1272,7 +1276,7 @@ * Reads the value from a field. Override this method for custom behavior * of @{method:getField} instead of overriding getField directly. * - * @param string Canonical field name + * @param string $field Canonical field name * @return mixed Value of the field * * @task hook @@ -1288,8 +1292,8 @@ * Writes a value to a field. Override this method for custom behavior of * setField($value) instead of overriding setField directly. * - * @param string Canonical field name - * @param mixed Value to write + * @param string $field Canonical field name + * @param mixed $value Value to write * * @task hook */ @@ -1476,7 +1480,8 @@ * Long-running processes can use this method to clean up connections which * have not been used recently. * - * @param int Close connections with no activity for this many seconds. + * @param int $idle_window Close connections with no activity for this many + * seconds. * @return void */ public static function closeInactiveConnections($idle_window) { @@ -1576,8 +1581,8 @@ /** * Black magic. Builds implied get*() and set*() for all properties. * - * @param string Method name. - * @param list Argument vector. + * @param string $method Method name. + * @param list $args Argument vector. * @return mixed get*() methods return the property value. set*() methods * return $this. * @task util @@ -1652,8 +1657,10 @@ /** * Increments a named counter and returns the next value. * - * @param AphrontDatabaseConnection Database where the counter resides. - * @param string Counter name to create or increment. + * @param AphrontDatabaseConnection $conn_w Database where the counter + * resides. + * @param string $counter_name Counter name to create + * or increment. * @return int Next counter value. * * @task util @@ -1686,8 +1693,9 @@ /** * Returns the current value of a named counter. * - * @param AphrontDatabaseConnection Database where the counter resides. - * @param string Counter name to read. + * @param AphrontDatabaseConnection $conn_r Database where the counter + * resides. + * @param string $counter_name Counter name to read. * @return int|null Current value, or `null` if the counter does not exist. * * @task util @@ -1714,8 +1722,10 @@ * * If the counter does not exist, it is created. * - * @param AphrontDatabaseConnection Database where the counter resides. - * @param string Counter name to create or overwrite. + * @param AphrontDatabaseConnection $conn_w Database where the counter + * resides. + * @param string $counter_name Counter name to create or overwrite. + * @param int $counter_value * @return void * * @task util 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 @@ -88,7 +88,7 @@ * (somewhat arbitrarily). In most cases this is fine, but this method can * be used to lock on a specific connection. * - * @param AphrontDatabaseConnection + * @param AphrontDatabaseConnection $conn * @return this */ public function setExternalConnection(AphrontDatabaseConnection $conn) { diff --git a/src/infrastructure/util/PhabricatorHash.php b/src/infrastructure/util/PhabricatorHash.php --- a/src/infrastructure/util/PhabricatorHash.php +++ b/src/infrastructure/util/PhabricatorHash.php @@ -11,7 +11,8 @@ * Because a SHA1 collision is now known, this method should be considered * weak. Callers should prefer @{method:digestWithNamedKey}. * - * @param string Input string. + * @param string $string Input string. + * @param string? $key * @return string 32-byte hexadecimal SHA1+HMAC hash. */ public static function weakDigest($string, $key = null) { @@ -38,7 +39,7 @@ * This method emphasizes compactness, and should not be used for security * related hashing (for general purpose hashing, see @{method:digest}). * - * @param string Input string. + * @param string $string Input string. * @return string 12-byte, case-sensitive, mostly-alphanumeric hash of * the string. */ @@ -72,7 +73,7 @@ * much stuff we're breaking by switching to it. For additional discussion, * see T13045. * - * @param string Input string. + * @param string $string Input string. * @return string 12-byte, case-sensitive, purely-alphanumeric hash of * the string. */ @@ -149,8 +150,8 @@ * maintaining a high degree of collision resistance and a moderate degree * of human readability. * - * @param string The string to shorten. - * @param int Maximum length of the result. + * @param string $string The string to shorten. + * @param int $length Maximum length of the result. * @return string String shortened in a collision-resistant way. */ public static function digestToLength($string, $length) { diff --git a/src/infrastructure/util/password/PhabricatorPasswordHasher.php b/src/infrastructure/util/password/PhabricatorPasswordHasher.php --- a/src/infrastructure/util/password/PhabricatorPasswordHasher.php +++ b/src/infrastructure/util/password/PhabricatorPasswordHasher.php @@ -101,7 +101,7 @@ /** * Produce a password hash. * - * @param PhutilOpaqueEnvelope Text to be hashed. + * @param PhutilOpaqueEnvelope $envelope Text to be hashed. * @return PhutilOpaqueEnvelope Hashed text. * @task hasher */ @@ -114,8 +114,8 @@ * The default implementation checks for equality; if a hasher embeds salt in * hashes it should override this method and perform a salt-aware comparison. * - * @param PhutilOpaqueEnvelope Password to compare. - * @param PhutilOpaqueEnvelope Bare password hash. + * @param PhutilOpaqueEnvelope $password Password to compare. + * @param PhutilOpaqueEnvelope $hash Bare password hash. * @return bool True if the passwords match. * @task hasher */ @@ -137,7 +137,7 @@ * have (for example) an internal cost function may be able to upgrade an * existing hash to a stronger one with a higher cost. * - * @param PhutilOpaqueEnvelope Bare hash. + * @param PhutilOpaqueEnvelope $hash Bare hash. * @return bool True if the hash can be upgraded without * changing the algorithm (for example, to a * higher cost). @@ -154,7 +154,7 @@ /** * Get the hash of a password for storage. * - * @param PhutilOpaqueEnvelope Password text. + * @param PhutilOpaqueEnvelope $envelope Password text. * @return PhutilOpaqueEnvelope Hashed text. * @task hashing */ @@ -349,7 +349,7 @@ /** * Generate a new hash for a password, using the best available hasher. * - * @param PhutilOpaqueEnvelope Password to hash. + * @param PhutilOpaqueEnvelope $password Password to hash. * @return PhutilOpaqueEnvelope Hashed password, using best available * hasher. * @task hashing @@ -364,8 +364,8 @@ /** * Compare a password to a stored hash. * - * @param PhutilOpaqueEnvelope Password to compare. - * @param PhutilOpaqueEnvelope Stored password hash. + * @param PhutilOpaqueEnvelope $password Password to compare. + * @param PhutilOpaqueEnvelope $hash Stored password hash. * @return bool True if the passwords match. * @task hashing */ @@ -383,7 +383,7 @@ /** * Get the human-readable algorithm name for a given hash. * - * @param PhutilOpaqueEnvelope Storage hash. + * @param PhutilOpaqueEnvelope $hash Storage hash. * @return string Human-readable algorithm name. */ public static function getCurrentAlgorithmName(PhutilOpaqueEnvelope $hash) { diff --git a/src/view/AphrontView.php b/src/view/AphrontView.php --- a/src/view/AphrontView.php +++ b/src/view/AphrontView.php @@ -16,7 +16,7 @@ /** * Set the user viewing this element. * - * @param PhabricatorUser Viewing user. + * @param PhabricatorUser $viewer Viewing user. * @return this */ public function setViewer(PhabricatorUser $viewer) { @@ -100,7 +100,7 @@ * This method will only work if the view supports children, which is * determined by @{method:canAppendChild}. * - * @param wild Something renderable. + * @param wild $child Something renderable. * @return this */ final public function appendChild($child) { @@ -152,7 +152,7 @@ * NOTE: Because View children are not rendered, a View which renders down * to nothing will not be reduced by this method. * - * @param list Renderable children. + * @param list $children Renderable children. * @return list Reduced list of children. * @task children */ diff --git a/src/view/control/AphrontTableView.php b/src/view/control/AphrontTableView.php --- a/src/view/control/AphrontTableView.php +++ b/src/view/control/AphrontTableView.php @@ -105,7 +105,7 @@ * * list($sort, $reverse) = AphrontTableView::parseSortParam($sort_param); * - * @param string Sort request parameter. + * @param string $sort Sort request parameter. * @return pair Sort value, sort direction. */ public static function parseSort($sort) { 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 @@ -120,7 +120,7 @@ * controls. It will propagate some information from the form to the * control to simplify rendering. * - * @param AphrontFormControl Control to append. + * @param AphrontFormControl $control Control to append. * @return this */ public function appendControl(AphrontFormControl $control) { diff --git a/src/view/form/control/AphrontFormControl.php b/src/view/form/control/AphrontFormControl.php --- a/src/view/form/control/AphrontFormControl.php +++ b/src/view/form/control/AphrontFormControl.php @@ -59,7 +59,7 @@ /** * Set the Caption * The Caption shows a tip usually nearby the related input field. - * @param string|PhutilSafeHTML|null + * @param string|PhutilSafeHTML|null $caption * @return self */ public function setCaption($caption) { diff --git a/src/view/page/PhabricatorStandardPageView.php b/src/view/page/PhabricatorStandardPageView.php --- a/src/view/page/PhabricatorStandardPageView.php +++ b/src/view/page/PhabricatorStandardPageView.php @@ -379,7 +379,7 @@ /** * Insert a HTML element into of the page to render. * - * @param PhutilSafeHTML HTML header to add + * @param PhutilSafeHTML $html HTML header to add */ public function addHeadItem($html) { if ($html instanceof PhutilSafeHTML) { diff --git a/src/view/phui/PHUIBoxView.php b/src/view/phui/PHUIBoxView.php --- a/src/view/phui/PHUIBoxView.php +++ b/src/view/phui/PHUIBoxView.php @@ -35,7 +35,8 @@ * Render PHUIBoxView as a
instead of a
HTML tag. * To be used for collapse/expand in combination with PHUIHeaderView. * - * @param bool True to wrap in instead of
HTML tag. + * @param bool $collapsible True to wrap in instead of
HTML + * tag. */ public function setCollapsible($collapsible) { $this->collapsible = $collapsible; 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 @@ -23,7 +23,7 @@ * Make this crumb always visible, even on devices where it would normally * be hidden. * - * @param bool True to make the crumb always visible. + * @param bool $always_visible True to make the crumb always visible. * @return this */ public function setAlwaysVisible($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 @@ -15,8 +15,8 @@ * Convenience method for adding a simple crumb with just text, or text and * a link. * - * @param string Text of the crumb. - * @param string? Optional href for the crumb. + * @param string $text Text of the crumb. + * @param string? $href Optional href for the crumb. * @return this */ public function addTextCrumb($text, $href = null) { diff --git a/src/view/phui/PHUIHeaderView.php b/src/view/phui/PHUIHeaderView.php --- a/src/view/phui/PHUIHeaderView.php +++ b/src/view/phui/PHUIHeaderView.php @@ -95,7 +95,8 @@ * Render PHUIHeaderView as a instead of a
HTML tag. * To be used for collapse/expand in combination with PHUIBoxView. * - * @param bool True to wrap in instead of
HTML tag. + * @param bool $collapsible True to wrap in instead of
HTML + * tag. */ public function setCollapsible($collapsible) { $this->collapsible = $collapsible; diff --git a/src/view/phui/PHUIPagerView.php b/src/view/phui/PHUIPagerView.php --- a/src/view/phui/PHUIPagerView.php +++ b/src/view/phui/PHUIPagerView.php @@ -95,7 +95,7 @@ * $pager->getPageSize() + 1); * $results = $pager->sliceResults($results); * - * @param list Result array. + * @param list $results Result array. * @return list One page of results. */ public function sliceResults(array $results) { diff --git a/src/view/viewutils.php b/src/view/viewutils.php --- a/src/view/viewutils.php +++ b/src/view/viewutils.php @@ -104,9 +104,9 @@ * @{function:phabricator_date}, @{function:phabricator_time}, or * @{function:phabricator_datetime}. * - * @param int Unix epoch timestamp. - * @param PhabricatorUser User viewing the timestamp. - * @param string Date format, as per DateTime class. + * @param int $epoch Unix epoch timestamp. + * @param PhabricatorUser $user User viewing the timestamp. + * @param string $format Date format, as per DateTime class. * @return string Formatted, local date/time. */ function phabricator_format_local_time($epoch, $user, $format) { diff --git a/support/startup/PhabricatorClientLimit.php b/support/startup/PhabricatorClientLimit.php --- a/support/startup/PhabricatorClientLimit.php +++ b/support/startup/PhabricatorClientLimit.php @@ -170,7 +170,7 @@ /** * Get the APC key for a given bucket. * - * @param int Bucket to get the key for. + * @param int $bucket_id Bucket to get the key for. * @return string APC key for the bucket. */ private function getBucketCacheKey($bucket_id) { @@ -182,9 +182,8 @@ /** * Add points to the rate limit score for some client. * - * @param string Some key which identifies the client making the request. - * @param float The cost for this request; more points pushes them toward - * the limit faster. + * @param float $score The cost for this request; more points pushes them + * toward the limit faster. * @return this */ private function addScore($score) { diff --git a/support/startup/PhabricatorStartup.php b/support/startup/PhabricatorStartup.php --- a/support/startup/PhabricatorStartup.php +++ b/support/startup/PhabricatorStartup.php @@ -112,7 +112,7 @@ /** - * @param float Request start time, from `microtime(true)`. + * @param float $start_time Request start time, from `microtime(true)`. * @task hook */ public static function didStartup($start_time) { @@ -255,7 +255,7 @@ * The limit is implemented with a tick function, so enabling it implies * some accounting overhead. * - * @param int Time limit in seconds. + * @param int $limit Time limit in seconds. * @return void */ public static function setDebugTimeLimit($limit) { @@ -312,11 +312,12 @@ * Fatal the request completely in response to an exception, sending a plain * text message to the client. Calls @{method:didFatal} internally. * - * @param string Brief description of the exception context, like + * @param string $note Brief description of the exception context, like * `"Rendering Exception"`. - * @param Throwable The exception itself. - * @param bool True if it's okay to show the exception's stack trace - * to the user. The trace will always be logged. + * @param Throwable $ex The exception itself. + * @param bool $show_trace True if it's okay to show the exception's + * stack trace to the user. The trace will always be + * logged. * * @task apocalypse */ @@ -342,11 +343,11 @@ /** * Fatal the request completely, sending a plain text message to the client. * - * @param string Plain text message to send to the client. - * @param string Plain text message to send to the error log. If not - * provided, the client message is used. You can pass a more - * detailed message here (e.g., with stack traces) to avoid - * showing it to users. + * @param string $message Plain text message to send to the client. + * @param string? $log_message Plain text message to send to the error log. + * If not provided, the client message is used. You can pass + * a more detailed message here (e.g., with stack traces) to + * avoid showing it to users. * @return exit This method **does not return**. * * @task apocalypse @@ -513,7 +514,7 @@ * Adjustments here primarily impact the environment as seen by subprocesses. * The environment is forwarded explicitly by @{class:ExecFuture}. * - * @param map Input `$_ENV`. + * @param map $env Input `$_ENV`. * @return map Suitable `$_ENV`. * @task validation */ @@ -653,7 +654,7 @@ /** * Add a new client limits. * - * @param PhabricatorClientLimit New limit. + * @param PhabricatorClientLimit $limit New limit. * @return PhabricatorClientLimit The limit. */ public static function addRateLimit(PhabricatorClientLimit $limit) { @@ -698,7 +699,8 @@ /** * Tear down rate limiting and allow limits to score the request. * - * @param map Additional, freeform request state. + * @param map $request_state Additional, freeform request + * state. * @return void * @task ratelimit */ @@ -745,7 +747,7 @@ * time and record it with @{method:recordStartupPhase} after the class is * available. * - * @param string Phase name. + * @param string $phase Phase name. * @task phases */ public static function beginStartupPhase($phase) { @@ -761,8 +763,8 @@ * record a time before the class loads, then hand it over once the class * becomes available. * - * @param string Phase name. - * @param float Phase start time, from `microtime(true)`. + * @param string $phase Phase name. + * @param float $time Phase start time, from `microtime(true)`. * @task phases */ public static function recordStartupPhase($phase, $time) { diff --git a/support/startup/preamble-utils.php b/support/startup/preamble-utils.php --- a/support/startup/preamble-utils.php +++ b/support/startup/preamble-utils.php @@ -4,7 +4,7 @@ * Parse the "X_FORWARDED_FOR" HTTP header to determine the original client * address. * - * @param int Number of devices to trust. + * @param int? $layers Number of devices to trust. * @return void */ function preamble_trust_x_forwarded_for_header($layers = 1) {