Thiemo Mättig (WMDE) has uploaded a new change for review.
https://gerrit.wikimedia.org/r/249437
Change subject: Rename $dataUpdate(s) variables to $dataUpdater with an r
......................................................................
Rename $dataUpdate(s) variables to $dataUpdater with an r
To restore consistency between class and variable names again.
Change-Id: Ibfd576d69cf77550c1a1175e2d690b6f305389e9
---
M repo/includes/ParserOutput/EntityParserOutputDataUpdater.php
M repo/includes/ParserOutput/EntityParserOutputGenerator.php
M repo/includes/ParserOutput/EntityParserOutputGeneratorFactory.php
M repo/tests/phpunit/includes/ParserOutput/EntityParserOutputDataUpdaterTest.php
M repo/tests/phpunit/includes/ParserOutput/EntityParserOutputGeneratorTest.php
M repo/tests/phpunit/includes/ParserOutput/GeoDataDataUpdaterTest.php
6 files changed, 50 insertions(+), 50 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase
refs/changes/37/249437/1
diff --git a/repo/includes/ParserOutput/EntityParserOutputDataUpdater.php
b/repo/includes/ParserOutput/EntityParserOutputDataUpdater.php
index be2ef6f..3b561f7 100644
--- a/repo/includes/ParserOutput/EntityParserOutputDataUpdater.php
+++ b/repo/includes/ParserOutput/EntityParserOutputDataUpdater.php
@@ -27,38 +27,38 @@
/**
* @var ParserOutputDataUpdater[]
*/
- private $dataUpdates;
+ private $dataUpdaters;
/**
* @var StatementDataUpdater[]
*/
- private $statementDataUpdates = array();
+ private $statementDataUpdaters = array();
/**
* @var SiteLinkDataUpdater[]
*/
- private $siteLinkDataUpdates = array();
+ private $siteLinkDataUpdaters = array();
/**
* @param ParserOutput $parserOutput
- * @param ParserOutputDataUpdater[] $dataUpdates
+ * @param ParserOutputDataUpdater[] $dataUpdaters
*
* @throws InvalidArgumentException
*/
- public function __construct( ParserOutput $parserOutput, array
$dataUpdates ) {
- foreach ( $dataUpdates as $dataUpdate ) {
- if ( $dataUpdate instanceof StatementDataUpdater ) {
- $this->statementDataUpdates[] = $dataUpdate;
- } elseif ( $dataUpdate instanceof SiteLinkDataUpdater )
{
- $this->siteLinkDataUpdates[] = $dataUpdate;
+ public function __construct( ParserOutput $parserOutput, array
$dataUpdaters ) {
+ foreach ( $dataUpdaters as $dataUpdater ) {
+ if ( $dataUpdater instanceof StatementDataUpdater ) {
+ $this->statementDataUpdaters[] = $dataUpdater;
+ } elseif ( $dataUpdater instanceof SiteLinkDataUpdater
) {
+ $this->siteLinkDataUpdaters[] = $dataUpdater;
} else {
- throw new InvalidArgumentException( 'Each
$dataUpdates element must be a '
+ throw new InvalidArgumentException( 'Each
$dataUpdaters element must be a '
. 'StatementDataUpdater,
SiteLinkDataUpdater or both' );
}
}
$this->parserOutput = $parserOutput;
- $this->dataUpdates = $dataUpdates;
+ $this->dataUpdaters = $dataUpdaters;
}
/**
@@ -78,13 +78,13 @@
* @param StatementListProvider $entity
*/
private function processStatementListProvider( StatementListProvider
$entity ) {
- if ( empty( $this->statementDataUpdates ) ) {
+ if ( empty( $this->statementDataUpdaters ) ) {
return;
}
foreach ( $entity->getStatements() as $statement ) {
- foreach ( $this->statementDataUpdates as $dataUpdate ) {
- $dataUpdate->processStatement( $statement );
+ foreach ( $this->statementDataUpdaters as $dataUpdater
) {
+ $dataUpdater->processStatement( $statement );
}
}
}
@@ -93,20 +93,20 @@
* @param Item $item
*/
private function processItem( Item $item ) {
- if ( empty( $this->siteLinkDataUpdates ) ) {
+ if ( empty( $this->siteLinkDataUpdaters ) ) {
return;
}
foreach ( $item->getSiteLinkList() as $siteLink ) {
- foreach ( $this->siteLinkDataUpdates as $dataUpdate ) {
- $dataUpdate->processSiteLink( $siteLink );
+ foreach ( $this->siteLinkDataUpdaters as $dataUpdater )
{
+ $dataUpdater->processSiteLink( $siteLink );
}
}
}
public function finish() {
- foreach ( $this->dataUpdates as $dataUpdate ) {
- $dataUpdate->updateParserOutput( $this->parserOutput );
+ foreach ( $this->dataUpdaters as $dataUpdater ) {
+ $dataUpdater->updateParserOutput( $this->parserOutput );
}
}
diff --git a/repo/includes/ParserOutput/EntityParserOutputGenerator.php
b/repo/includes/ParserOutput/EntityParserOutputGenerator.php
index d23b7b1..f6e6a1d 100644
--- a/repo/includes/ParserOutput/EntityParserOutputGenerator.php
+++ b/repo/includes/ParserOutput/EntityParserOutputGenerator.php
@@ -78,7 +78,7 @@
/**
* @var ParserOutputDataUpdater[]
*/
- private $dataUpdates;
+ private $dataUpdaters;
/**
* @var string
@@ -93,7 +93,7 @@
LanguageFallbackChain $languageFallbackChain,
TemplateFactory $templateFactory,
EntityDataFormatProvider $entityDataFormatProvider,
- array $dataUpdates,
+ array $dataUpdaters,
$languageCode
) {
$this->entityViewFactory = $entityViewFactory;
@@ -104,7 +104,7 @@
$this->languageCode = $languageCode;
$this->templateFactory = $templateFactory;
$this->entityDataFormatProvider = $entityDataFormatProvider;
- $this->dataUpdates = $dataUpdates;
+ $this->dataUpdaters = $dataUpdaters;
$this->languageCode = $languageCode;
}
@@ -142,7 +142,7 @@
$entity = $entityRevision->getEntity();
- $dataUpdater = new EntityParserOutputDataUpdater(
$parserOutput, $this->dataUpdates );
+ $dataUpdater = new EntityParserOutputDataUpdater(
$parserOutput, $this->dataUpdaters );
$dataUpdater->processEntity( $entity );
$dataUpdater->finish();
diff --git a/repo/includes/ParserOutput/EntityParserOutputGeneratorFactory.php
b/repo/includes/ParserOutput/EntityParserOutputGeneratorFactory.php
index 02ef68c..e053b1c 100644
--- a/repo/includes/ParserOutput/EntityParserOutputGeneratorFactory.php
+++ b/repo/includes/ParserOutput/EntityParserOutputGeneratorFactory.php
@@ -165,7 +165,7 @@
private function getDataUpdates() {
$propertyDataTypeMatcher = new PropertyDataTypeMatcher(
$this->propertyDataTypeLookup );
- $dataUpdates = array(
+ $dataUpdaters = array(
new ReferencedEntitiesDataUpdater(
$this->entityTitleLookup,
$this->externalEntityIdParser
@@ -175,18 +175,18 @@
);
if ( !empty( $this->preferredPageImagesProperties ) ) {
- $dataUpdates[] = new PageImagesDataUpdater(
$this->preferredPageImagesProperties );
+ $dataUpdaters[] = new PageImagesDataUpdater(
$this->preferredPageImagesProperties );
}
if ( class_exists( 'GeoData' ) ) {
- $dataUpdates[] = new GeoDataDataUpdater(
+ $dataUpdaters[] = new GeoDataDataUpdater(
$propertyDataTypeMatcher,
$this->preferredGeoDataProperties,
$this->globeUris
);
}
- return $dataUpdates;
+ return $dataUpdaters;
}
}
diff --git
a/repo/tests/phpunit/includes/ParserOutput/EntityParserOutputDataUpdaterTest.php
b/repo/tests/phpunit/includes/ParserOutput/EntityParserOutputDataUpdaterTest.php
index b573199..19b0d8e 100644
---
a/repo/tests/phpunit/includes/ParserOutput/EntityParserOutputDataUpdaterTest.php
+++
b/repo/tests/phpunit/includes/ParserOutput/EntityParserOutputDataUpdaterTest.php
@@ -29,21 +29,21 @@
->disableOriginalConstructor()
->getMock();
- $statementDataUpdate = $this->getMock(
'Wikibase\Repo\ParserOutput\StatementDataUpdater' );
- $statementDataUpdate->expects( $this->exactly( $statements ) )
+ $statementDataUpdater = $this->getMock(
'Wikibase\Repo\ParserOutput\StatementDataUpdater' );
+ $statementDataUpdater->expects( $this->exactly( $statements ) )
->method( 'processStatement' );
- $statementDataUpdate->expects( $this->once() )
+ $statementDataUpdater->expects( $this->once() )
->method( 'updateParserOutput' );
- $siteLinkDataUpdate = $this->getMock(
'Wikibase\Repo\ParserOutput\SiteLinkDataUpdater' );
- $siteLinkDataUpdate->expects( $this->exactly( $siteLinks ) )
+ $siteLinkDataUpdater = $this->getMock(
'Wikibase\Repo\ParserOutput\SiteLinkDataUpdater' );
+ $siteLinkDataUpdater->expects( $this->exactly( $siteLinks ) )
->method( 'processSiteLink' );
- $siteLinkDataUpdate->expects( $this->once() )
+ $siteLinkDataUpdater->expects( $this->once() )
->method( 'updateParserOutput' );
$instance = new EntityParserOutputDataUpdater( $parserOutput,
array(
- $statementDataUpdate,
- $siteLinkDataUpdate,
+ $statementDataUpdater,
+ $siteLinkDataUpdater,
) );
foreach ( $entities as $entity ) {
$instance->processEntity( $entity );
diff --git
a/repo/tests/phpunit/includes/ParserOutput/EntityParserOutputGeneratorTest.php
b/repo/tests/phpunit/includes/ParserOutput/EntityParserOutputGeneratorTest.php
index f9d86ff..e72d82e 100644
---
a/repo/tests/phpunit/includes/ParserOutput/EntityParserOutputGeneratorTest.php
+++
b/repo/tests/phpunit/includes/ParserOutput/EntityParserOutputGeneratorTest.php
@@ -143,7 +143,7 @@
$propertyDataTypeMatcher = new PropertyDataTypeMatcher(
$this->getPropertyDataTypeLookup() );
- $dataUpdates = array(
+ $dataUpdaters = array(
new ExternalLinksDataUpdater( $propertyDataTypeMatcher
),
new ImageLinksDataUpdater( $propertyDataTypeMatcher ),
new ReferencedEntitiesDataUpdater(
@@ -160,7 +160,7 @@
$this->newLanguageFallbackChain(),
TemplateFactory::getDefaultInstance(),
$entityDataFormatProvider,
- $dataUpdates,
+ $dataUpdaters,
'en'
);
}
diff --git
a/repo/tests/phpunit/includes/ParserOutput/GeoDataDataUpdaterTest.php
b/repo/tests/phpunit/includes/ParserOutput/GeoDataDataUpdaterTest.php
index 6d0d03a..8cc236d 100644
--- a/repo/tests/phpunit/includes/ParserOutput/GeoDataDataUpdaterTest.php
+++ b/repo/tests/phpunit/includes/ParserOutput/GeoDataDataUpdaterTest.php
@@ -52,18 +52,18 @@
* @dataProvider processStatementProvider
*/
public function testProcessStatement( array $expected, array
$statements, $message ) {
- $dataUpdate = $this->newGeoDataDataUpdate(
+ $dataUpdater = $this->newGeoDataDataUpdate(
array( 'P625', 'P9000' )
);
foreach ( $statements as $statement ) {
- $dataUpdate->processStatement( $statement );
+ $dataUpdater->processStatement( $statement );
}
$this->assertAttributeEquals(
$expected,
'coordinates',
- $dataUpdate,
+ $dataUpdater,
$message
);
}
@@ -160,12 +160,12 @@
}
public function
testUpdateParserOutput_withPrimaryCoordPreferredStatement() {
- $dataUpdate = $this->newGeoDataDataUpdate(
+ $dataUpdater = $this->newGeoDataDataUpdate(
array( 'P9000', 'P625' )
);
foreach ( $this->getStatements() as $statement ) {
- $dataUpdate->processStatement( $statement );
+ $dataUpdater->processStatement( $statement );
}
$coords = $this->getCoords();
@@ -183,18 +183,18 @@
}
$parserOutput = new ParserOutput();
- $dataUpdate->updateParserOutput( $parserOutput );
+ $dataUpdater->updateParserOutput( $parserOutput );
$this->assertEquals( $expected, $parserOutput->geoData );
}
public function
testUpdateParserOutput_withPrimaryCoordNormalStatement() {
- $dataUpdate = $this->newGeoDataDataUpdate(
+ $dataUpdater = $this->newGeoDataDataUpdate(
array( 'P625', 'P10' )
);
foreach ( $this->getStatements() as $statement ) {
- $dataUpdate->processStatement( $statement );
+ $dataUpdater->processStatement( $statement );
}
$expected = new CoordinatesOutput();
@@ -211,18 +211,18 @@
}
$parserOutput = new ParserOutput();
- $dataUpdate->updateParserOutput( $parserOutput );
+ $dataUpdater->updateParserOutput( $parserOutput );
$this->assertEquals( $expected, $parserOutput->geoData );
}
public function testUpdateParserOutput_noPrimaryCoord() {
- $dataUpdate = $this->newGeoDataDataUpdate(
+ $dataUpdater = $this->newGeoDataDataUpdate(
array( 'P17', 'P404', 'P10', 'P20', 'P9000', 'P9001',
'P625' )
);
foreach ( $this->getStatements() as $statement ) {
- $dataUpdate->processStatement( $statement );
+ $dataUpdater->processStatement( $statement );
}
$expected = new CoordinatesOutput();
@@ -232,7 +232,7 @@
}
$parserOutput = new ParserOutput();
- $dataUpdate->updateParserOutput( $parserOutput );
+ $dataUpdater->updateParserOutput( $parserOutput );
$this->assertEquals( $expected, $parserOutput->geoData );
}
--
To view, visit https://gerrit.wikimedia.org/r/249437
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibfd576d69cf77550c1a1175e2d690b6f305389e9
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Thiemo Mättig (WMDE) <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
