Jeroen De Dauw has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/175760

Change subject: Rename class from LanguageLinkBadgeDisplay to 
SiteLinkBadgeDisplay
......................................................................

Rename class from LanguageLinkBadgeDisplay to SiteLinkBadgeDisplay

Not 100% sure this was wrong, please verify

Change-Id: I7b46000b3d1ef5b95e2c63caeb0e0efdb3cec326
---
M client/includes/LangLinkHandler.php
M client/includes/WikibaseClient.php
M client/includes/hooks/SidebarHookHandlers.php
R client/includes/hooks/SiteLinkBadgeDisplay.php
M client/tests/phpunit/includes/LangLinkHandlerTest.php
M client/tests/phpunit/includes/hooks/ParserAfterParserHookHandlerTest.php
M client/tests/phpunit/includes/hooks/SidebarHookHandlersTest.php
R client/tests/phpunit/includes/hooks/SiteLinkBadgeDisplayTest.php
8 files changed, 21 insertions(+), 21 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase 
refs/changes/60/175760/1

diff --git a/client/includes/LangLinkHandler.php 
b/client/includes/LangLinkHandler.php
index 0cb47ab..773f17f 100644
--- a/client/includes/LangLinkHandler.php
+++ b/client/includes/LangLinkHandler.php
@@ -6,7 +6,7 @@
 use Site;
 use SiteList;
 use Title;
-use Wikibase\Client\Hooks\LanguageLinkBadgeDisplay;
+use Wikibase\Client\Hooks\SiteLinkBadgeDisplay;
 use Wikibase\Client\Hooks\OtherProjectsSidebarGeneratorFactory;
 use Wikibase\Client\Usage\ParserOutputUsageAccumulator;
 use Wikibase\DataModel\Entity\Item;
@@ -34,7 +34,7 @@
        private $otherProjectsSidebarGeneratorFactory;
 
        /**
-        * @var LanguageLinkBadgeDisplay
+        * @var SiteLinkBadgeDisplay
         */
        private $badgeDisplay;
 
@@ -75,7 +75,7 @@
 
        /**
         * @param OtherProjectsSidebarGeneratorFactory 
$otherProjectsSidebarGeneratorFactory
-        * @param LanguageLinkBadgeDisplay $badgeDisplay
+        * @param SiteLinkBadgeDisplay $badgeDisplay
         * @param string $siteId The global site ID for the local wiki
         * @param NamespaceChecker $namespaceChecker determines which 
namespaces wikibase is enabled on
         * @param SiteLinkLookup $siteLinkLookup A site link lookup service
@@ -85,7 +85,7 @@
         */
        public function __construct(
                OtherProjectsSidebarGeneratorFactory 
$otherProjectsSidebarGeneratorFactory,
-               LanguageLinkBadgeDisplay $badgeDisplay,
+               SiteLinkBadgeDisplay $badgeDisplay,
                $siteId,
                NamespaceChecker $namespaceChecker,
                SiteLinkLookup $siteLinkLookup,
diff --git a/client/includes/WikibaseClient.php 
b/client/includes/WikibaseClient.php
index 5c1cbcf..1dd04e8 100644
--- a/client/includes/WikibaseClient.php
+++ b/client/includes/WikibaseClient.php
@@ -17,7 +17,7 @@
 use Wikibase\Client\Changes\ChangeHandler;
 use Wikibase\Client\Changes\ChangeRunCoalescer;
 use Wikibase\Client\Changes\WikiPageUpdater;
-use Wikibase\Client\Hooks\LanguageLinkBadgeDisplay;
+use Wikibase\Client\Hooks\SiteLinkBadgeDisplay;
 use Wikibase\Client\Hooks\OtherProjectsSidebarGeneratorFactory;
 use Wikibase\Client\Hooks\ParserFunctionRegistrant;
 use Wikibase\Client\Store\TitleFactory;
@@ -539,14 +539,14 @@
        }
 
        /**
-        * @return LanguageLinkBadgeDisplay
+        * @return SiteLinkBadgeDisplay
         */
        public function getLanguageLinkBadgeDisplay() {
                global $wgLang;
 
                $badgeClassNames = $this->settings->getSetting( 
'badgeClassNames' );
 
-               return new LanguageLinkBadgeDisplay(
+               return new SiteLinkBadgeDisplay(
                        $this->getEntityLookup(),
                        is_array( $badgeClassNames ) ? $badgeClassNames : 
array(),
                        $wgLang
diff --git a/client/includes/hooks/SidebarHookHandlers.php 
b/client/includes/hooks/SidebarHookHandlers.php
index a703555..852cf47 100644
--- a/client/includes/hooks/SidebarHookHandlers.php
+++ b/client/includes/hooks/SidebarHookHandlers.php
@@ -35,7 +35,7 @@
        private $namespaceChecker;
 
        /**
-        * @var LanguageLinkBadgeDisplay
+        * @var SiteLinkBadgeDisplay
         */
        private $badgeDisplay;
 
@@ -61,7 +61,7 @@
                $entityLookup = $wikibaseClient->getStore()->getEntityLookup();
                $badgeClassNames = $settings->getSetting( 'badgeClassNames' );
 
-               $badgeDisplay = new LanguageLinkBadgeDisplay(
+               $badgeDisplay = new SiteLinkBadgeDisplay(
                        $entityLookup,
                        is_array( $badgeClassNames ) ? $badgeClassNames : 
array(),
                        $wgLang
@@ -120,7 +120,7 @@
 
        public function __construct(
                NamespaceChecker $namespaceChecker,
-               LanguageLinkBadgeDisplay $badgeDisplay,
+               SiteLinkBadgeDisplay $badgeDisplay,
                OtherProjectsSidebarGeneratorFactory 
$otherProjectsSidebarGeneratorFactory,
                $otherProjectsLinksBeta,
                $otherProjectsLinksDefault
diff --git a/client/includes/hooks/LanguageLinkBadgeDisplay.php 
b/client/includes/hooks/SiteLinkBadgeDisplay.php
similarity index 99%
rename from client/includes/hooks/LanguageLinkBadgeDisplay.php
rename to client/includes/hooks/SiteLinkBadgeDisplay.php
index e0a666f..95b9888 100644
--- a/client/includes/hooks/LanguageLinkBadgeDisplay.php
+++ b/client/includes/hooks/SiteLinkBadgeDisplay.php
@@ -21,7 +21,7 @@
  * @author Bene* < benestar.wikime...@gmail.com >
  * @author Daniel Kinzler
  */
-class LanguageLinkBadgeDisplay {
+class SiteLinkBadgeDisplay {
 
        /**
         * @var EntityLookup
diff --git a/client/tests/phpunit/includes/LangLinkHandlerTest.php 
b/client/tests/phpunit/includes/LangLinkHandlerTest.php
index e98fb3e..48198be 100644
--- a/client/tests/phpunit/includes/LangLinkHandlerTest.php
+++ b/client/tests/phpunit/includes/LangLinkHandlerTest.php
@@ -5,7 +5,7 @@
 use MediaWikiSite;
 use ParserOutput;
 use Title;
-use Wikibase\Client\Hooks\LanguageLinkBadgeDisplay;
+use Wikibase\Client\Hooks\SiteLinkBadgeDisplay;
 use Wikibase\Client\Hooks\OtherProjectsSidebarGenerator;
 use Wikibase\Client\Usage\EntityUsage;
 use Wikibase\Client\Usage\ParserOutputUsageAccumulator;
@@ -127,7 +127,7 @@
        }
 
        /**
-        * @return LanguageLinkBadgeDisplay
+        * @return SiteLinkBadgeDisplay
         */
        private function getLanguageLinkBadgeDisplay() {
                $badgeDisplay = $this->getMockBuilder( 
'Wikibase\Client\Hooks\LanguageLinkBadgeDisplay' )
diff --git 
a/client/tests/phpunit/includes/hooks/ParserAfterParserHookHandlerTest.php 
b/client/tests/phpunit/includes/hooks/ParserAfterParserHookHandlerTest.php
index 8ba63b9..7ba0fda 100644
--- a/client/tests/phpunit/includes/hooks/ParserAfterParserHookHandlerTest.php
+++ b/client/tests/phpunit/includes/hooks/ParserAfterParserHookHandlerTest.php
@@ -11,7 +11,7 @@
 use SiteStore;
 use StripState;
 use Title;
-use Wikibase\Client\Hooks\LanguageLinkBadgeDisplay;
+use Wikibase\Client\Hooks\SiteLinkBadgeDisplay;
 use Wikibase\Client\Hooks\OtherProjectsSidebarGeneratorFactory;
 use Wikibase\Client\Hooks\ParserAfterParseHookHandler;
 use Wikibase\Client\Usage\EntityUsage;
@@ -165,7 +165,7 @@
                $mockRepo = $this->getMockRepo( $links );
                $mockRepo->putEntity( $this->getBadgeItem() );
 
-               $badgeDisplay = new LanguageLinkBadgeDisplay(
+               $badgeDisplay = new SiteLinkBadgeDisplay(
                        $mockRepo,
                        array( 'Q17' => 'featured' ),
                        Language::factory( 'en' )
diff --git a/client/tests/phpunit/includes/hooks/SidebarHookHandlersTest.php 
b/client/tests/phpunit/includes/hooks/SidebarHookHandlersTest.php
index 19a00b9..76874b5 100644
--- a/client/tests/phpunit/includes/hooks/SidebarHookHandlersTest.php
+++ b/client/tests/phpunit/includes/hooks/SidebarHookHandlersTest.php
@@ -13,7 +13,7 @@
 use SiteStore;
 use Skin;
 use Title;
-use Wikibase\Client\Hooks\LanguageLinkBadgeDisplay;
+use Wikibase\Client\Hooks\SiteLinkBadgeDisplay;
 use Wikibase\Client\Hooks\OtherProjectsSidebarGeneratorFactory;
 use Wikibase\Client\Hooks\SidebarHookHandlers;
 use Wikibase\Client\Usage\EntityUsage;
@@ -204,7 +204,7 @@
                $mockRepo = $this->getMockRepository( $siteLinksPerItem );
                $mockRepo->putEntity( $this->getBadgeItem() );
 
-               $badgeDisplay = new LanguageLinkBadgeDisplay(
+               $badgeDisplay = new SiteLinkBadgeDisplay(
                        $mockRepo,
                        array( 'Q17' => 'featured' ),
                        $en
diff --git 
a/client/tests/phpunit/includes/hooks/LanguageLinkBadgeDisplayTest.php 
b/client/tests/phpunit/includes/hooks/SiteLinkBadgeDisplayTest.php
similarity index 94%
rename from client/tests/phpunit/includes/hooks/LanguageLinkBadgeDisplayTest.php
rename to client/tests/phpunit/includes/hooks/SiteLinkBadgeDisplayTest.php
index e328348..8c6244a 100644
--- a/client/tests/phpunit/includes/hooks/LanguageLinkBadgeDisplayTest.php
+++ b/client/tests/phpunit/includes/hooks/SiteLinkBadgeDisplayTest.php
@@ -8,14 +8,14 @@
 use ParserOutput;
 use RequestContext;
 use Title;
-use Wikibase\Client\Hooks\LanguageLinkBadgeDisplay;
+use Wikibase\Client\Hooks\SiteLinkBadgeDisplay;
 use Wikibase\DataModel\Entity\Item;
 use Wikibase\DataModel\Entity\ItemId;
 use Wikibase\DataModel\SiteLink;
 use Wikibase\Test\MockRepository;
 
 /**
- * @covers Wikibase\Client\Hooks\LanguageLinkBadgeDisplay
+ * @covers Wikibase\Client\Hooks\SiteLinkBadgeDisplay
  *
  * @since 0.5
  *
@@ -26,7 +26,7 @@
  * @licence GNU GPL v2+
  * @author Bene* < benestar.wikime...@gmail.com >
  */
-class LanguageLinkBadgeDisplayTest extends \MediaWikiTestCase {
+class SiteLinkBadgeDisplayTest extends \MediaWikiTestCase {
 
        private function getItems() {
                $items = array();
@@ -70,7 +70,7 @@
 
                $badgeClassNames = array( 'Q4' => 'foo', 'Q3' => 'bar' );
 
-               return new LanguageLinkBadgeDisplay(
+               return new SiteLinkBadgeDisplay(
                        $mockRepo,
                        $badgeClassNames,
                        Language::factory( 'de' )

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7b46000b3d1ef5b95e2c63caeb0e0efdb3cec326
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Jeroen De Dauw <jeroended...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to