Improve some parameter docs
authorUmherirrender <umherirrender_de.wp@web.de>
Sun, 7 Jan 2018 10:38:43 +0000 (11:38 +0100)
committerUmherirrender <umherirrender_de.wp@web.de>
Sun, 7 Jan 2018 10:39:08 +0000 (11:39 +0100)
Change-Id: I31e983d7ac287158101b18ad95779d83537302a2

21 files changed:
includes/Feed.php
includes/Preferences.php
includes/Revision.php
includes/Storage/RevisionStore.php
includes/Xml.php
includes/collation/IcuCollation.php
includes/libs/objectcache/WANObjectCache.php
includes/preferences/DefaultPreferencesFactory.php
includes/resourceloader/ResourceLoader.php
includes/shell/Result.php
includes/specialpage/LoginSignupSpecialPage.php
includes/watcheditem/WatchedItemStore.php
includes/widget/ComplexNamespaceInputWidget.php
includes/widget/ComplexTitleInputWidget.php
includes/widget/DateInputWidget.php
includes/widget/DateTimeInputWidget.php
includes/widget/NamespaceInputWidget.php
includes/widget/SearchInputWidget.php
includes/widget/SelectWithInputWidget.php
includes/widget/TitleInputWidget.php
includes/widget/UsersMultiselectWidget.php

index 0e715df..92eec08 100644 (file)
@@ -153,6 +153,7 @@ class FeedItem {
        /**
         * Get the description of this item without any escaping
         *
+        * @return string
         */
        public function getDescriptionUnescaped() {
                return $this->description;
index 0a41573..37de2a5 100644 (file)
@@ -315,6 +315,7 @@ class Preferences {
        /**
         * @param array $formData
         * @param PreferencesForm $form
+        * @return Status
         */
        public static function tryUISubmit( $formData, $form ) {
                $preferencesFactory = self::getDefaultPreferencesFactory();
index 54c6de0..b1440d0 100644 (file)
@@ -65,7 +65,7 @@ class Revision implements IDBAccessObject {
        }
 
        /**
-        * @param bool|string $wikiId The ID of the target wiki database. Use false for the local wiki.
+        * @param bool|string $wiki The ID of the target wiki database. Use false for the local wiki.
         *
         * @return SqlBlobStore
         */
index 2e953fc..0950109 100644 (file)
@@ -562,6 +562,7 @@ class RevisionStore implements IDBAccessObject, RevisionFactory, RevisionLookup
        /**
         * MCR migration note: this replaces Revision::isUnpatrolled
         *
+        * @param RevisionRecord $rev
         * @return int Rcid of the unpatrolled row, zero if there isn't one
         */
        public function isUnpatrolled( RevisionRecord $rev ) {
index 0091513..b20f66e 100644 (file)
@@ -532,8 +532,8 @@ class Xml {
         *
         * @param string $list Correctly formatted text (newline delimited) to be
         *   used to generate the options.
-        * @param array $params Extra parameters
-        * @param string $params['other'] If set, add an option with this as text and a value of 'other'
+        * @param array $params Extra parameters:
+        *   - string $params['other'] If set, add an option with this as text and a value of 'other'
         * @return array Array keys are textual labels, values are internal values
         */
        public static function listDropDownOptions( $list, $params = [] ) {
index 23f02de..36efdb3 100644 (file)
@@ -492,7 +492,6 @@ class IcuCollation extends Collation {
        }
 
        /**
-        * @param string $index
         * @return string
         * @since 1.16.3
         */
index 562819e..409f507 100644 (file)
@@ -214,6 +214,9 @@ class WANObjectCache implements IExpiringStore, LoggerAwareInterface {
                $this->asyncHandler = isset( $params['asyncHandler'] ) ? $params['asyncHandler'] : null;
        }
 
+       /**
+        * @param LoggerInterface $logger
+        */
        public function setLogger( LoggerInterface $logger ) {
                $this->logger = $logger;
        }
@@ -1238,7 +1241,7 @@ class WANObjectCache implements IExpiringStore, LoggerAwareInterface {
         * @param string $key
         * @param bool $versioned
         * @param float $minTime
-        * @param mixed $asOf
+        * @param mixed &$asOf
         * @return mixed
         */
        protected function getInterimValue( $key, $versioned, $minTime, &$asOf ) {
index a30aceb..0391b30 100644 (file)
@@ -70,6 +70,12 @@ class DefaultPreferencesFactory implements PreferencesFactory {
        /** @var LinkRenderer */
        protected $linkRenderer;
 
+       /**
+        * @param Config $config
+        * @param Language $contLang
+        * @param AuthManager $authManager
+        * @param LinkRenderer $linkRenderer
+        */
        public function __construct(
                Config $config,
                Language $contLang,
index 63a495a..e1d4f26 100644 (file)
@@ -690,7 +690,6 @@ class ResourceLoader implements LoggerAwareInterface {
         *
         * @since 1.28
         * @param ResourceLoaderContext $context
-        * @param string[] $modules List of module names
         * @return string Hash
         */
        public function makeVersionQuery( ResourceLoaderContext $context ) {
index 1e18210..a105cd1 100644 (file)
@@ -38,6 +38,7 @@ class Result {
        /**
         * @param int $exitCode
         * @param string $stdout
+        * @param string|null $stderr
         */
        public function __construct( $exitCode, $stdout, $stderr = null ) {
                $this->exitCode = $exitCode;
index 963840b..d6ace0a 100644 (file)
@@ -478,7 +478,6 @@ abstract class LoginSignupSpecialPage extends AuthManagerSpecialPage {
        /**
         * Replace some globals to make sure the fact that the user has just been logged in is
         * reflected in the current request.
-        * @param User $user
         */
        protected function setSessionUserForCurrentRequest() {
                global $wgUser, $wgLang;
index f29bd47..c68d6d7 100644 (file)
@@ -78,6 +78,9 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
                $this->revisionGetTimestampFromIdCallback = [ 'Revision', 'getTimestampFromId' ];
        }
 
+       /**
+        * @param StatsdDataFactoryInterface $stats
+        */
        public function setStatsdDataFactory( StatsdDataFactoryInterface $stats ) {
                $this->stats = $stats;
        }
@@ -241,6 +244,8 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.31
+        * @param User $user
+        * @return int
         */
        public function countWatchedItems( User $user ) {
                $dbr = $this->getConnectionRef( DB_REPLICA );
@@ -258,6 +263,8 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param LinkTarget $target
+        * @return int
         */
        public function countWatchers( LinkTarget $target ) {
                $dbr = $this->getConnectionRef( DB_REPLICA );
@@ -276,6 +283,9 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param LinkTarget $target
+        * @param string|int $threshold
+        * @return int
         */
        public function countVisitingWatchers( LinkTarget $target, $threshold ) {
                $dbr = $this->getConnectionRef( DB_REPLICA );
@@ -297,6 +307,9 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param LinkTarget[] $targets
+        * @param array $options
+        * @return array
         */
        public function countWatchersMultiple( array $targets, array $options = [] ) {
                $dbOptions = [ 'GROUP BY' => [ 'wl_namespace', 'wl_title' ] ];
@@ -330,6 +343,9 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param array $targetsWithVisitThresholds
+        * @param int|null $minimumWatchers
+        * @return array
         */
        public function countVisitingWatchersMultiple(
                array $targetsWithVisitThresholds,
@@ -410,6 +426,9 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param User $user
+        * @param LinkTarget $target
+        * @return bool
         */
        public function getWatchedItem( User $user, LinkTarget $target ) {
                if ( $user->isAnon() ) {
@@ -427,6 +446,9 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param User $user
+        * @param LinkTarget $target
+        * @return bool
         */
        public function loadWatchedItem( User $user, LinkTarget $target ) {
                // Only loggedin user can have a watchlist
@@ -458,6 +480,9 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param User $user
+        * @param array $options
+        * @return WatchedItem[]
         */
        public function getWatchedItemsForUser( User $user, array $options = [] ) {
                $options += [ 'forWrite' => false ];
@@ -499,6 +524,9 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param User $user
+        * @param LinkTarget $target
+        * @return bool
         */
        public function isWatched( User $user, LinkTarget $target ) {
                return (bool)$this->getWatchedItem( $user, $target );
@@ -506,6 +534,9 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param User $user
+        * @param LinkTarget[] $targets
+        * @return array
         */
        public function getNotificationTimestampsBatch( User $user, array $targets ) {
                $timestamps = [];
@@ -555,6 +586,8 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param User $user
+        * @param LinkTarget $target
         */
        public function addWatch( User $user, LinkTarget $target ) {
                $this->addWatchBatchForUser( $user, [ $target ] );
@@ -562,6 +595,9 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param User $user
+        * @param LinkTarget[] $targets
+        * @return bool
         */
        public function addWatchBatchForUser( User $user, array $targets ) {
                if ( $this->readOnlyMode->isReadOnly() ) {
@@ -611,6 +647,9 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param User $user
+        * @param LinkTarget $target
+        * @return bool
         */
        public function removeWatch( User $user, LinkTarget $target ) {
                // Only logged in user can have a watchlist
@@ -635,6 +674,10 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param User $user
+        * @param string|int $timestamp
+        * @param LinkTarget[] $targets
+        * @return bool
         */
        public function setNotificationTimestampsForUser( User $user, $timestamp, array $targets = [] ) {
                // Only loggedin user can have a watchlist
@@ -668,6 +711,10 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param User $editor
+        * @param LinkTarget $target
+        * @param string|int $timestamp
+        * @return int
         */
        public function updateNotificationTimestamp( User $editor, LinkTarget $target, $timestamp ) {
                $dbw = $this->getConnectionRef( DB_MASTER );
@@ -724,6 +771,11 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param User $user
+        * @param Title $title
+        * @param string $force
+        * @param int $oldid
+        * @return bool
         */
        public function resetNotificationTimestamp( User $user, Title $title, $force = '', $oldid = 0 ) {
                // Only loggedin user can have a watchlist
@@ -813,6 +865,9 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param User $user
+        * @param int|null $unreadLimit
+        * @return int|bool
         */
        public function countUnreadNotifications( User $user, $unreadLimit = null ) {
                $queryOptions = [];
@@ -846,6 +901,8 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param LinkTarget $oldTarget
+        * @param LinkTarget $newTarget
         */
        public function duplicateAllAssociatedEntries( LinkTarget $oldTarget, LinkTarget $newTarget ) {
                $oldTarget = Title::newFromLinkTarget( $oldTarget );
@@ -857,6 +914,8 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
 
        /**
         * @since 1.27
+        * @param LinkTarget $oldTarget
+        * @param LinkTarget $newTarget
         */
        public function duplicateEntry( LinkTarget $oldTarget, LinkTarget $newTarget ) {
                $dbw = $this->getConnectionRef( DB_MASTER );
index cdcdd24..69844d9 100644 (file)
@@ -22,20 +22,20 @@ class ComplexNamespaceInputWidget extends \OOUI\Widget {
 
        /**
         * @param array $config Configuration options
-        * @param array $config['namespace'] Configuration for the NamespaceInputWidget
-        *  dropdown with list of namespaces
-        * @param string $config['namespace']['includeAllValue'] If specified,
-        *  add an "all namespaces" option to the dropdown, and use this as the input value for it
-        * @param array|null $config['invert'] Configuration for the "invert selection"
-        *  CheckboxInputWidget. If null, the checkbox will not be generated.
-        * @param array|null $config['associated'] Configuration for the "include associated namespace"
-        *  CheckboxInputWidget. If null, the checkbox will not be generated.
-        * @param array $config['invertLabel'] Configuration for the FieldLayout with label
-        *  wrapping the "invert selection" checkbox
-        * @param string $config['invertLabel']['label'] Label text for the label
-        * @param array $config['associatedLabel'] Configuration for the FieldLayout with label
-        *  wrapping the "include associated namespace" checkbox
-        * @param string $config['associatedLabel']['label'] Label text for the label
+        *   - array $config['namespace'] Configuration for the NamespaceInputWidget
+        *     dropdown with list of namespaces
+        *   - string $config['namespace']['includeAllValue'] If specified,
+        *     add an "all namespaces" option to the dropdown, and use this as the input value for it
+        *   - array|null $config['invert'] Configuration for the "invert selection"
+        *     CheckboxInputWidget. If null, the checkbox will not be generated.
+        *   - array|null $config['associated'] Configuration for the "include associated namespace"
+        *     CheckboxInputWidget. If null, the checkbox will not be generated.
+        *   - array $config['invertLabel'] Configuration for the FieldLayout with label
+        *     wrapping the "invert selection" checkbox
+        *   - string $config['invertLabel']['label'] Label text for the label
+        *   - array $config['associatedLabel'] Configuration for the FieldLayout with label
+        *     wrapping the "include associated namespace" checkbox
+        *   - string $config['associatedLabel']['label'] Label text for the label
         */
        public function __construct( array $config = [] ) {
                // Configuration initialization
index aec6619..e111a97 100644 (file)
@@ -19,9 +19,9 @@ class ComplexTitleInputWidget extends \OOUI\Widget {
         * Like TitleInputWidget, but the namespace has to be input through a separate dropdown field.
         *
         * @param array $config Configuration options
-        * @param array $config['namespace'] Configuration for the NamespaceInputWidget dropdown
-        *  with list of namespaces
-        * @param array $config['title'] Configuration for the TitleInputWidget text field
+        *   - array $config['namespace'] Configuration for the NamespaceInputWidget dropdown
+        *     with list of namespaces
+        *   - array $config['title'] Configuration for the TitleInputWidget text field
         */
        public function __construct( array $config = [] ) {
                // Configuration initialization
index 0231cac..afa6570 100644 (file)
@@ -28,29 +28,29 @@ class DateInputWidget extends \OOUI\TextInputWidget {
 
        /**
         * @param array $config Configuration options
-        * @param string $config['inputFormat'] Date format string to use for the textual input field.
-        *   Displayed while the widget is active, and the user can type in a date in this format.
-        *   Should be short and easy to type. (default: 'YYYY-MM-DD' or 'YYYY-MM', depending on
-        *   `precision`)
-        * @param string $config['displayFormat'] Date format string to use for the clickable label.
-        *   while the widget is inactive. Should be as unambiguous as possible (for example, prefer
-        *   to spell out the month, rather than rely on the order), even if that makes it longer.
-        *   Applicable only if the widget is infused. (default: language-specific)
-        * @param string $config['longDisplayFormat'] If a custom displayFormat is not specified, use
-        *   unabbreviated day of the week and month names in the default language-specific
-        *   displayFormat. (default: false)
-        * @param string $config['placeholderLabel'] Placeholder text shown when the widget is not
-        *   selected. Applicable only if the widget is infused. (default: taken from message
-        *   `mw-widgets-dateinput-no-date`)
-        * @param string $config['placeholderDateFormat'] User-visible date format string displayed
-        *   in the textual input field when it's empty. Should be the same as `inputFormat`, but
-        *   translated to the user's language. (default: 'YYYY-MM-DD' or 'YYYY-MM', depending on
-        *   `precision`)
-        * @param string $config['precision'] Date precision to use, 'day' or 'month' (default: 'day')
-        * @param string $config['mustBeAfter'] Validates the date to be after this.
-        *   In the 'YYYY-MM-DD' or 'YYYY-MM' format, depending on `precision`.
-        * @param string $config['mustBeBefore'] Validates the date to be before this.
-        *   In the 'YYYY-MM-DD' or 'YYYY-MM' format, depending on `precision`.
+        *   - string $config['inputFormat'] Date format string to use for the textual input field.
+        *     Displayed while the widget is active, and the user can type in a date in this format.
+        *     Should be short and easy to type. (default: 'YYYY-MM-DD' or 'YYYY-MM', depending on
+        *     `precision`)
+        *   - string $config['displayFormat'] Date format string to use for the clickable label.
+        *     while the widget is inactive. Should be as unambiguous as possible (for example, prefer
+        *     to spell out the month, rather than rely on the order), even if that makes it longer.
+        *     Applicable only if the widget is infused. (default: language-specific)
+        *   - string $config['longDisplayFormat'] If a custom displayFormat is not specified, use
+        *     unabbreviated day of the week and month names in the default language-specific
+        *     displayFormat. (default: false)
+        *   - string $config['placeholderLabel'] Placeholder text shown when the widget is not
+        *     selected. Applicable only if the widget is infused. (default: taken from message
+        *     `mw-widgets-dateinput-no-date`)
+        *   - string $config['placeholderDateFormat'] User-visible date format string displayed
+        *     in the textual input field when it's empty. Should be the same as `inputFormat`, but
+        *     translated to the user's language. (default: 'YYYY-MM-DD' or 'YYYY-MM', depending on
+        *     `precision`)
+        *   - string $config['precision'] Date precision to use, 'day' or 'month' (default: 'day')
+        *   - string $config['mustBeAfter']    Validates the date to be after this.
+        *     In the 'YYYY-MM-DD' or 'YYYY-MM' format, depending on `precision`.
+        *   - string $config['mustBeBefore'] Validates the date to be before this.
+        *     In the 'YYYY-MM-DD' or 'YYYY-MM' format, depending on `precision`.
         */
        public function __construct( array $config = [] ) {
                $config = array_merge( [
index 4c41212..b726f3c 100644 (file)
@@ -21,10 +21,10 @@ class DateTimeInputWidget extends \OOUI\InputWidget {
 
        /**
         * @param array $config Configuration options
-        * @param string $config['type'] 'date', 'time', or 'datetime'
-        * @param string $config['min'] Minimum date, time, or datetime
-        * @param string $config['max'] Maximum date, time, or datetime
-        * @param bool $config['clearable'] Whether to provide for blanking the value.
+        *   - string $config['type'] 'date', 'time', or 'datetime'
+        *   - string $config['min'] Minimum date, time, or datetime
+        *   - string $config['max'] Maximum date, time, or datetime
+        *   - bool $config['clearable'] Whether to provide for blanking the value.
         */
        public function __construct( array $config = [] ) {
                // We need $this->type set before calling the parent constructor
index c638891..6a0c0da 100644 (file)
@@ -16,9 +16,9 @@ class NamespaceInputWidget extends \OOUI\DropdownInputWidget {
 
        /**
         * @param array $config Configuration options
-        * @param string $config['includeAllValue'] If specified, add a "all namespaces" option to the
+        *   - string $config['includeAllValue'] If specified, add a "all namespaces" option to the
         *     namespace dropdown, and use this as the input value for it
-        * @param int[] $config['exclude'] List of namespace numbers to exclude from the selector
+        *   - int[] $config['exclude'] List of namespace numbers to exclude from the selector
         */
        public function __construct( array $config = [] ) {
                // Configuration initialization
index e2428ba..e468396 100644 (file)
@@ -20,13 +20,13 @@ class SearchInputWidget extends TitleInputWidget {
 
        /**
         * @param array $config Configuration options
-        * @param int|null $config['pushPending'] Whether the input should be visually marked as
-        *  "pending", while requesting suggestions (default: false)
-        * @param bool|null $config['performSearchOnClick'] If true, the script will start a search
-        *  whenever a user hits a suggestion. If false, the text of the suggestion is inserted into the
-        *  text field only (default: true)
-        * @param string $config['dataLocation'] Where the search input field will be
-        *  used (header or content, default: header)
+        *   - int|null $config['pushPending'] Whether the input should be visually marked as
+        *     "pending", while requesting suggestions (default: false)
+        *   - bool|null $config['performSearchOnClick'] If true, the script will start a search
+        *     whenever a user hits a suggestion. If false, the text of the suggestion is inserted into
+        *     the text field only (default: true)
+        *   - string $config['dataLocation'] Where the search input field will be
+        *     used (header or content, default: header)
         */
        public function __construct( array $config = [] ) {
                $config = array_merge( [
index 45dd1aa..994906a 100644 (file)
@@ -22,9 +22,9 @@ class SelectWithInputWidget extends \OOUI\Widget {
         * A version of the SelectWithInputWidget, with `or` set to true.
         *
         * @param array $config Configuration options
-        * @param array $config['textinput'] Configuration for the TextInputWidget
-        * @param array $config['dropdowninput'] Configuration for the DropdownInputWidget
-        * @param bool $config['or'] Configuration for whether the widget is dropdown AND input
+        *   - array $config['textinput'] Configuration for the TextInputWidget
+        *   - array $config['dropdowninput'] Configuration for the DropdownInputWidget
+        *   - bool $config['or'] Configuration for whether the widget is dropdown AND input
         *                              or dropdown OR input
         */
        public function __construct( array $config = [] ) {
index 15f48e5..aca9163 100644 (file)
@@ -20,14 +20,14 @@ class TitleInputWidget extends \OOUI\TextInputWidget {
 
        /**
         * @param array $config Configuration options
-        * @param int|null $config['namespace'] Namespace to prepend to queries
-        * @param bool|null $config['relative'] If a namespace is set,
-        *  return a title relative to it (default: true)
-        * @param bool|null $config['suggestions'] Display search suggestions (default: true)
-        * @param bool|null $config['highlightFirst'] Automatically highlight
-        *  the first result (default: true)
-        * @param bool|null $config['validateTitle'] Whether the input must
-        *  be a valid title (default: true)
+        *   - int|null $config['namespace'] Namespace to prepend to queries
+        *   - bool|null $config['relative'] If a namespace is set,
+        *     return a title relative to it (default: true)
+        *   - bool|null $config['suggestions'] Display search suggestions (default: true)
+        *   - bool|null $config['highlightFirst'] Automatically highlight
+        *     the first result (default: true)
+        *   - bool|null $config['validateTitle'] Whether the input must
+        *     be a valid title (default: true)
         */
        public function __construct( array $config = [] ) {
                parent::__construct(
index 5c4a91f..69fa1f8 100644 (file)
@@ -20,9 +20,9 @@ class UsersMultiselectWidget extends \OOUI\Widget {
 
        /**
         * @param array $config Configuration options
-        * @param array $config['users'] Array of usernames to use as preset data
-        * @param array $config['placeholder'] Placeholder message for input
-        * @param array $config['name'] Name attribute (used in forms)
+        *   - array $config['users'] Array of usernames to use as preset data
+        *   - array $config['placeholder'] Placeholder message for input
+        *   - array $config['name'] Name attribute (used in forms)
         */
        public function __construct( array $config = [] ) {
                parent::__construct( $config );