From: Timo Tijhof Date: Sat, 8 Jul 2017 00:19:39 +0000 (-0700) Subject: statsd: Rename MediawikiStatsdDataFactory to IBufferingStatsdDataFactory X-Git-Tag: 1.31.0-rc.0~2758^2 X-Git-Url: https://git.cyclocoop.org/%7B%7B%20url_for%28%27votes%27%2C%20votes=%27waiting%27%29%20%7D%7D?a=commitdiff_plain;h=01938ae7db2a5911c6e88b515acf259377a5b54c;p=lhc%2Fweb%2Fwiklou.git statsd: Rename MediawikiStatsdDataFactory to IBufferingStatsdDataFactory Follows-up I2874175647e (7fdc3d09a3b). * Avoid "MediaWiki" in class names. If unavoidable, always camelcase. * Use a more descriptive name, matching the implementing the class ("Buffering"). * Use "I" prefix for consistency with other MediaWiki-provided interfaces. (Avoid "Interface" suffix, which is foreign to MediaWiki and is only used in the StatsdClient namespace, which this interface is not in). Bug: T166354 Change-Id: I06de59122625f9c23e7c1a1bfa69a7ddabbf379e --- diff --git a/autoload.php b/autoload.php index 2560bdbdb9..88b61700e3 100644 --- a/autoload.php +++ b/autoload.php @@ -604,6 +604,7 @@ $wgAutoloadLocalClasses = [ 'HttpError' => __DIR__ . '/includes/exception/HttpError.php', 'HttpStatus' => __DIR__ . '/includes/libs/HttpStatus.php', 'IApiMessage' => __DIR__ . '/includes/api/ApiMessage.php', + 'IBufferingStatsdDataFactory' => __DIR__ . '/includes/libs/stats/IBufferingStatsdDataFactory.php', 'ICacheHelper' => __DIR__ . '/includes/cache/CacheHelper.php', 'IContextSource' => __DIR__ . '/includes/context/IContextSource.php', 'IDBAccessObject' => __DIR__ . '/includes/dao/IDBAccessObject.php', @@ -964,7 +965,6 @@ $wgAutoloadLocalClasses = [ 'MediaWiki\\Widget\\TitleInputWidget' => __DIR__ . '/includes/widget/TitleInputWidget.php', 'MediaWiki\\Widget\\UserInputWidget' => __DIR__ . '/includes/widget/UserInputWidget.php', 'MediaWiki\\Widget\\UsersMultiselectWidget' => __DIR__ . '/includes/widget/UsersMultiselectWidget.php', - 'MediawikiStatsdDataFactory' => __DIR__ . '/includes/libs/stats/MediawikiStatsdDataFactory.php', 'MemCachedClientforWiki' => __DIR__ . '/includes/compat/MemcachedClientCompat.php', 'MemcLockManager' => __DIR__ . '/includes/libs/lockmanager/MemcLockManager.php', 'MemcachedBagOStuff' => __DIR__ . '/includes/libs/objectcache/MemcachedBagOStuff.php', diff --git a/includes/MediaWikiServices.php b/includes/MediaWikiServices.php index ea0ec15c9d..84fc959fa7 100644 --- a/includes/MediaWikiServices.php +++ b/includes/MediaWikiServices.php @@ -9,7 +9,7 @@ use EventRelayerGroup; use GenderCache; use GlobalVarConfig; use Hooks; -use MediawikiStatsdDataFactory; +use IBufferingStatsdDataFactory; use Wikimedia\Rdbms\LBFactory; use LinkCache; use Wikimedia\Rdbms\LoadBalancer; @@ -447,7 +447,7 @@ class MediaWikiServices extends ServiceContainer { /** * @since 1.27 - * @return MediawikiStatsdDataFactory + * @return IBufferingStatsdDataFactory */ public function getStatsdDataFactory() { return $this->getService( 'StatsdDataFactory' ); diff --git a/includes/context/ContextSource.php b/includes/context/ContextSource.php index 36d6df2c57..434201a174 100644 --- a/includes/context/ContextSource.php +++ b/includes/context/ContextSource.php @@ -170,7 +170,7 @@ abstract class ContextSource implements IContextSource { * @deprecated since 1.27 use a StatsdDataFactory from MediaWikiServices (preferably injected) * * @since 1.25 - * @return MediawikiStatsdDataFactory + * @return IBufferingStatsdDataFactory */ public function getStats() { return MediaWikiServices::getInstance()->getStatsdDataFactory(); diff --git a/includes/context/DerivativeContext.php b/includes/context/DerivativeContext.php index 9c3c42a92d..0d0c149d80 100644 --- a/includes/context/DerivativeContext.php +++ b/includes/context/DerivativeContext.php @@ -109,7 +109,7 @@ class DerivativeContext extends ContextSource implements MutableContext { * * @deprecated since 1.27 use a StatsdDataFactory from MediaWikiServices (preferably injected) * - * @return MediawikiStatsdDataFactory + * @return IBufferingStatsdDataFactory */ public function getStats() { return MediaWikiServices::getInstance()->getStatsdDataFactory(); diff --git a/includes/context/IContextSource.php b/includes/context/IContextSource.php index d13e1a5705..895e9e4b55 100644 --- a/includes/context/IContextSource.php +++ b/includes/context/IContextSource.php @@ -131,7 +131,7 @@ interface IContextSource extends MessageLocalizer { * @deprecated since 1.27 use a StatsdDataFactory from MediaWikiServices (preferably injected) * * @since 1.25 - * @return MediawikiStatsdDataFactory + * @return IBufferingStatsdDataFactory */ public function getStats(); diff --git a/includes/context/RequestContext.php b/includes/context/RequestContext.php index 2cabfe1013..2ac41925d8 100644 --- a/includes/context/RequestContext.php +++ b/includes/context/RequestContext.php @@ -138,7 +138,7 @@ class RequestContext implements IContextSource, MutableContext { * * @deprecated since 1.27 use a StatsdDataFactory from MediaWikiServices (preferably injected) * - * @return MediawikiStatsdDataFactory + * @return IBufferingStatsdDataFactory */ public function getStats() { return MediaWikiServices::getInstance()->getStatsdDataFactory(); diff --git a/includes/libs/stats/BufferingStatsdDataFactory.php b/includes/libs/stats/BufferingStatsdDataFactory.php index f687254e3c..73c6a8f55c 100644 --- a/includes/libs/stats/BufferingStatsdDataFactory.php +++ b/includes/libs/stats/BufferingStatsdDataFactory.php @@ -32,7 +32,7 @@ use Liuggio\StatsdClient\Factory\StatsdDataFactory; * * @since 1.25 */ -class BufferingStatsdDataFactory extends StatsdDataFactory implements MediawikiStatsdDataFactory { +class BufferingStatsdDataFactory extends StatsdDataFactory implements IBufferingStatsdDataFactory { protected $buffer = []; /** * Collection enabled? diff --git a/includes/libs/stats/IBufferingStatsdDataFactory.php b/includes/libs/stats/IBufferingStatsdDataFactory.php new file mode 100644 index 0000000000..64ee2676f6 --- /dev/null +++ b/includes/libs/stats/IBufferingStatsdDataFactory.php @@ -0,0 +1,30 @@ + [ 'MainConfig', Config::class ], 'SiteStore' => [ 'SiteStore', SiteStore::class ], 'SiteLookup' => [ 'SiteLookup', SiteLookup::class ], - 'StatsdDataFactory' => [ 'StatsdDataFactory', MediawikiStatsdDataFactory::class ], + 'StatsdDataFactory' => [ 'StatsdDataFactory', IBufferingStatsdDataFactory::class ], 'InterwikiLookup' => [ 'InterwikiLookup', InterwikiLookup::class ], 'EventRelayerGroup' => [ 'EventRelayerGroup', EventRelayerGroup::class ], 'SearchEngineFactory' => [ 'SearchEngineFactory', SearchEngineFactory::class ],