Hoo man has submitted this change and it was merged.

Change subject: Rework deprecated code in TestChanges
......................................................................


Rework deprecated code in TestChanges

This avoids:
* type hinting against Entity,
* calling deprecated methods.

Change-Id: Id4aaee764c27b16fe346bb5a0337ce34bb8e6ce5
---
M lib/tests/phpunit/changes/TestChanges.php
1 file changed, 6 insertions(+), 9 deletions(-)

Approvals:
  Hoo man: Looks good to me, approved



diff --git a/lib/tests/phpunit/changes/TestChanges.php 
b/lib/tests/phpunit/changes/TestChanges.php
index fcadf65..d64249e 100644
--- a/lib/tests/phpunit/changes/TestChanges.php
+++ b/lib/tests/phpunit/changes/TestChanges.php
@@ -6,7 +6,6 @@
 use Wikibase\ChangeRow;
 use Wikibase\ChangesTable;
 use Wikibase\DataModel\Claim\Claim;
-use Wikibase\DataModel\Entity\Entity;
 use Wikibase\DataModel\Entity\Item;
 use Wikibase\DataModel\Entity\ItemId;
 use Wikibase\DataModel\Entity\Property;
@@ -15,6 +14,7 @@
 use Wikibase\DataModel\Snak\PropertyNoValueSnak;
 use Wikibase\DataModel\Statement\Statement;
 use Wikibase\DataModel\Statement\StatementList;
+use Wikibase\DataModel\Term\FingerprintProvider;
 use Wikibase\DiffChange;
 use Wikibase\EntityChange;
 use Wikibase\EntityFactory;
@@ -300,19 +300,16 @@
        public static function getEntities() {
                $entityList = array();
 
+               /** @var FingerprintProvider[] $entities */
                $entities = array(
-                       Item::newEmpty(),
-                       Property::newFromType( 'string' ),
+                       new Item( new ItemId( 'Q112' ) ),
+                       new Property( new PropertyId( 'P112' ), null, 'string' 
),
                );
 
-               /**
-                * @var Entity $entity
-                */
-               foreach( $entities as $entity ) {
+               foreach ( $entities as $entity ) {
                        $entityList[] = $entity;
 
-                       $entity->setId( 112 );
-                       $entity->setLabel( 'ja', '\u30d3\u30fc\u30eb' );
+                       $entity->getFingerprint()->setLabel( 'ja', 
'\u30d3\u30fc\u30eb' );
 
                        $entityList[] = $entity;
                }

-- 
To view, visit https://gerrit.wikimedia.org/r/189739
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id4aaee764c27b16fe346bb5a0337ce34bb8e6ce5
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Thiemo Mättig (WMDE) <[email protected]>
Gerrit-Reviewer: Aude <[email protected]>
Gerrit-Reviewer: Hoo man <[email protected]>
Gerrit-Reviewer: Jeroen De Dauw <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to