diff --git a/src/aphront/AphrontRequest.php b/src/aphront/AphrontRequest.php --- a/src/aphront/AphrontRequest.php +++ b/src/aphront/AphrontRequest.php @@ -714,7 +714,7 @@ * This is primarily useful if you want to ask the user for more input and * then resubmit their request. * - * @return dict Original request parameters. + * @return array Original request parameters. */ public function getPassthroughRequestParameters($include_quicksand = false) { return self::flattenData( @@ -724,7 +724,7 @@ /** * Get request data other than "magic" parameters. * - * @return dict Request data, with magic filtered out. + * @return array Request data, with magic filtered out. */ public function getPassthroughRequestData($include_quicksand = false) { $data = $this->getRequestData(); @@ -748,9 +748,9 @@ * into a list of key-value pairs suitable for submitting via HTTP request * (with arrays flattened). * - * @param dict $data Data to flatten. - * @return dict Flat data suitable for inclusion in an HTTP - * request. + * @param array $data Data to flatten. + * @return array Flat data suitable for inclusion in an HTTP + * request. */ public static function flattenData(array $data) { $result = array(); 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 @@ -387,8 +387,8 @@ /** * Build a controller to respond to the request. * - * @return pair Controller and dictionary of request - * parameters. + * @return pair Controller and dictionary of request + * parameters. * @task routing */ private function buildController() { @@ -512,8 +512,9 @@ * * @param list $maps List of routing maps. * @param string $path Path to route. - * @return pair|null Controller and dictionary of - * request parameters, or null if no paths to route were found. + * @return array>|null Controller + * subclass and dictionary of request parameters, or null if no paths to + * route were found. * @task routing */ private function routePath(array $maps, $path) { 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 @@ -21,6 +21,9 @@ return $this; } + /** + * @return AphrontSite + */ public function getSite() { return $this->site; } @@ -30,6 +33,9 @@ return $this; } + /** + * @return PhabricatorApplication + */ public function getApplication() { return $this->application; } @@ -87,7 +93,8 @@ * @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. + * @return array|string>|null Match details, if path + * matches sub-map. * @task routing */ private function tryRoute($route, $value, $path) { diff --git a/src/aphront/site/AphrontRoutingResult.php b/src/aphront/site/AphrontRoutingResult.php --- a/src/aphront/site/AphrontRoutingResult.php +++ b/src/aphront/site/AphrontRoutingResult.php @@ -21,6 +21,9 @@ return $this; } + /** + * @return AphrontSite + */ public function getSite() { return $this->site; } @@ -30,6 +33,9 @@ return $this; } + /** + * @return PhabricatorApplication + */ public function getApplication() { return $this->application; } @@ -39,6 +45,9 @@ return $this; } + /** + * @return AphrontController + */ public function getController() { return $this->controller; } @@ -48,6 +57,9 @@ return $this; } + /** + * @return array + */ public function getURIData() { return $this->uriData; } 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 @@ -160,7 +160,7 @@ * Authenticate the client making the request to a Phabricator user account. * * @param ConduitAPIRequest $api_request Request being executed. - * @param dict $metadata Request metadata. + * @param array $metadata Dictionary of 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/differential/parser/DifferentialHunkParser.php b/src/applications/differential/parser/DifferentialHunkParser.php --- a/src/applications/differential/parser/DifferentialHunkParser.php +++ b/src/applications/differential/parser/DifferentialHunkParser.php @@ -14,7 +14,8 @@ * datastructure is used to determine when to render "Context not available." * in diffs with multiple hunks. * - * @return dict Map of lines where hunks start, other than line 1. + * @return array Map of lines where hunks start, other than + * line 1. */ public function getHunkStartLines(array $hunks) { assert_instances_of($hunks, 'DifferentialHunk'); 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,8 @@ * construct appropriate @{class:PhabricatorFeedStory} wrappers for each * data row. * - * @param list $rows List of @{class:PhabricatorFeedStoryData} rows - * from the database. + * @param array> $rows List of + * @{class:PhabricatorFeedStoryData} rows from the database. * @param PhabricatorUser $viewer * @return list List of @{class:PhabricatorFeedStory} * objects. 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,8 +96,8 @@ * `PHID-FILE-aaaa` and all transformations of the file with PHID * `PHID-FILE-bbbb`. * - * @param list $specs List of transform specifications, described - * above. + * @param array> $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 @@ -1215,7 +1215,7 @@ * @param PhabricatorUser $user Viewing user. * @param list $builtins List of builtin file * specs. - * @return dict Dictionary of named builtins. + * @return array Dictionary of named builtins. */ public static function loadBuiltins(PhabricatorUser $user, array $builtins) { $builtins = mpull($builtins, null, 'getBuiltinFileKey'); 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 @@ -191,7 +191,7 @@ * @{function:vcsprintf}. * @param string $pattern User-provided pattern string containing * `${variables}`. - * @param dict $variables List of available replacement variables. + * @param array $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/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,8 +26,8 @@ * ), * ) * - * @param string $body Raw mail text body. - * @return dict Parsed body. + * @param string $body Raw mail text body. + * @return array Parsed body. */ public function parseBody($body) { $body = $this->stripTextBody($body); 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 @@ -147,7 +147,7 @@ * To get PHID types a given user has access to, see * @{method:getAllInstalledTypes}. * - * @return dict Map of type constants to types. + * @return array Map of type constants to types. */ final public static function getAllTypes() { return self::newClassMapQuery() @@ -172,7 +172,7 @@ * Get all PHID types of applications installed for a given viewer. * * @param PhabricatorUser $viewer Viewing user. - * @return dict Map of constants to installed + * @return array Map of constants to installed * types. */ public static function getAllInstalledTypes(PhabricatorUser $viewer) { @@ -216,7 +216,7 @@ * Get all PHID types of an application. * * @param string $application Class name of an application - * @return dict Map of constants of application + * @return array Map of constants of application */ public static function getAllTypesForApplication( string $application) { 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 @@ -4548,7 +4548,7 @@ * * This data will be loaded with @{method:loadWorkerState} in the worker. * - * @return dict Serializable editor state. + * @return array Serializable editor state. * @task workers */ private function getWorkerState() { @@ -4573,7 +4573,7 @@ /** * Hook; return custom properties which need to be passed to workers. * - * @return dict Custom properties. + * @return array Custom properties. * @task workers */ protected function getCustomWorkerState() { @@ -4588,7 +4588,7 @@ * This primarily allows binary data to be passed to workers and survive * JSON encoding. * - * @return dict Property encodings. + * @return array Property encodings. * @task workers */ protected function getCustomWorkerStateEncoding() { @@ -4601,7 +4601,7 @@ * * This method is used to load state when running worker operations. * - * @param dict $state Editor state, from + * @param array $state Editor state, from @{method:getWorkerState}. * @return $this * @task workers @@ -4628,7 +4628,7 @@ * Hook; set custom properties on the editor from data emitted by * @{method:getCustomWorkerState}. * - * @param dict $state Custom state, + * @param array $state Custom state, * from @{method:getCustomWorkerState}. * @return $this * @task workers 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 @@ -77,7 +77,7 @@ * Get data from the cache. * * @param list $keys List of cache keys to retrieve. - * @return dict Dictionary of keys that were found in the + * @return array Dictionary of keys that were found in the * cache. Keys not present in the cache are * omitted, so you can detect a cache miss. * @task kvimpl @@ -92,7 +92,7 @@ * after a specified number of seconds. By default, there is no expiration * policy and data will persist in cache indefinitely. * - * @param dict $keys Map of cache keys to values. + * @param array $keys Map of cache keys to values. * @param int|null $ttl (optional) TTL for cache keys, in seconds. * @return $this * @task kvimpl 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 $servers List of server specifications. + * @param array $servers List of server specifications. * @return $this * @task memcache */ 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 @@ -329,7 +329,7 @@ * Dispatch an event to event listeners. * * @param string $type Event type. - * @param dict $params (optional) Event parameters. + * @param array $params (optional) Event parameters. * @return void */ private function dispatchEvent($type, array $params = array()) { 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 @@ -60,10 +60,10 @@ * This method acts as @{function:phutil_tag}, but checks attributes before * using them. * - * @param string $name Tag name. - * @param dict $attrs Tag attributes. - * @param wild $content (optional) Tag content. - * @return PhutilSafeHTML Tag object. + * @param string $name Tag name. + * @param array $attrs Dictionary of tag attributes. + * @param wild $content (optional) Tag content. + * @return PhutilSafeHTML Tag object. */ protected function newTag($name, array $attrs, $content = null) { foreach ($attrs as $key => $attr) { 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 @@ -384,7 +384,7 @@ * advanced, specialized feature and there are usually better approaches for * most locking/contention problems. * - * @return dictionary Map of configuration options to values. + * @return array Map of configuration options to values. * * @task config */ @@ -450,8 +450,8 @@ /** * Loads all of the objects, unconditionally. * - * @return dict Dictionary of all persisted objects of this type, keyed - * on object ID. + * @return array Dictionary of all persisted objects of this + * type, keyed on object ID. * * @task load */ @@ -470,7 +470,7 @@ * * @param string $pattern queryfx()-style SQL WHERE clause. * @param mixed $args,... Zero or more conversions. - * @return dict Dictionary of matching objects, keyed on ID. + * @return array Dictionary of matching objects, keyed on ID. * * @task load */ @@ -574,9 +574,9 @@ * convenient to pull data from elsewhere directly (e.g., a complicated * join via @{method:queryData}) and then load from an array representation. * - * @param dict $row Dictionary of properties, which should be equivalent - * to selecting a row from the table or calling - * @{method:getProperties}. + * @param array $row Dictionary of properties, which + * should be equivalent to selecting a row from the table or calling + * @{method:getProperties}. * @return $this * * @task load @@ -650,7 +650,7 @@ * This is a lot messier than @{method:loadAllWhere}, but more flexible. * * @param list $rows List of property dictionaries. - * @return dict List of constructed objects, keyed on ID. + * @return array List of constructed objects, keyed on ID. * * @task load */ @@ -739,8 +739,8 @@ * database. * Properties that should not be persisted must be declared as private. * - * @return dict Dictionary of normalized (lowercase) to canonical (original - * case) property names. + * @return array Dictionary of normalized (lowercase) to + * canonical (original case) property names. * * @task info */ @@ -874,7 +874,7 @@ * using legacy features with CONFIG_CONVERT_CAMELCASE, but in that case you * should just go ahead and die in a fire). * - * @return dict Dictionary of object properties. + * @return array Dictionary of object properties. * * @task info */