Ataherivand has submitted this change and it was merged.
Change subject: Update SiteLink usage in ChangeHandlerTest
......................................................................
Update SiteLink usage in ChangeHandlerTest
Change-Id: I945b13cb72c8132bdb9a3f72a24b23bdb45a4cc3
---
M client/tests/phpunit/includes/ChangeHandlerTest.php
1 file changed, 4 insertions(+), 5 deletions(-)
Approvals:
Ataherivand: Looks good to me, approved
jenkins-bot: Verified
diff --git a/client/tests/phpunit/includes/ChangeHandlerTest.php
b/client/tests/phpunit/includes/ChangeHandlerTest.php
index f4e4fe7..095f13e 100644
--- a/client/tests/phpunit/includes/ChangeHandlerTest.php
+++ b/client/tests/phpunit/includes/ChangeHandlerTest.php
@@ -3,6 +3,7 @@
namespace Wikibase\Test;
use Site;
use Wikibase\ChangeHandler;
+use Wikibase\DataModel\SimpleSiteLink;
use Wikibase\EntityUsageIndex;
use Wikibase\Item;
use Wikibase\Property;
@@ -1108,14 +1109,12 @@
foreach ( $v as $siteId => $page ) {
if ( is_int( $siteId ) ) {
- $site = $this->site;
+ $siteIdentifier =
$this->site->getGlobalId();
} else {
- $site = new \MediaWikiSite();
- $site->setGlobalId( $siteId );
+ $siteIdentifier = $siteId;
}
- $link = new \Wikibase\SiteLink( $site,
$page );
- $entity->addSiteLink( $link, "add" );
+ $entity->addSimpleSiteLink( new
SimpleSiteLink( $siteIdentifier, $page ) );
}
} else {
$entity = $v;
--
To view, visit https://gerrit.wikimedia.org/r/67500
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I945b13cb72c8132bdb9a3f72a24b23bdb45a4cc3
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Jeroen De Dauw <[email protected]>
Gerrit-Reviewer: Ataherivand <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
