Hoo man has submitted this change and it was merged.

Change subject: Fix usage of SimpleSiteLink in info action hook 
......................................................................


Fix usage of SimpleSiteLink in info action hook 

Change-Id: I385a84a8693a8a3299dd4752de54c335f6a40803
---
M client/WikibaseClient.hooks.php
1 file changed, 1 insertion(+), 3 deletions(-)

Approvals:
  Jeroen De Dauw: Verified; Looks good to me, approved



diff --git a/client/WikibaseClient.hooks.php b/client/WikibaseClient.hooks.php
index 23acc6a..3685087 100644
--- a/client/WikibaseClient.hooks.php
+++ b/client/WikibaseClient.hooks.php
@@ -646,11 +646,9 @@
 
                if ( $title->exists() && $namespaceChecker->isWikibaseEnabled( 
$title->getNamespace() ) ) {
 
-                       $site = \MediaWikiSite::newFromGlobalId( Settings::get( 
'siteGlobalID' ) );
-
                        $siteLinkLookup = 
WikibaseClient::getDefaultInstance()->getStore()->getSiteLinkTable();
                        $entityId = $siteLinkLookup->getEntityIdForSiteLink(
-                               new SimpleSiteLink( $site, 
$title->getFullText() )
+                               new SimpleSiteLink( Settings::get( 
'siteGlobalID' ), $title->getFullText() )
                        );
 
                        if( $entityId ) {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I385a84a8693a8a3299dd4752de54c335f6a40803
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Aude <[email protected]>
Gerrit-Reviewer: Hoo man <[email protected]>
Gerrit-Reviewer: Jeroen De Dauw <[email protected]>

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

Reply via email to