jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/381294 )
Change subject: Move static hook handler to the class
......................................................................
Move static hook handler to the class
It's better to have related code together.
Change-Id: I9712ce13bb3ee2806c622a80e71d494d41166cd1
---
M client/WikibaseClient.hooks.php
M client/WikibaseClient.php
M client/includes/Hooks/InfoActionHookHandler.php
3 files changed, 40 insertions(+), 42 deletions(-)
Approvals:
jenkins-bot: Verified
Thiemo Mättig (WMDE): Looks good to me, approved
diff --git a/client/WikibaseClient.hooks.php b/client/WikibaseClient.hooks.php
index 4853c6d..584b0b6 100644
--- a/client/WikibaseClient.hooks.php
+++ b/client/WikibaseClient.hooks.php
@@ -6,7 +6,6 @@
use BaseTemplate;
use EchoEvent;
use EditPage;
-use IContextSource;
use OutputPage;
use Parser;
use Skin;
@@ -21,14 +20,12 @@
use Wikibase\Client\Hooks\DeletePageNoticeCreator;
use Wikibase\Client\Hooks\EchoNotificationsHandlers;
use Wikibase\Client\Hooks\EditActionHookHandler;
-use Wikibase\Client\Hooks\InfoActionHookHandler;
+use Wikibase\Client\Specials\SpecialEntityUsage;
use Wikibase\Client\Specials\SpecialPagesWithBadges;
use Wikibase\Client\Specials\SpecialUnconnectedPages;
-use Wikibase\Client\Specials\SpecialEntityUsage;
use Wikibase\Client\WikibaseClient;
use Wikibase\DataModel\Entity\EntityId;
use Wikibase\Lib\AutoCommentFormatter;
-use Wikibase\Lib\Store\LanguageFallbackLabelDescriptionLookupFactory;
/**
* File defining the hook handlers for the Wikibase Client extension.
@@ -280,42 +277,6 @@
*/
public static function onParserFirstCallInit( Parser &$parser ) {
WikibaseClient::getDefaultInstance()->getParserFunctionRegistrant()->register(
$parser );
-
- return true;
- }
-
- /**
- * Adds the Entity ID of the corresponding Wikidata item in action=info
- *
- * @param IContextSource $context
- * @param array $pageInfo
- *
- * @return bool
- */
- public static function onInfoAction( IContextSource $context, array
&$pageInfo ) {
- $wikibaseClient = WikibaseClient::getDefaultInstance();
- $settings = $wikibaseClient->getSettings();
-
- $namespaceChecker = $wikibaseClient->getNamespaceChecker();
- $usageLookup = $wikibaseClient->getStore()->getUsageLookup();
- $labelDescriptionLookupFactory = new
LanguageFallbackLabelDescriptionLookupFactory(
- $wikibaseClient->getLanguageFallbackChainFactory(),
- $wikibaseClient->getTermLookup(),
- $wikibaseClient->getTermBuffer()
- );
- $idParser = $wikibaseClient->getEntityIdParser();
-
- $infoActionHookHandler = new InfoActionHookHandler(
- $namespaceChecker,
- $wikibaseClient->newRepoLinker(),
- $wikibaseClient->getStore()->getSiteLinkLookup(),
- $settings->getSetting( 'siteGlobalID' ),
- $usageLookup,
- $labelDescriptionLookupFactory,
- $idParser
- );
-
- $pageInfo = $infoActionHookHandler->handle( $context, $pageInfo
);
return true;
}
diff --git a/client/WikibaseClient.php b/client/WikibaseClient.php
index f71a631..baf0bec 100644
--- a/client/WikibaseClient.php
+++ b/client/WikibaseClient.php
@@ -139,7 +139,7 @@
$wgHooks['BeforePageDisplay'][] =
'\Wikibase\ClientHooks::onBeforePageDisplay';
$wgHooks['BeforePageDisplay'][] =
'\Wikibase\ClientHooks::onBeforePageDisplayAddJsConfig';
$wgHooks['ScribuntoExternalLibraries'][] =
'\Wikibase\ClientHooks::onScribuntoExternalLibraries';
- $wgHooks['InfoAction'][] = '\Wikibase\ClientHooks::onInfoAction';
+ $wgHooks['InfoAction'][] =
'\Wikibase\Client\Hooks\InfoActionHookHandler::onInfoAction';
$wgHooks['EditPage::showStandardInputs:options'][] =
'\Wikibase\ClientHooks::onEditAction';
$wgHooks['BaseTemplateAfterPortlet'][] =
'\Wikibase\ClientHooks::onBaseTemplateAfterPortlet';
$wgHooks['GetBetaFeaturePreferences'][] =
'\Wikibase\ClientHooks::onGetBetaFeaturePreferences';
diff --git a/client/includes/Hooks/InfoActionHookHandler.php
b/client/includes/Hooks/InfoActionHookHandler.php
index a87e6d5..2478b89 100644
--- a/client/includes/Hooks/InfoActionHookHandler.php
+++ b/client/includes/Hooks/InfoActionHookHandler.php
@@ -9,9 +9,10 @@
use Wikibase\Client\Usage\UsageLookup;
use Wikibase\DataModel\Entity\EntityIdParser;
use Wikibase\DataModel\Entity\ItemId;
-use Wikibase\Lib\Store\SiteLinkLookup;
use Wikibase\Lib\Store\LanguageFallbackLabelDescriptionLookupFactory;
+use Wikibase\Lib\Store\SiteLinkLookup;
use Wikibase\Client\NamespaceChecker;
+use Wikibase\Client\WikibaseClient;
/**
* @license GPL-2.0+
@@ -73,6 +74,42 @@
}
/**
+ * Adds the Entity ID of the corresponding Wikidata item in action=info
+ *
+ * @param IContextSource $context
+ * @param array $pageInfo
+ *
+ * @return bool
+ */
+ public static function onInfoAction( IContextSource $context, array
&$pageInfo ) {
+ $wikibaseClient = WikibaseClient::getDefaultInstance();
+ $settings = $wikibaseClient->getSettings();
+
+ $namespaceChecker = $wikibaseClient->getNamespaceChecker();
+ $usageLookup = $wikibaseClient->getStore()->getUsageLookup();
+ $labelDescriptionLookupFactory = new
LanguageFallbackLabelDescriptionLookupFactory(
+ $wikibaseClient->getLanguageFallbackChainFactory(),
+ $wikibaseClient->getTermLookup(),
+ $wikibaseClient->getTermBuffer()
+ );
+ $idParser = $wikibaseClient->getEntityIdParser();
+
+ $self = new self(
+ $namespaceChecker,
+ $wikibaseClient->newRepoLinker(),
+ $wikibaseClient->getStore()->getSiteLinkLookup(),
+ $settings->getSetting( 'siteGlobalID' ),
+ $usageLookup,
+ $labelDescriptionLookupFactory,
+ $idParser
+ );
+
+ $pageInfo = $self->handle( $context, $pageInfo );
+
+ return true;
+ }
+
+ /**
* @param IContextSource $context
* @param array $pageInfo
*
--
To view, visit https://gerrit.wikimedia.org/r/381294
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9712ce13bb3ee2806c622a80e71d494d41166cd1
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Matěj Suchánek <matejsuchane...@gmail.com>
Gerrit-Reviewer: Aleksey Bekh-Ivanov (WMDE) <aleksey.bekh-iva...@wikimedia.de>
Gerrit-Reviewer: Aude <aude.w...@gmail.com>
Gerrit-Reviewer: Thiemo Mättig (WMDE) <thiemo.maet...@wikimedia.de>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
