Merge "Revert "Adding sanity check to Title::isRedirect().""
[lhc/web/wiklou.git] / includes / upload / UploadStash.php
index 6727fb0..12531c2 100644 (file)
@@ -1,4 +1,26 @@
 <?php
+/**
+ * Temporary storage for uploaded files.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * http://www.gnu.org/copyleft/gpl.html
+ *
+ * @file
+ * @ingroup Upload
+ */
+
 /**
  * UploadStash is intended to accomplish a few things:
  *   - enable applications to temporarily stash files without publishing them to the wiki.
@@ -16,6 +38,8 @@
  * UploadStash represents the entire stash of temporary files.
  * UploadStashFile is a filestore for the actual physical disk files.
  * UploadFromStash extends UploadBase, and represents a single stashed file as it is moved from the stash to the regular file repository
+ *
+ * @ingroup Upload
  */
 class UploadStash {
 
@@ -48,7 +72,7 @@ class UploadStash {
         *
         * @param $repo FileRepo
         */
-       public function __construct( $repo, $user = null ) {
+       public function __construct( FileRepo $repo, $user = null ) {
                // this might change based on wiki's configuration.
                $this->repo = $repo;
 
@@ -91,12 +115,10 @@ class UploadStash {
                        }
                }
 
-               $dbr = $this->repo->getSlaveDb();
-
                if ( !isset( $this->fileMetadata[$key] ) ) {
                        if ( !$this->fetchFileMetadata( $key ) ) {
                                // If nothing was received, it's likely due to replication lag.  Check the master to see if the record is there.
-                               $this->fetchFileMetadata( $key, true );
+                               $this->fetchFileMetadata( $key, DB_MASTER );
                        }
 
                        if ( !isset( $this->fileMetadata[$key] ) ) {
@@ -108,10 +130,7 @@ class UploadStash {
 
                        // fetch fileprops
                        $path = $this->fileMetadata[$key]['us_path'];
-                       if ( $this->repo->isVirtualUrl( $path ) ) {
-                               $path = $this->repo->resolveVirtualUrl( $path );
-                       }
-                       $this->fileProps[$key] = File::getPropsFromPath( $path );
+                       $this->fileProps[$key] = $this->repo->getFileProps( $path );
                }
 
                if ( ! $this->files[$key]->exists() ) {
@@ -155,18 +174,17 @@ class UploadStash {
         *
         * @param $path String: path to file you want stashed
         * @param $sourceType String: the type of upload that generated this file (currently, I believe, 'file' or null)
-        * @param $key String: optional, unique key for this file. Used for directory hashing when storing, otherwise not important
         * @throws UploadStashBadPathException
         * @throws UploadStashFileException
         * @throws UploadStashNotLoggedInException
         * @return UploadStashFile: file, or null on failure
         */
-       public function stashFile( $path, $sourceType = null, $key = null ) {
+       public function stashFile( $path, $sourceType = null ) {
                if ( ! file_exists( $path ) ) {
                        wfDebug( __METHOD__ . " tried to stash file at '$path', but it doesn't exist\n" );
                        throw new UploadStashBadPathException( "path doesn't exist" );
                }
-               $fileProps = File::getPropsFromPath( $path );
+               $fileProps = FSFile::getPropsFromPath( $path );
                wfDebug( __METHOD__ . " stashing file at '$path'\n" );
 
                // we will be initializing from some tmpnam files that don't have extensions.
@@ -181,17 +199,16 @@ class UploadStash {
                }
 
                // If no key was supplied, make one.  a mysql insertid would be totally reasonable here, except
-               // that some users of this function might expect to supply the key instead of using the generated one.
-               if ( is_null( $key ) ) {
-                       // some things that when combined will make a suitably unique key.
-                       // see: http://www.jwz.org/doc/mid.html
-                       list ($usec, $sec) = explode( ' ', microtime() );
-                       $usec = substr($usec, 2);
-                       $key = wfBaseConvert( $sec . $usec, 10, 36 ) . '.' .
-                               wfBaseConvert( mt_rand(), 10, 36 ) . '.'.
-                               $this->userId . '.' . 
-                               $extension;
-               }
+               // that for historical reasons, the key is this random thing instead.  At least it's not guessable.
+               //
+               // some things that when combined will make a suitably unique key.
+               // see: http://www.jwz.org/doc/mid.html
+               list ($usec, $sec) = explode( ' ', microtime() );
+               $usec = substr($usec, 2);
+               $key = wfBaseConvert( $sec . $usec, 10, 36 ) . '.' .
+                       wfBaseConvert( mt_rand(), 10, 36 ) . '.'.
+                       $this->userId . '.' .
+                       $extension;
 
                $this->fileProps[$key] = $fileProps;
 
@@ -219,7 +236,8 @@ class UploadStash {
                                        $error = array( 'unknown', 'no error recorded' );
                                }
                        }
-                       throw new UploadStashFileException( "error storing file in '$path': " . implode( '; ', $error ) );
+                       // at this point, $error should contain the single "most important" error, plus any parameters.
+                       throw new UploadStashFileException( "Error storing file in '$path': " . wfMessage( $error )->text() );
                }
                $stashPath = $storeStatus->value;
 
@@ -232,36 +250,12 @@ class UploadStash {
                wfDebug( __METHOD__ . " inserting $stashPath under $key\n" );
                $dbw = $this->repo->getMasterDb();
 
-               // select happens on the master so this can all be in a transaction, which
-               // avoids a race condition that's likely with multiple people uploading from the same
-               // set of files
-               $dbw->begin();
-               // first, check to see if it's already there.
-               $row = $dbw->selectRow(
-                       'uploadstash',
-                       'us_user, us_timestamp',
-                       array( 'us_key' => $key ),
-                       __METHOD__
-               );
-
-               // The current user can't have this key if:
-               // - the key is owned by someone else and
-               // - the age of the key is less than $wgUploadStashMaxAge
-               if ( is_object( $row ) ) {
-                       global $wgUploadStashMaxAge;
-                       if ( $row->us_user != $this->userId &&
-                               $row->wfTimestamp( TS_UNIX, $row->us_timestamp ) > time() - $wgUploadStashMaxAge
-                       ) {
-                               $dbw->rollback();
-                               throw new UploadStashWrongOwnerException( "Attempting to upload a duplicate of a file that someone else has stashed" );
-                       }
-               }
-
                $this->fileMetadata[$key] = array(
+                       'us_id' => $dbw->nextSequenceValue( 'uploadstash_us_id_seq' ),
                        'us_user' => $this->userId,
                        'us_key' => $key,
                        'us_orig_path' => $path,
-                       'us_path' => $stashPath,
+                       'us_path' => $stashPath, // virtual URL
                        'us_size' => $fileProps['size'],
                        'us_sha1' => $fileProps['sha1'],
                        'us_mime' => $fileProps['mime'],
@@ -274,14 +268,11 @@ class UploadStash {
                        'us_status' => 'finished'
                );
 
-               // if a row exists but previous checks on it passed, let the current user take over this key.
-               $dbw->replace(
+               $dbw->insert(
                        'uploadstash',
-                       'us_key',
                        $this->fileMetadata[$key],
                        __METHOD__
                );
-               $dbw->commit();
 
                // store the insertid in the class variable so immediate retrieval (possibly laggy) isn't necesary.
                $this->fileMetadata[$key]['us_id'] = $dbw->insertId();
@@ -304,7 +295,7 @@ class UploadStash {
                        throw new UploadStashNotLoggedInException( __METHOD__ . ' No user is logged in, files must belong to users' );
                }
 
-               wfDebug( __METHOD__ . " clearing all rows for user $userId\n" );
+               wfDebug( __METHOD__ . ' clearing all rows for user ' . $this->userId . "\n" );
                $dbw = $this->repo->getMasterDb();
                $dbw->delete(
                        'uploadstash',
@@ -365,13 +356,13 @@ class UploadStash {
                $dbw = $this->repo->getMasterDb();
 
                // this gets its own transaction since it's called serially by the cleanupUploadStash maintenance script
-               $dbw->begin();
+               $dbw->begin( __METHOD__ );
                $dbw->delete(
                        'uploadstash',
                        array( 'us_key' => $key ),
                        __METHOD__
                );
-               $dbw->commit();
+               $dbw->commit( __METHOD__ );
 
                // TODO: look into UnregisteredLocalFile and find out why the rv here is sometimes wrong (false when file was removed)
                // for now, ignore.
@@ -422,6 +413,7 @@ class UploadStash {
         * with an extension.
         * XXX this is somewhat redundant with the checks that ApiUpload.php does with incoming
         * uploads versus the desired filename. Maybe we can get that passed to us...
+        * @return string
         */
        public static function getExtensionForPath( $path ) {
                // Does this have an extension?
@@ -452,10 +444,10 @@ class UploadStash {
         * @param $key String: key
         * @return boolean
         */
-       protected function fetchFileMetadata( $key, $readFromMaster = false ) {
+       protected function fetchFileMetadata( $key, $readFromDB = DB_SLAVE ) {
                // populate $fileMetadata[$key]
                $dbr = null;
-               if( $readFromMaster ) {
+               if( $readFromDB === DB_MASTER ) {
                        // sometimes reading from the master is necessary, if there's replication lag.
                        $dbr = $this->repo->getMasterDb();
                } else {
@@ -506,7 +498,7 @@ class UploadStashFile extends UnregisteredLocalFile {
         * A LocalFile wrapper around a file that has been temporarily stashed, so we can do things like create thumbnails for it
         * Arguably UnregisteredLocalFile should be handling its own file repo but that class is a bit retarded currently
         *
-        * @param $repo FSRepo: repository where we should find the path
+        * @param $repo FileRepo: repository where we should find the path
         * @param $path String: path to file
         * @param $key String: key to store the path and any stashed data under
         * @throws UploadStashBadPathException
@@ -529,7 +521,7 @@ class UploadStashFile extends UnregisteredLocalFile {
                        }
 
                        // check if path exists! and is a plain file.
-                       if ( ! $repo->fileExists( $path, FileRepo::FILES_ONLY ) ) {
+                       if ( ! $repo->fileExists( $path ) ) {
                                wfDebug( "UploadStash: tried to construct an UploadStashFile from a file that should already exist at '$path', but path is not found\n" );
                                throw new UploadStashFileNotFoundException( 'cannot find path, or not a plain file' );
                        }
@@ -653,7 +645,7 @@ class UploadStashFile extends UnregisteredLocalFile {
         * @return Status: success
         */
        public function remove() {
-               if ( !$this->repo->fileExists( $this->path, FileRepo::FILES_ONLY ) ) {
+               if ( !$this->repo->fileExists( $this->path ) ) {
                        // Maybe the file's already been removed? This could totally happen in UploadBase.
                        return true;
                }
@@ -662,7 +654,7 @@ class UploadStashFile extends UnregisteredLocalFile {
        }
 
        public function exists() {
-               return $this->repo->fileExists( $this->path, FileRepo::FILES_ONLY );
+               return $this->repo->fileExists( $this->path );
        }
 
 }