Followup to r44204. Hardcoding image because it should (in theory) work always and...
[lhc/web/wiklou.git] / includes / filerepo / File.php
index 90937d2..4969647 100644 (file)
@@ -1,15 +1,15 @@
 <?php
 
 /**
- * Implements some public methods and some protected utility functions which 
- * are required by multiple child classes. Contains stub functionality for 
+ * Implements some public methods and some protected utility functions which
+ * are required by multiple child classes. Contains stub functionality for
  * unimplemented public methods.
  *
- * Stub functions which should be overridden are marked with STUB. Some more 
+ * Stub functions which should be overridden are marked with STUB. Some more
  * concrete functions are also typically overridden by child classes.
  *
  * Note that only the repo object knows what its file class is called. You should
- * never name a file class explictly outside of the repo class. Instead use the 
+ * never name a file class explictly outside of the repo class. Instead use the
  * repo's factory functions to generate file objects, for example:
  *
  * RepoGroup::singleton()->getLocalRepo()->newFile($title);
@@ -17,7 +17,7 @@
  * The convenience functions wfLocalFile() and wfFindFile() should be sufficient
  * in most cases.
  *
- * @addtogroup FileRepo
+ * @ingroup FileRepo
  */
 abstract class File {
        const DELETED_FILE = 1;
@@ -28,17 +28,17 @@ abstract class File {
 
        const DELETE_SOURCE = 1;
 
-       /** 
-        * Some member variables can be lazy-initialised using __get(). The 
+       /**
+        * Some member variables can be lazy-initialised using __get(). The
         * initialisation function for these variables is always a function named
-        * like getVar(), where Var is the variable name with upper-case first 
+        * like getVar(), where Var is the variable name with upper-case first
         * letter.
         *
         * The following variables are initialised in this way in this base class:
-        *    name, extension, handler, path, canRender, isSafeFile, 
+        *    name, extension, handler, path, canRender, isSafeFile,
         *    transformScript, hashPath, pageCount, url
         *
-        * Code within this class should generally use the accessor function 
+        * Code within this class should generally use the accessor function
         * directly, since __get() isn't re-entrant and therefore causes bugs that
         * depend on initialisation order.
         */
@@ -46,7 +46,7 @@ abstract class File {
        /**
         * The following member variables are not lazy-initialised
         */
-       var $repo, $title, $lastError;
+       var $repo, $title, $lastError, $redirected, $redirectedTitle;
 
        /**
         * Call this constructor from child classes
@@ -79,7 +79,8 @@ abstract class File {
                        'htm' => 'html',
                        'jpeg' => 'jpg',
                        'mpeg' => 'mpg',
-                       'tiff' => 'tif' );
+                       'tiff' => 'tif',
+                       'ogv' => 'ogg' );
                if( isset( $squish[$lower] ) ) {
                        return $squish[$lower];
                } elseif( preg_match( '/^[0-9a-z]+$/', $lower ) ) {
@@ -89,6 +90,21 @@ abstract class File {
                }
        }
 
+       /**
+        * Checks if file extensions are compatible
+        *
+        * @param $old File Old file
+        * @param $new string New name
+        */
+       static function checkExtensionCompatibility( File $old, $new ) {
+               $oldMime = $old->getMimeType();
+               $n = strrpos( $new, '.' );
+               $newExt = self::normalizeExtension(
+                       $n ? substr( $new, $n + 1 ) : '' );
+               $mimeMagic = MimeMagic::singleton();
+               return $mimeMagic->isMatchingExtension( $newExt, $oldMime );
+       }
+
        /**
         * Upgrade the database row if there is one
         * Called by ImagePage
@@ -110,7 +126,7 @@ abstract class File {
                        return array( $mime, 'unknown' );
                }
        }
-       
+
        /**
         * Return the name of this file
         */
@@ -118,7 +134,7 @@ abstract class File {
                if ( !isset( $this->name ) ) {
                        $this->name = $this->repo->getNameFromTitle( $this->title );
                }
-               return $this->name; 
+               return $this->name;
        }
 
        /**
@@ -127,7 +143,7 @@ abstract class File {
        function getExtension() {
                if ( !isset( $this->extension ) ) {
                        $n = strrpos( $this->getName(), '.' );
-                       $this->extension = self::normalizeExtension( 
+                       $this->extension = self::normalizeExtension(
                                $n ? substr( $this->getName(), $n + 1 ) : '' );
                }
                return $this->extension;
@@ -135,19 +151,36 @@ abstract class File {
 
        /**
         * Return the associated title object
-        * @public
         */
-       function getTitle() { return $this->title; }
+       public function getTitle() { return $this->title; }
+       
+       /**
+        * Return the title used to find this file
+        */
+       public function getOriginalTitle() {
+               if ( $this->redirected )
+                       return $this->getRedirectedTitle();
+               return $this->title;
+       }
 
        /**
         * Return the URL of the file
-        * @public
         */
-       function getUrl() { 
+       public function getUrl() {
                if ( !isset( $this->url ) ) {
                        $this->url = $this->repo->getZoneUrl( 'public' ) . '/' . $this->getUrlRel();
                }
-               return $this->url; 
+               return $this->url;
+       }
+
+       /**
+        * Return a fully-qualified URL to the file.
+        * Upload URL paths _may or may not_ be fully qualified, so
+        * we check. Local paths are assumed to belong on $wgServer.
+        * @return string
+        */
+       public function getFullUrl() {
+               return wfExpandUrl( $this->getUrl() );
        }
 
        function getViewURL() {
@@ -173,10 +206,8 @@ abstract class File {
        * or in hashed paths like /images/3/3c.
        *
        * May return false if the file is not locally accessible.
-       *
-       * @public
        */
-       function getPath() {
+       public function getPath() {
                if ( !isset( $this->path ) ) {
                        $this->path = $this->repo->getZonePath('public') . '/' . $this->getRel();
                }
@@ -185,14 +216,13 @@ abstract class File {
 
        /**
        * Alias for getPath()
-       * @public
        */
-       function getFullPath() {
+       public function getFullPath() {
                return $this->getPath();
        }
 
        /**
-        * Return the width of the image. Returns false if the width is unknown 
+        * Return the width of the image. Returns false if the width is unknown
         * or undefined.
         *
         * STUB
@@ -201,7 +231,7 @@ abstract class File {
        public function getWidth( $page = 1 ) { return false; }
 
        /**
-        * Return the height of the image. Returns false if the height is unknown 
+        * Return the height of the image. Returns false if the height is unknown
         * or undefined
         *
         * STUB
@@ -209,12 +239,39 @@ abstract class File {
         */
        public function getHeight( $page = 1 ) { return false; }
 
+       /**
+        * Returns ID or name of user who uploaded the file
+        * STUB
+        *
+        * @param $type string 'text' or 'id'
+        */
+       public function getUser( $type='text' ) { return null; }
+
+       /**
+        * Get the duration of a media file in seconds
+        */
+       public function getLength() {
+               $handler = $this->getHandler();
+               if ( $handler ) {
+                       return $handler->getLength( $this );
+               } else {
+                       return 0;
+               }
+       }
+
        /**
         * Get handler-specific metadata
         * Overridden by LocalFile, UnregisteredLocalFile
         * STUB
         */
-       function getMetadata() { return false; }
+       public function getMetadata() { return false; }
+
+       /**
+        * Return the bit depth of the file
+        * Overridden by LocalFile
+        * STUB
+        */
+       public function getBitDepth() { return 0; }
 
        /**
         * Return the size of the image file, in bytes
@@ -239,7 +296,9 @@ abstract class File {
        function getMediaType() { return MEDIATYPE_UNKNOWN; }
 
        /**
-        * Checks if the file can be presented to the browser as a bitmap.
+        * Checks if the output of transform() for this file is likely
+        * to be valid. If this is false, various user elements will
+        * display a placeholder instead.
         *
         * Currently, this checks if the file is an image format
         * that can be converted to a format
@@ -248,7 +307,7 @@ abstract class File {
         */
        function canRender() {
                if ( !isset( $this->canRender ) ) {
-                       $this->canRender = $this->getHandler() && $this->handler->canRender();
+                       $this->canRender = $this->getHandler() && $this->handler->canRender( $this );
                }
                return $this->canRender;
        }
@@ -271,16 +330,11 @@ abstract class File {
         * @return bool
         */
        function mustRender() {
-               return $this->getHandler() && $this->handler->mustRender();
+               return $this->getHandler() && $this->handler->mustRender( $this );
        }
 
        /**
-        * Determines if this media file may be shown inline on a page.
-        *
-        * This is currently synonymous to canRender(), but this could be
-        * extended to also allow inline display of other media,
-        * like flash animations or videos. If you do so, please keep in mind that
-        * that could be a security risk.
+        * Alias for canRender()
         */
        function allowInlineDisplay() {
                return $this->canRender();
@@ -303,7 +357,7 @@ abstract class File {
                }
                return $this->isSafeFile;
        }
-       
+
        /** Accessor for __get() */
        protected function getIsSafeFile() {
                return $this->isSafeFile();
@@ -349,13 +403,24 @@ abstract class File {
         * Returns true if file exists in the repository.
         *
         * Overridden by LocalFile to avoid unnecessary stat calls.
-        * 
+        *
         * @return boolean Whether file exists in the repository.
         */
        public function exists() {
                return $this->getPath() && file_exists( $this->path );
        }
 
+       /**
+        * Returns true if file exists in the repository and can be included in a page.
+        * It would be unsafe to include private images, making public thumbnails inadvertently
+        *
+        * @return boolean Whether file exists in the repository and is includable.
+        * @public
+        */
+       function isVisible() {
+               return $this->exists();
+       }
+
        function getTransformScript() {
                if ( !isset( $this->transformScript ) ) {
                        $this->transformScript = false;
@@ -441,8 +506,7 @@ abstract class File {
         *
         * @param integer $width        maximum width of the generated thumbnail
         * @param integer $height       maximum height of the image (optional)
-        * @param boolean $render       True to render the thumbnail if it doesn't exist,
-        *                              false to just return the URL
+        * @param boolean $render       Deprecated
         *
         * @return ThumbnailImage or null on failure
         *
@@ -453,14 +517,13 @@ abstract class File {
                if ( $height != -1 ) {
                        $params['height'] = $height;
                }
-               $flags = $render ? self::RENDER_NOW : 0;
-               return $this->transform( $params, $flags );
+               return $this->transform( $params, 0 );
        }
 
        /**
         * Transform a media file
         *
-        * @param array $params An associative array of handler-specific parameters. Typical 
+        * @param array $params An associative array of handler-specific parameters. Typical
         *                      keys are width, height and page.
         * @param integer $flags A bitfield, may contain self::RENDER_NOW to force rendering
         * @return MediaTransformOutput
@@ -470,7 +533,7 @@ abstract class File {
 
                wfProfileIn( __METHOD__ );
                do {
-                       if ( !$this->getHandler() || !$this->handler->canRender() ) {
+                       if ( !$this->canRender() ) {
                                // not a bitmap or renderable image, don't try.
                                $thumb = $this->iconThumb();
                                break;
@@ -478,14 +541,16 @@ abstract class File {
 
                        $script = $this->getTransformScript();
                        if ( $script && !($flags & self::RENDER_NOW) ) {
-                               // Use a script to transform on client request
+                               // Use a script to transform on client request, if possible
                                $thumb = $this->handler->getScriptedTransform( $this, $script, $params );
-                               break;
+                               if( $thumb ) {
+                                       break;
+                               }
                        }
 
                        $normalisedParams = $params;
                        $this->handler->normaliseParams( $this, $normalisedParams );
-                       $thumbName = $this->thumbName( $normalisedParams );     
+                       $thumbName = $this->thumbName( $normalisedParams );
                        $thumbPath = $this->getThumbPath( $thumbName );
                        $thumbUrl = $this->getThumbUrl( $thumbName );
 
@@ -512,8 +577,11 @@ abstract class File {
                                }
                        }
                        
-                       if ( $wgUseSquid ) {
-                               wfPurgeSquidServers( array( $thumbUrl ) );
+                       // Purge. Useful in the event of Core -> Squid connection failure or squid 
+                       // purge collisions from elsewhere during failure. Don't keep triggering for 
+                       // "thumbs" which have the main image URL though (bug 13776)
+                       if ( $wgUseSquid && ( !$thumb || $thumb->isError() || $thumb->getUrl() != $this->getURL()) ) {
+                               SquidUpdate::purge( array( $thumbUrl ) );
                        }
                } while (false);
 
@@ -521,7 +589,7 @@ abstract class File {
                return $thumb;
        }
 
-       /** 
+       /**
         * Hook into transform() to allow migration of thumbnail files
         * STUB
         * Overridden by LocalFile
@@ -531,7 +599,7 @@ abstract class File {
        /**
         * Get a MediaHandler instance for this file
         */
-       function getHandler() { 
+       function getHandler() {
                if ( !isset( $this->handler ) ) {
                        $this->handler = MediaHandler::getHandler( $this->getMimeType() );
                }
@@ -550,7 +618,7 @@ abstract class File {
                        $path = '/common/images/icons/' . $icon;
                        $filepath = $wgStyleDirectory . $path;
                        if( file_exists( $filepath ) ) {
-                               return new ThumbnailImage( $wgStylePath . $path, 120, 120 );
+                               return new ThumbnailImage( $this, $wgStylePath . $path, 120, 120 );
                        }
                }
                return null;
@@ -576,7 +644,7 @@ abstract class File {
         * STUB
         * Overridden by LocalFile
         */
-       function purgeCache( $archiveFiles = array() ) {}
+       function purgeCache() {}
 
        /**
         * Purge the file description page, but don't go after
@@ -590,14 +658,12 @@ abstract class File {
                        $title->purgeSquid();
                }
        }
-       
+
        /**
         * Purge metadata and all affected pages when the file is created,
-        * deleted, or majorly updated. A set of additional URLs may be
-        * passed to purge, such as specific file files which have changed.
-        * @param $urlArray array
+        * deleted, or majorly updated.
         */
-       function purgeEverything( $urlArr=array() ) {
+       function purgeEverything() {
                // Delete thumbnails and refresh file metadata cache
                $this->purgeCache();
                $this->purgeDescription();
@@ -611,8 +677,20 @@ abstract class File {
        }
 
        /**
-        * Return the history of this file, line by line. Starts with current version, 
-        * then old versions. Should return an object similar to an image/oldimage 
+        * Return a fragment of the history of file.
+        *
+        * STUB
+        * @param $limit integer Limit of rows to return
+        * @param $start timestamp Only revisions older than $start will be returned
+        * @param $end timestamp Only revisions newer than $end will be returned
+        */
+       function getHistory($limit = null, $start = null, $end = null) {
+               return array();
+       }
+
+       /**
+        * Return the history of this file, line by line. Starts with current version,
+        * then old versions. Should return an object similar to an image/oldimage
         * database row.
         *
         * STUB
@@ -656,9 +734,9 @@ abstract class File {
                return $this->getHashPath() . rawurlencode( $this->getName() );
        }
 
-       /** Get the path of the archive directory, or a particular file if $suffix is specified */
-       function getArchivePath( $suffix = false ) {
-               $path = $this->repo->getZonePath('public') . '/archive/' . $this->getHashPath();
+       /** Get the relative path for an archive file */
+       function getArchiveRel( $suffix = false ) {
+               $path = 'archive/' . $this->getHashPath();
                if ( $suffix === false ) {
                        $path = substr( $path, 0, -1 );
                } else {
@@ -667,15 +745,25 @@ abstract class File {
                return $path;
        }
 
-       /** Get the path of the thumbnail directory, or a particular file if $suffix is specified */
-       function getThumbPath( $suffix = false ) {
-               $path = $this->repo->getZonePath('public') . '/thumb/' . $this->getRel();
+       /** Get relative path for a thumbnail file */
+       function getThumbRel( $suffix = false ) {
+               $path = 'thumb/' . $this->getRel();
                if ( $suffix !== false ) {
                        $path .= '/' . $suffix;
                }
                return $path;
        }
 
+       /** Get the path of the archive directory, or a particular file if $suffix is specified */
+       function getArchivePath( $suffix = false ) {
+               return $this->repo->getZonePath('public') . '/' . $this->getArchiveRel( $suffix );
+       }
+
+       /** Get the path of the thumbnail directory, or a particular file if $suffix is specified */
+       function getThumbPath( $suffix = false ) {
+               return $this->repo->getZonePath('public') . '/' . $this->getThumbRel( $suffix );
+       }
+
        /** Get the URL of the archive directory, or a particular file if $suffix is specified */
        function getArchiveUrl( $suffix = false ) {
                $path = $this->repo->getZoneUrl('public') . '/archive/' . $this->getHashPath();
@@ -723,7 +811,7 @@ abstract class File {
                        $path .= '/' . rawurlencode( $suffix );
                }
                return $path;
-       }       
+       }
 
        /**
         * @return bool
@@ -741,25 +829,25 @@ abstract class File {
         * STUB
         * Overridden by LocalFile
         */
-       function recordUpload( $oldver, $desc, $license = '', $copyStatus = '', $source = '', $watch = false ) { 
-               $this->readOnlyError(); 
+       function recordUpload( $oldver, $desc, $license = '', $copyStatus = '', $source = '', $watch = false ) {
+               $this->readOnlyError();
        }
 
        /**
-        * Move or copy a file to its public location. If a file exists at the  
-        * destination, move it to an archive. Returns the archive name on success 
-        * or an empty string if it was a new file, and a wikitext-formatted 
-        * WikiError object on failure. 
+        * Move or copy a file to its public location. If a file exists at the
+        * destination, move it to an archive. Returns the archive name on success
+        * or an empty string if it was a new file, and a wikitext-formatted
+        * WikiError object on failure.
         *
         * The archive name should be passed through to recordUpload for database
         * registration.
         *
         * @param string $sourcePath Local filesystem path to the source image
         * @param integer $flags A bitwise combination of:
-        *     File::DELETE_SOURCE    Delete the source file, i.e. move 
+        *     File::DELETE_SOURCE    Delete the source file, i.e. move
         *         rather than copy
-        * @return The archive name on success or an empty string if it was a new 
-        *     file, and a wikitext-formatted WikiError object on failure. 
+        * @return The archive name on success or an empty string if it was a new
+        *     file, and a wikitext-formatted WikiError object on failure.
         *
         * STUB
         * Overridden by LocalFile
@@ -785,18 +873,19 @@ abstract class File {
                } else {
                        $db = wfGetDB( DB_SLAVE );
                }
-               $linkCache =& LinkCache::singleton();
+               $linkCache = LinkCache::singleton();
 
                list( $page, $imagelinks ) = $db->tableNamesN( 'page', 'imagelinks' );
                $encName = $db->addQuotes( $this->getName() );
-               $sql = "SELECT page_namespace,page_title,page_id FROM $page,$imagelinks WHERE page_id=il_from AND il_to=$encName $options";
+               $sql = "SELECT page_namespace,page_title,page_id,page_len,page_is_redirect,
+                       FROM $page,$imagelinks WHERE page_id=il_from AND il_to=$encName $options";
                $res = $db->query( $sql, __METHOD__ );
 
                $retVal = array();
                if ( $db->numRows( $res ) ) {
                        while ( $row = $db->fetchObject( $res ) ) {
-                               if ( $titleObj = Title::makeTitle( $row->page_namespace, $row->page_title ) ) {
-                                       $linkCache->addGoodLinkObj( $row->page_id, $titleObj );
+                               if ( $titleObj = Title::newFromRow( $row ) ) {
+                                       $linkCache->addGoodLinkObj( $row->page_id, $titleObj, $row->page_len, $row->page_is_redirect );
                                        $retVal[] = $titleObj;
                                }
                        }
@@ -806,22 +895,11 @@ abstract class File {
                return $retVal;
        }
 
-       function getExifData() {
-               if ( !$this->getHandler() || $this->handler->getMetadataType( $this ) != 'exif' ) {
-                       return array();
-               }
-               $metadata = $this->getMetadata();
-               if ( !$metadata ) {
-                       return array();
-               }
-               $exif = unserialize( $metadata );
-               if ( !$exif ) {
-                       return array();
+       function formatMetadata() {
+               if ( !$this->getHandler() ) {
+                       return false;
                }
-               unset( $exif['MEDIAWIKI_EXIF_VERSION'] );
-               $format = new FormatExif( $exif );
-
-               return $format->getFormattedData();
+               return $this->getHandler()->formatMetadata( $this, $this->getMetadata() );
        }
 
        /**
@@ -829,8 +907,8 @@ abstract class File {
         *
         * @return bool
         */
-       function isLocal() { 
-               return $this->getRepoName() == 'local'; 
+       function isLocal() {
+               return $this->getRepoName() == 'local';
        }
 
        /**
@@ -838,8 +916,14 @@ abstract class File {
         *
         * @return string
         */
-       function getRepoName() { 
-               return $this->repo ? $this->repo->getName() : 'unknown'; 
+       function getRepoName() {
+               return $this->repo ? $this->repo->getName() : 'unknown';
+       }
+       /*
+        * Returns the repository
+        */
+       function getRepo() {
+               return $this->repo;
        }
 
        /**
@@ -868,6 +952,22 @@ abstract class File {
                return $title && $title->isDeleted() > 0;
        }
 
+       /**
+        * Move file to the new title
+        *
+        * Move current, old version and all thumbnails
+        * to the new filename. Old file is deleted.
+        *
+        * Cache purging is done; checks for validity
+        * and logging are caller's responsibility
+        *
+        * @param $target Title New file name
+        * @return FileRepoStatus object.
+        */
+        function move( $target ) {
+               $this->readOnlyError();
+        }
+
        /**
         * Delete all versions of the file.
         *
@@ -877,11 +977,12 @@ abstract class File {
         * Cache purging is done; logging is caller's responsibility.
         *
         * @param $reason
+        * @param $suppress, hide content from sysops?
         * @return true on success, false on some kind of failure
         * STUB
         * Overridden by LocalFile
         */
-       function delete( $reason, $suppress=false ) {
+       function delete( $reason, $suppress = false ) {
                $this->readOnlyError();
        }
 
@@ -893,12 +994,13 @@ abstract class File {
         *
         * @param $versions set of record ids of deleted items to restore,
         *                    or empty to restore all revisions.
+        * @param $unsuppress, remove restrictions on content upon restoration?
         * @return the number of file revisions restored if successful,
         *         or false on failure
         * STUB
         * Overridden by LocalFile
         */
-       function restore( $versions=array(), $Unsuppress=false ) {
+       function restore( $versions=array(), $unsuppress=false ) {
                $this->readOnlyError();
        }
 
@@ -909,7 +1011,7 @@ abstract class File {
         * @return Bool
         */
        function isMultipage() {
-               return $this->getHandler() && $this->handler->isMultiPage();
+               return $this->getHandler() && $this->handler->isMultiPage( $this );
        }
 
        /**
@@ -918,7 +1020,7 @@ abstract class File {
         */
        function pageCount() {
                if ( !isset( $this->pageCount ) ) {
-                       if ( $this->getHandler() && $this->handler->isMultiPage() ) {
+                       if ( $this->getHandler() && $this->handler->isMultiPage( $this ) ) {
                                $this->pageCount = $this->handler->pageCount( $this );
                        } else {
                                $this->pageCount = false;
@@ -938,9 +1040,9 @@ abstract class File {
                        return round( $srcHeight * $dstWidth / $srcWidth );
                }
        }
-       
+
        /**
-        * Get an image size array like that returned by getimagesize(), or false if it 
+        * Get an image size array like that returned by getimagesize(), or false if it
         * can't be determined.
         *
         * @param string $fileName The filename
@@ -965,34 +1067,63 @@ abstract class File {
         * Get the HTML text of the description page, if available
         */
        function getDescriptionText() {
+               global $wgMemc;
                if ( !$this->repo->fetchDescription ) {
                        return false;
                }
                $renderUrl = $this->repo->getDescriptionRenderUrl( $this->getName() );
                if ( $renderUrl ) {
+                       if ( $this->repo->descriptionCacheExpiry > 0 ) {
+                               wfDebug("Attempting to get the description from cache...");
+                               $key = wfMemcKey( 'RemoteFileDescription', 'url', md5($renderUrl) );
+                               $obj = $wgMemc->get($key);
+                               if ($obj) {
+                                       wfDebug("success!\n");
+                                       return $obj;
+                               }
+                               wfDebug("miss\n");
+                       }
                        wfDebug( "Fetching shared description from $renderUrl\n" );
-                       return Http::get( $renderUrl );
+                       $res = Http::get( $renderUrl );
+                       if ( $res && $this->repo->descriptionCacheExpiry > 0 ) $wgMemc->set( $key, $res, $this->repo->descriptionCacheExpiry );
+                       return $res;
                } else {
                        return false;
                }
        }
 
        /**
-        * Get the 14-character timestamp of the file upload, or false if 
+        * Get discription of file revision
+        * STUB
+        */
+       function getDescription() {
+               return null;
+       }
+
+       /**
+        * Get the 14-character timestamp of the file upload, or false if
+        * it doesn't exist
         */
-       function getTimestmap() {
+       function getTimestamp() {
                $path = $this->getPath();
                if ( !file_exists( $path ) ) {
                        return false;
                }
-               return wfTimestamp( filemtime( $path ) );
+               return wfTimestamp( TS_MW, filemtime( $path ) );
        }
-       
+
+       /**
+        * Get the SHA-1 base 36 hash of the file
+        */
+       function getSha1() {
+               return self::sha1Base36( $this->getPath() );
+       }
+
        /**
         * Determine if the current user is allowed to view a particular
         * field of this file, if it's marked as deleted.
         * STUB
-        * @param int $field                                    
+        * @param int $field
         * @return bool
         */
        function userCan( $field ) {
@@ -1000,16 +1131,18 @@ abstract class File {
        }
 
        /**
-        * Get an associative array containing information about a file in the local filesystem\
+        * Get an associative array containing information about a file in the local filesystem.
         *
         * @param string $path Absolute local filesystem path
-        * @param mixed $ext The file extension, or true to extract it from the filename. 
+        * @param mixed $ext The file extension, or true to extract it from the filename.
         *                   Set it to false to ignore the extension.
         */
        static function getPropsFromPath( $path, $ext = true ) {
                wfProfileIn( __METHOD__ );
                wfDebug( __METHOD__.": Getting file info for $path\n" );
-               $info = array( 'fileExists' => file_exists( $path ) );
+               $info = array(
+                       'fileExists' => file_exists( $path ) && !is_dir( $path )
+               );
                $gis = false;
                if ( $info['fileExists'] ) {
                        $magic = MimeMagic::singleton();
@@ -1025,18 +1158,20 @@ abstract class File {
                        $handler = MediaHandler::getHandler( $info['mime'] );
                        if ( $handler ) {
                                $tempImage = (object)array();
-                               $gis = $handler->getImageSize( $tempImage, $path );
                                $info['metadata'] = $handler->getMetadata( $tempImage, $path );
+                               $gis = $handler->getImageSize( $tempImage, $path, $info['metadata'] );
                        } else {
                                $gis = false;
                                $info['metadata'] = '';
                        }
+                       $info['sha1'] = self::sha1Base36( $path );
 
                        wfDebug(__METHOD__.": $path loaded, {$info['size']} bytes, {$info['mime']}.\n");
                } else {
                        $info['mime'] = NULL;
                        $info['media_type'] = MEDIATYPE_UNKNOWN;
                        $info['metadata'] = '';
+                       $info['sha1'] = '';
                        wfDebug(__METHOD__.": $path NOT FOUND!\n");
                }
                if( $gis ) {
@@ -1056,6 +1191,74 @@ abstract class File {
                wfProfileOut( __METHOD__ );
                return $info;
        }
-}
 
+       /**
+        * Get a SHA-1 hash of a file in the local filesystem, in base-36 lower case
+        * encoding, zero padded to 31 digits.
+        *
+        * 160 log 2 / log 36 = 30.95, so the 160-bit hash fills 31 digits in base 36
+        * fairly neatly.
+        *
+        * Returns false on failure
+        */
+       static function sha1Base36( $path ) {
+               wfSuppressWarnings();
+               $hash = sha1_file( $path );
+               wfRestoreWarnings();
+               if ( $hash === false ) {
+                       return false;
+               } else {
+                       return wfBaseConvert( $hash, 16, 36, 31 );
+               }
+       }
+
+       function getLongDesc() {
+               $handler = $this->getHandler();
+               if ( $handler ) {
+                       return $handler->getLongDesc( $this );
+               } else {
+                       return MediaHandler::getLongDesc( $this );
+               }
+       }
 
+       function getShortDesc() {
+               $handler = $this->getHandler();
+               if ( $handler ) {
+                       return $handler->getShortDesc( $this );
+               } else {
+                       return MediaHandler::getShortDesc( $this );
+               }
+       }
+
+       function getDimensionsString() {
+               $handler = $this->getHandler();
+               if ( $handler ) {
+                       return $handler->getDimensionsString( $this );
+               } else {
+                       return '';
+               }
+       }
+
+       function getRedirected() {
+               return $this->redirected;
+       }
+       
+       function getRedirectedTitle() {
+               if ( $this->redirected ) {
+                       if ( !$this->redirectTitle )
+                               $this->redirectTitle = Title::makeTitle( NS_FILE, $this->redirected );
+                       return $this->redirectTitle;
+               }
+       }
+
+       function redirectedFrom( $from ) {
+               $this->redirected = $from;
+       }
+}
+/**
+ * Aliases for backwards compatibility with 1.6
+ */
+define( 'MW_IMG_DELETED_FILE', File::DELETED_FILE );
+define( 'MW_IMG_DELETED_COMMENT', File::DELETED_COMMENT );
+define( 'MW_IMG_DELETED_USER', File::DELETED_USER );
+define( 'MW_IMG_DELETED_RESTRICTED', File::DELETED_RESTRICTED );