Fix docs on File::getImageSize() they were very misleading
authorBrian Wolff <bawolff+wn@gmail.com>
Sat, 24 May 2014 17:27:36 +0000 (14:27 -0300)
committerBrian Wolff <bawolff+wn@gmail.com>
Sat, 24 May 2014 17:31:14 +0000 (14:31 -0300)
Change-Id: I2a3e1e8c10da7ade4ea713a561e05fd4ab9203c8

includes/filerepo/file/File.php

index 5895dda..a45cd72 100644 (file)
@@ -1728,17 +1728,20 @@ abstract class File {
 
        /**
         * Get an image size array like that returned by getImageSize(), or false if it
-        * can't be determined.
+        * can't be determined. Loads the image size directly from the file ignoring caches.
         *
-        * @param string $fileName The filename
-        * @return array
+        * @note Use getWidth()/getHeight() instead of this method unless you have a
+        *  a good reason. This method skips all caches.
+        *
+        * @param string $fileName The path to the file (e.g. From getLocalPathRef() )
+        * @return array The width, followed by height, with optionally more things after
         */
-       function getImageSize( $fileName ) {
+       function getImageSize( $filePath ) {
                if ( !$this->getHandler() ) {
                        return false;
                }
 
-               return $this->handler->getImageSize( $this, $fileName );
+               return $this->handler->getImageSize( $this, $filePath );
        }
 
        /**