From d81f562b712f2387fa02290bf2ca86392ab356f2 Mon Sep 17 00:00:00 2001 From: Pierre Schmitz Date: Wed, 11 Oct 2006 20:21:25 +0000 Subject: Aktualisierung auf Version 1.8.1 --- includes/FileStore.php | 32 +++++++++++--------------------- 1 file changed, 11 insertions(+), 21 deletions(-) (limited to 'includes/FileStore.php') diff --git a/includes/FileStore.php b/includes/FileStore.php index 85aaedfe..35ebd554 100644 --- a/includes/FileStore.php +++ b/includes/FileStore.php @@ -36,18 +36,16 @@ class FileStore { * @fixme Probably only works on MySQL. Abstract to the Database class? */ static function lock() { - $fname = __CLASS__ . '::' . __FUNCTION__; - $dbw = wfGetDB( DB_MASTER ); $lockname = $dbw->addQuotes( FileStore::lockName() ); - $result = $dbw->query( "SELECT GET_LOCK($lockname, 5) AS lockstatus", $fname ); + $result = $dbw->query( "SELECT GET_LOCK($lockname, 5) AS lockstatus", __METHOD__ ); $row = $dbw->fetchObject( $result ); $dbw->freeResult( $result ); if( $row->lockstatus == 1 ) { return true; } else { - wfDebug( "$fname failed to acquire lock\n" ); + wfDebug( __METHOD__." failed to acquire lock\n" ); return false; } } @@ -56,18 +54,15 @@ class FileStore { * Release the global file store lock. */ static function unlock() { - $fname = __CLASS__ . '::' . __FUNCTION__; - $dbw = wfGetDB( DB_MASTER ); $lockname = $dbw->addQuotes( FileStore::lockName() ); - $result = $dbw->query( "SELECT RELEASE_LOCK($lockname)", $fname ); + $result = $dbw->query( "SELECT RELEASE_LOCK($lockname)", __METHOD__ ); $row = $dbw->fetchObject( $result ); $dbw->freeResult( $result ); } private static function lockName() { - global $wgDBname, $wgDBprefix; - return "MediaWiki.{$wgDBname}.{$wgDBprefix}FileStore"; + return 'MediaWiki.' . wfWikiID() . '.FileStore'; } /** @@ -103,8 +98,6 @@ class FileStore { } private function copyFile( $sourcePath, $destPath, $flags=0 ) { - $fname = __CLASS__ . '::' . __FUNCTION__; - if( !file_exists( $sourcePath ) ) { // Abort! Abort! throw new FSException( "missing source file '$sourcePath'\n" ); @@ -135,11 +128,11 @@ class FileStore { wfRestoreWarnings(); if( $ok ) { - wfDebug( "$fname copied '$sourcePath' to '$destPath'\n" ); + wfDebug( __METHOD__." copied '$sourcePath' to '$destPath'\n" ); $transaction->addRollback( FSTransaction::DELETE_FILE, $destPath ); } else { throw new FSException( - "$fname failed to copy '$sourcePath' to '$destPath'\n" ); + __METHOD__." failed to copy '$sourcePath' to '$destPath'\n" ); } } @@ -239,13 +232,11 @@ class FileStore { * @return string or false if could not open file or bad extension */ static function calculateKey( $path, $extension ) { - $fname = __CLASS__ . '::' . __FUNCTION__; - wfSuppressWarnings(); $hash = sha1_file( $path ); wfRestoreWarnings(); if( $hash === false ) { - wfDebug( "$fname: couldn't hash file '$path'\n" ); + wfDebug( __METHOD__.": couldn't hash file '$path'\n" ); return false; } @@ -260,7 +251,7 @@ class FileStore { if( self::validKey( $key ) ) { return $key; } else { - wfDebug( "$fname: generated bad key '$key'\n" ); + wfDebug( __METHOD__.": generated bad key '$key'\n" ); return false; } } @@ -353,7 +344,6 @@ class FSTransaction { } private function apply( $actions ) { - $fname = __CLASS__ . '::' . __FUNCTION__; $result = true; foreach( $actions as $item ) { list( $action, $path ) = $item; @@ -362,9 +352,9 @@ class FSTransaction { $ok = unlink( $path ); wfRestoreWarnings(); if( $ok ) - wfDebug( "$fname: deleting file '$path'\n" ); + wfDebug( __METHOD__.": deleting file '$path'\n" ); else - wfDebug( "$fname: failed to delete file '$path'\n" ); + wfDebug( __METHOD__.": failed to delete file '$path'\n" ); $result = $result && $ok; } } @@ -374,4 +364,4 @@ class FSTransaction { class FSException extends MWException { } -?> \ No newline at end of file +?> -- cgit v1.2.3-54-g00ecf