Mark CONSTRUCTOR_OPTIONS as internal

These were never meant to be part of the public interface and should not
ever have been marked with @since. They're only useful for constructing
the respective objects, which no outside users should be doing.

Change-Id: I86e01272d46fc72af32172d8a12b9180971d4613
This commit is contained in:
Aryeh Gregor 2019-10-25 11:07:22 +03:00 committed by C. Scott Ananian
parent a1ca5339e3
commit a24e8a06b5
26 changed files with 52 additions and 23 deletions

View file

@ -106,6 +106,9 @@ class MovePage {
*/
private $hookRunner;
/**
* @internal For use by MovePageTest
*/
public const CONSTRUCTOR_OPTIONS = [
'CategoryCollation'
];

View file

@ -58,8 +58,7 @@ class PermissionManager {
public const RIGOR_SECURE = 'secure';
/**
* @since 1.34
* @var array
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'WhitelistRead',

View file

@ -55,8 +55,7 @@ class BlobStoreFactory {
private $options;
/**
* @var array
* @since 1.34
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'CompressRevisions',

View file

@ -80,7 +80,7 @@ class PageUpdater {
/**
* Options that have to be present in the ServiceOptions object passed to the constructor.
*
* @since 1.36
* @internal
*/
public const CONSTRUCTOR_OPTIONS = [
'ManualRevertSearchRadius'

View file

@ -51,8 +51,7 @@ class BlockManager {
private $options;
/**
* @var array
* @since 1.34
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'ApplyIpBlocksToXff',

View file

@ -46,6 +46,9 @@ class DatabaseBlockStore {
/** @var ServiceOptions */
private $options;
/**
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'PutIPinRC',
'BlockDisablesLogin',

View file

@ -230,8 +230,7 @@ class LocalisationCache {
}
/**
* @var array
* @since 1.34
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
// True to treat all files as expired until they are regenerated by this object.

View file

@ -36,8 +36,7 @@ abstract class MWLBFactory {
private static $loggedDeprecations = [];
/**
* @var array
* @since 1.34
* @internal For use by ServiceWiring
*/
public const APPLY_DEFAULT_CONFIG_OPTIONS = [
'DBcompress',

View file

@ -63,7 +63,7 @@ class FileBackendGroup {
private $objectFactory;
/**
* @internal
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'DirectoryMode',

View file

@ -41,6 +41,9 @@ class HttpRequestFactory {
/** @var LoggerInterface */
private $logger;
/**
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'HTTPTimeout',
'HTTPConnectTimeout',

View file

@ -40,6 +40,9 @@ use Wikimedia\ScopedCallback;
*/
class JobRunner implements LoggerAwareInterface {
/**
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'JobBackoffThrottling',
'JobClasses',

View file

@ -65,8 +65,7 @@ class LanguageFactory {
private $parentLangCache = [];
/**
* @since 1.35
* @var array
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'DummyLanguageCodes',

View file

@ -75,6 +75,9 @@ class LanguageNameUtils {
*/
private $validCodeCache = [];
/**
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'ExtraLanguageNames',
'UsePigLatinVariant',

View file

@ -75,6 +75,9 @@ class PageCommandFactory implements ContentModelChangeFactory, MergeHistoryFacto
/** @var HookContainer */
private $hookContainer;
/**
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'CategoryCollation'
];

View file

@ -360,8 +360,7 @@ class Parser {
private $hookRunner;
/**
* @var array
* @since 1.35
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
// Deprecated and unused; from $wgParserConf

View file

@ -92,8 +92,7 @@ class DefaultPreferencesFactory implements PreferencesFactory {
private $hookRunner;
/**
* @var array
* @since 1.34
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'AllowRequiringEmailForResets',

View file

@ -298,8 +298,7 @@ class SpecialPageFactory {
private $hookRunner;
/**
* @var array
* @since 1.35
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'DisableInternalSearch',

View file

@ -83,8 +83,7 @@ class NamespaceInfo {
];
/**
* @since 1.34
* @internal
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'AllowImageMoving',

View file

@ -34,6 +34,9 @@ use Wikimedia\Assert\Assert;
*/
class DefaultOptionsLookup extends UserOptionsLookup {
/**
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'DefaultSkin',
'DefaultUserOptions',

View file

@ -68,6 +68,9 @@ class PasswordReset implements LoggerAwareInterface {
*/
private $permissionCache;
/**
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'AllowRequiringEmailForResets',
'EnableEmail',

View file

@ -34,6 +34,9 @@ use Wikimedia\Rdbms\ILoadBalancer;
*/
class TalkPageNotificationManager {
/**
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'DisableAnonTalk'
];

View file

@ -50,6 +50,9 @@ use Wikimedia\Rdbms\ILoadBalancer;
*/
class UserGroupManager implements IDBAccessObject {
/**
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'Autopromote',
'AutopromoteOnce',

View file

@ -41,6 +41,9 @@ use Wikimedia\Message\MessageValue;
*/
class UserNameUtils {
/**
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'MaxNameChars',
'ReservedUsernames',

View file

@ -43,6 +43,9 @@ use Wikimedia\Rdbms\ILoadBalancer;
*/
class UserOptionsManager extends UserOptionsLookup {
/**
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'HiddenPrefs'
];

View file

@ -39,6 +39,9 @@ use WatchedItemStoreInterface;
*/
class WatchlistNotificationManager {
/**
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'UseEnotif',
'ShowUpdatedMarker',

View file

@ -26,7 +26,7 @@ use Wikimedia\ScopedCallback;
class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterface {
/**
* @since 1.35
* @internal For use by ServiceWiring
*/
public const CONSTRUCTOR_OPTIONS = [
'UpdateRowsPerQuery',