Siebrand has uploaded a new change for review.
https://gerrit.wikimedia.org/r/97323
Change subject: Fix various type and reference issues
......................................................................
Fix various type and reference issues
Change-Id: Iced174d9f94faf80c5fa21e15fa6c81b6d8fa21c
---
M includes/filebackend/SwiftFileBackend.php
M includes/filebackend/TempFSFile.php
M includes/filebackend/lockmanager/FSLockManager.php
M includes/filebackend/lockmanager/LockManager.php
M includes/filebackend/lockmanager/MemcLockManager.php
M includes/filebackend/lockmanager/RedisLockManager.php
6 files changed, 10 insertions(+), 10 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/23/97323/1
diff --git a/includes/filebackend/SwiftFileBackend.php
b/includes/filebackend/SwiftFileBackend.php
index 5c9fdc2..f0bbbe5 100644
--- a/includes/filebackend/SwiftFileBackend.php
+++ b/includes/filebackend/SwiftFileBackend.php
@@ -1090,7 +1090,7 @@
try {
$container = $this->getContainer( $fullCont );
$prefix = ( $dir == '' ) ? null : "{$dir}/";
- $objects = array(); // list of unfiltered names or
CF_Object items
+
// Non-recursive: only list files right under $dir
if ( !empty( $params['topOnly'] ) ) {
if ( !empty( $params['adviseStat'] ) ) {
diff --git a/includes/filebackend/TempFSFile.php
b/includes/filebackend/TempFSFile.php
index c0070ca..f06a4cd 100644
--- a/includes/filebackend/TempFSFile.php
+++ b/includes/filebackend/TempFSFile.php
@@ -85,7 +85,7 @@
/**
* Clean up the temporary file only after an object goes out of scope
*
- * @param Object $object
+ * @param FileBackendStoreOpHandle $object
* @return TempFSFile This object
*/
public function bind( $object ) {
diff --git a/includes/filebackend/lockmanager/FSLockManager.php
b/includes/filebackend/lockmanager/FSLockManager.php
index e97ec11..bce6b34 100644
--- a/includes/filebackend/lockmanager/FSLockManager.php
+++ b/includes/filebackend/lockmanager/FSLockManager.php
@@ -102,7 +102,7 @@
/**
* Lock a single resource key
*
- * @param array $path
+ * @param string $path
* @param int $type
* @return Status
*/
@@ -147,7 +147,7 @@
/**
* Unlock a single resource key
*
- * @param array $path
+ * @param string $path
* @param int $type
* @return Status
*/
diff --git a/includes/filebackend/lockmanager/LockManager.php
b/includes/filebackend/lockmanager/LockManager.php
index 7e06e48..df8d2d4 100644
--- a/includes/filebackend/lockmanager/LockManager.php
+++ b/includes/filebackend/lockmanager/LockManager.php
@@ -204,8 +204,8 @@
$lockedByType[$type] = $paths;
} else {
// Release the subset of locks that were
acquired
- foreach ( $lockedByType as $type => $paths ) {
- $status->merge( $this->doUnlock(
$paths, $type ) );
+ foreach ( $lockedByType as $lType => $lPaths ) {
+ $status->merge( $this->doUnlock(
$lPaths, $lType ) );
}
break;
}
diff --git a/includes/filebackend/lockmanager/MemcLockManager.php
b/includes/filebackend/lockmanager/MemcLockManager.php
index 254c693..ccf8613 100644
--- a/includes/filebackend/lockmanager/MemcLockManager.php
+++ b/includes/filebackend/lockmanager/MemcLockManager.php
@@ -100,8 +100,8 @@
? array_merge( $lockedPaths[$type],
$paths )
: $paths;
} else {
- foreach ( $lockedPaths as $type => $paths ) {
- $status->merge(
$this->doFreeLocksOnServer( $lockSrv, $paths, $type ) );
+ foreach ( $lockedPaths as $lType => $lPaths ) {
+ $status->merge(
$this->doFreeLocksOnServer( $lockSrv, $lPaths, $lType ) );
}
break;
}
diff --git a/includes/filebackend/lockmanager/RedisLockManager.php
b/includes/filebackend/lockmanager/RedisLockManager.php
index 5540f61..9d5612a 100644
--- a/includes/filebackend/lockmanager/RedisLockManager.php
+++ b/includes/filebackend/lockmanager/RedisLockManager.php
@@ -90,8 +90,8 @@
? array_merge( $lockedPaths[$type],
$paths )
: $paths;
} else {
- foreach ( $lockedPaths as $type => $paths ) {
- $status->merge(
$this->doFreeLocksOnServer( $lockSrv, $paths, $type ) );
+ foreach ( $lockedPaths as $lType => $lPaths ) {
+ $status->merge(
$this->doFreeLocksOnServer( $lockSrv, $lPaths, $lType ) );
}
break;
}
--
To view, visit https://gerrit.wikimedia.org/r/97323
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iced174d9f94faf80c5fa21e15fa6c81b6d8fa21c
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Siebrand <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
