jenkins-bot has submitted this change and it was merged.
Change subject: Remove unused vars
......................................................................
Remove unused vars
Change-Id: Ie78b7d79933e3159765a4d69a4f1ed643085e866
---
M repo/includes/api/ApiErrorReporter.php
M repo/includes/store/sql/EntityPerPageBuilder.php
M repo/includes/store/sql/ItemsPerSiteBuilder.php
M repo/includes/store/sql/PropertyInfoTableBuilder.php
M repo/includes/store/sql/TermSearchKeyBuilder.php
5 files changed, 5 insertions(+), 5 deletions(-)
Approvals:
WikidataJenkins: Verified
Addshore: Looks good to me, approved
jenkins-bot: Verified
diff --git a/repo/includes/api/ApiErrorReporter.php
b/repo/includes/api/ApiErrorReporter.php
index 6d2e0a1..bbace38 100644
--- a/repo/includes/api/ApiErrorReporter.php
+++ b/repo/includes/api/ApiErrorReporter.php
@@ -134,7 +134,7 @@
$this->addStatusToResult( $status, $extradata );
//XXX: when to prefer $statusCode over $errorCode?
- list( $statusCode, $description ) =
$this->apiModule->getErrorFromStatus( $status );
+ list( , $description ) = $this->apiModule->getErrorFromStatus(
$status );
$this->throwUsageException( $description, $errorCode,
$httpRespCode, $extradata );
diff --git a/repo/includes/store/sql/EntityPerPageBuilder.php
b/repo/includes/store/sql/EntityPerPageBuilder.php
index 48a5ac5..2c9183f 100644
--- a/repo/includes/store/sql/EntityPerPageBuilder.php
+++ b/repo/includes/store/sql/EntityPerPageBuilder.php
@@ -247,7 +247,7 @@
$lb = wfGetLB(); //TODO: allow foreign DB, get from $this->table
while ( true ) {
- list( $host, $maxLag ) = $lb->getMaxLag();
+ list( , $maxLag ) = $lb->getMaxLag();
if ( $maxLag < 2 ) {
break;
}
diff --git a/repo/includes/store/sql/ItemsPerSiteBuilder.php
b/repo/includes/store/sql/ItemsPerSiteBuilder.php
index 62d1bc4..faab184 100644
--- a/repo/includes/store/sql/ItemsPerSiteBuilder.php
+++ b/repo/includes/store/sql/ItemsPerSiteBuilder.php
@@ -119,7 +119,7 @@
$lb = wfGetLB(); //TODO: allow foreign DB, get from $this->table
while ( true ) {
- list( $host, $maxLag ) = $lb->getMaxLag();
+ list( , $maxLag ) = $lb->getMaxLag();
if ( $maxLag < 2 ) {
break;
}
diff --git a/repo/includes/store/sql/PropertyInfoTableBuilder.php
b/repo/includes/store/sql/PropertyInfoTableBuilder.php
index 092e9c8..d5825e7 100644
--- a/repo/includes/store/sql/PropertyInfoTableBuilder.php
+++ b/repo/includes/store/sql/PropertyInfoTableBuilder.php
@@ -234,7 +234,7 @@
$lb = wfGetLB(); //TODO: allow foreign DB, get from
$this->propertyInfoTable
while ( true ) {
- list( $host, $maxLag ) = $lb->getMaxLag();
+ list( , $maxLag ) = $lb->getMaxLag();
if ( $maxLag < 2 ) {
break;
}
diff --git a/repo/includes/store/sql/TermSearchKeyBuilder.php
b/repo/includes/store/sql/TermSearchKeyBuilder.php
index 8b98168..7ba2899 100644
--- a/repo/includes/store/sql/TermSearchKeyBuilder.php
+++ b/repo/includes/store/sql/TermSearchKeyBuilder.php
@@ -198,7 +198,7 @@
$lb = wfGetLB(); //TODO: allow foreign DB, get from $this->table
while ( true ) {
- list( $host, $maxLag ) = $lb->getMaxLag();
+ list( , $maxLag ) = $lb->getMaxLag();
if ( $maxLag < 2 ) {
break;
}
--
To view, visit https://gerrit.wikimedia.org/r/153711
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie78b7d79933e3159765a4d69a4f1ed643085e866
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Jeroen De Dauw <[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
