wikimedia/mediawiki-extensions-DiscussionTools
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/DiscussionTools synced 2024-11-24 00:13:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

CommentParser: Inject a forgotten service

Browse source 
Also sort alphabetically.

Change-Id: I9e77c4aa1fba930f382e3c4f17ac0504c2f06668
This commit is contained in:
Bartosz Dziewoński 2022-02-21 18:39:36 +01:00
parent aeabff63c9
commit 85165543f4
3 changed files with 21 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
includes/CommentParser.php
View file

@ -8,7 +8,7 @@ use DateTime;
use DateTimeImmutable;
use DateTimeZone;
use Language;
use MediaWiki\MediaWikiServices;
use MediaWiki\Languages\LanguageConverterFactory;
use MWException;
use Title;
use Wikimedia\IPUtils;
@ -22,16 +22,12 @@ use Wikimedia\Parsoid\Utils\DOMCompat;
class CommentParser {
private const SIGNATURE_SCAN_LIMIT = 100;
/** @var Element */
private $rootNode;
/** @var Title */
private $title;
/** @var Config */
private $config;
/** @var Language */
private $language;
/** @var LanguageConverterFactory */
private $languageConverterFactory;
private $dateFormat;
private $digits;
@ -40,16 +36,26 @@ class CommentParser {
private $localTimezone;
private $timezones;
/** @var Element */
private $rootNode;
/** @var Title */
private $title;
/**
* @param Language $language Content language
* @param Config $config
* @param Language $language Content language
* @param LanguageConverterFactory $languageConverterFactory
* @param LanguageData $languageData
*/
public function __construct(
Language $language, Config $config, LanguageData $languageData
Config $config,
Language $language,
LanguageConverterFactory $languageConverterFactory,
LanguageData $languageData
) {
$this->config = $config;
$this->language = $language;
$this->languageConverterFactory = $languageConverterFactory;
$data = $languageData->getLocalData();
$this->dateFormat = $data['dateFormat'];
@ -464,8 +470,7 @@ class CommentParser {
* @return string[] Regular expressions
*/
public function getLocalTimestampRegexps(): array {
$langConv = MediaWikiServices::getInstance()->getLanguageConverterFactory()
->getLanguageConverter( $this->language );
$langConv = $this->languageConverterFactory->getLanguageConverter( $this->language );
return array_map( function ( $contLangVariant ) {
return $this->getTimestampRegexp(
$contLangVariant,
@ -485,8 +490,7 @@ class CommentParser {
* @return callable[] Parser functions
*/
private function getLocalTimestampParsers(): array {
$langConv = MediaWikiServices::getInstance()->getLanguageConverterFactory()
->getLanguageConverter( $this->language );
$langConv = $this->languageConverterFactory->getLanguageConverter( $this->language );
return array_map( function ( $contLangVariant ) {
return $this->getTimestampParser(
$contLangVariant,

3
includes/ServiceWiring.php
View file

@ -7,8 +7,9 @@ use MediaWiki\MediaWikiServices;
return [
'DiscussionTools.CommentParser' => static function ( MediaWikiServices $services ): CommentParser {
return new CommentParser(
$services->getContentLanguage(),
$services->getMainConfig(),
$services->getContentLanguage(),
$services->getLanguageConverterFactory(),
$services->getService( 'DiscussionTools.LanguageData' )
);
},

3
tests/phpunit/TestUtils.php
View file

@ -124,8 +124,9 @@ trait TestUtils {
public static function createParser( array $data ): CommentParser {
$services = MediaWikiServices::getInstance();
return new CommentParser(
$services->getContentLanguage(),
$services->getMainConfig(),
$services->getContentLanguage(),
$services->getLanguageConverterFactory(),
new MockLanguageData( $data )
);
}