Addshore has submitted this change and it was merged.
Change subject: Fix style issues, wrap long lines in LangLinkHandler
......................................................................
Fix style issues, wrap long lines in LangLinkHandler
Change-Id: Id7e94612d0810bc9bdde93c0046bc8a8f88cac3f
---
M client/includes/LangLinkHandler.php
1 file changed, 15 insertions(+), 5 deletions(-)
Approvals:
WikidataJenkins: Verified
Addshore: Looks good to me, approved
diff --git a/client/includes/LangLinkHandler.php
b/client/includes/LangLinkHandler.php
index d596994..7dd5c2a 100644
--- a/client/includes/LangLinkHandler.php
+++ b/client/includes/LangLinkHandler.php
@@ -79,7 +79,8 @@
*/
public function getEntityLinks( Title $title ) {
wfProfileIn( __METHOD__ );
- wfDebugLog( __CLASS__, __FUNCTION__ . ": Looking for sitelinks
defined by the corresponding item on the wikibase repo." );
+ wfDebugLog( __CLASS__, __FUNCTION__ . ": Looking for sitelinks
defined by the '
+ . 'corresponding item on the wikibase repo." );
$links = array();
@@ -87,10 +88,13 @@
$itemId = $this->siteLinkLookup->getEntityIdForSiteLink(
$siteLink );
if ( $itemId !== null ) {
- wfDebugLog( __CLASS__, __FUNCTION__ . ": Item ID for "
. $title->getFullText() . " is " . $itemId->getPrefixedId() );
+ wfDebugLog( __CLASS__, __FUNCTION__ . ": Item ID for "
. $title->getFullText()
+ . " is " . $itemId->getPrefixedId() );
+
$links = $this->siteLinkLookup->getSiteLinksForItem(
$itemId );
} else {
- wfDebugLog( __CLASS__, __FUNCTION__ . ": No
corresponding item found for " . $title->getFullText() );
+ wfDebugLog( __CLASS__, __FUNCTION__ . ": No
corresponding item found for "
+ . $title->getFullText() );
}
wfDebugLog( __CLASS__, __FUNCTION__ . ": Found " . count(
$links ) . " links." );
@@ -201,7 +205,9 @@
}
if ( !in_array( $site->getGroup(), $allowedGroups ) ) {
- wfDebugLog( __CLASS__, __FUNCTION__ . ':
skipping link to other group: ' . $wiki . ' belongs to ' . $site->getGroup() );
+ wfDebugLog( __CLASS__, __FUNCTION__ . ':
skipping link to other group: ' . $wiki
+ . ' belongs to ' . $site->getGroup() );
+
unset( $repoLinks[$wiki] );
continue;
}
@@ -430,7 +436,11 @@
public function updateItemIdProperty( Title $title, ParserOutput $out )
{
wfProfileIn( __METHOD__ );
- $entityIdPropertyUpdater = new EntityIdPropertyUpdater(
$this->siteLinkLookup, $this->siteId );
+ $entityIdPropertyUpdater = new EntityIdPropertyUpdater(
+ $this->siteLinkLookup,
+ $this->siteId
+ );
+
$entityIdPropertyUpdater->updateItemIdProperty( $out, $title );
wfProfileOut( __METHOD__ );
--
To view, visit https://gerrit.wikimedia.org/r/119713
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id7e94612d0810bc9bdde93c0046bc8a8f88cac3f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Aude <[email protected]>
Gerrit-Reviewer: Addshore <[email protected]>
Gerrit-Reviewer: WikidataJenkins <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
