| Index: trunk/phase3/includes/filerepo/OldLocalFile.php |
| — | — | @@ -19,8 +19,9 @@ |
| 20 | 20 | |
| 21 | 21 | static function newFromTitle( $title, $repo, $time = null ) { |
| 22 | 22 | # The null default value is only here to avoid an E_STRICT |
| 23 | | - if( $time === null ) |
| | 23 | + if ( $time === null ) { |
| 24 | 24 | throw new MWException( __METHOD__.' got null for $time parameter' ); |
| | 25 | + } |
| 25 | 26 | return new self( $title, $repo, $time, null ); |
| 26 | 27 | } |
| 27 | 28 | |
| — | — | @@ -36,19 +37,25 @@ |
| 37 | 38 | } |
| 38 | 39 | |
| 39 | 40 | /** |
| 40 | | - * @param $sha1 |
| | 41 | + * Create a OldLocalFile from a SHA-1 key |
| | 42 | + * Do not call this except from inside a repo class. |
| | 43 | + * |
| | 44 | + * @param $sha1 string base-36 SHA-1 |
| 41 | 45 | * @param $repo LocalRepo |
| 42 | | - * @param bool $timestamp |
| | 46 | + * @param string|bool $timestamp MW_timestamp (optional) |
| | 47 | + * |
| 43 | 48 | * @return bool|OldLocalFile |
| 44 | 49 | */ |
| 45 | 50 | static function newFromKey( $sha1, $repo, $timestamp = false ) { |
| | 51 | + $dbr = $repo->getSlaveDB(); |
| | 52 | + |
| 46 | 53 | $conds = array( 'oi_sha1' => $sha1 ); |
| 47 | | - if( $timestamp ) { |
| 48 | | - $conds['oi_timestamp'] = $timestamp; |
| | 54 | + if ( $timestamp ) { |
| | 55 | + $conds['oi_timestamp'] = $dbr->timestamp( $timestamp ); |
| 49 | 56 | } |
| 50 | | - $dbr = $repo->getSlaveDB(); |
| | 57 | + |
| 51 | 58 | $row = $dbr->selectRow( 'oldimage', self::selectFields(), $conds, __METHOD__ ); |
| 52 | | - if( $row ) { |
| | 59 | + if ( $row ) { |
| 53 | 60 | return self::newFromRow( $row, $repo ); |
| 54 | 61 | } else { |
| 55 | 62 | return false; |
| Index: trunk/phase3/includes/filerepo/LocalFile.php |
| — | — | @@ -95,22 +95,21 @@ |
| 96 | 96 | * Create a LocalFile from a SHA-1 key |
| 97 | 97 | * Do not call this except from inside a repo class. |
| 98 | 98 | * |
| 99 | | - * @param $sha1 string |
| | 99 | + * @param $sha1 string base-36 SHA-1 |
| 100 | 100 | * @param $repo LocalRepo |
| 101 | | - * @param $timestamp string |
| | 101 | + * @param string|bool $timestamp MW_timestamp (optional) |
| 102 | 102 | * |
| 103 | | - * @return LocalFile |
| | 103 | + * @return bool|LocalFile |
| 104 | 104 | */ |
| 105 | 105 | static function newFromKey( $sha1, $repo, $timestamp = false ) { |
| 106 | | - $conds = array( 'img_sha1' => $sha1 ); |
| | 106 | + $dbr = $repo->getSlaveDB(); |
| 107 | 107 | |
| | 108 | + $conds = array( 'img_sha1' => $sha1 ); |
| 108 | 109 | if ( $timestamp ) { |
| 109 | | - $conds['img_timestamp'] = $timestamp; |
| | 110 | + $conds['img_timestamp'] = $dbr->timestamp( $timestamp ); |
| 110 | 111 | } |
| 111 | 112 | |
| 112 | | - $dbr = $repo->getSlaveDB(); |
| 113 | 113 | $row = $dbr->selectRow( 'image', self::selectFields(), $conds, __METHOD__ ); |
| 114 | | - |
| 115 | 114 | if ( $row ) { |
| 116 | 115 | return self::newFromRow( $row, $repo ); |
| 117 | 116 | } else { |
| Index: trunk/phase3/includes/filerepo/FileRepo.php |
| — | — | @@ -74,7 +74,7 @@ |
| 75 | 75 | * @return File |
| 76 | 76 | */ |
| 77 | 77 | function newFile( $title, $time = false ) { |
| 78 | | - if ( !($title instanceof Title) ) { |
| | 78 | + if ( !( $title instanceof Title ) ) { |
| 79 | 79 | $title = Title::makeTitleSafe( NS_FILE, $title ); |
| 80 | 80 | if ( !is_object( $title ) ) { |
| 81 | 81 | return null; |
| — | — | @@ -130,10 +130,10 @@ |
| 131 | 131 | if ( $time !== false ) { |
| 132 | 132 | $img = $this->newFile( $title, $time ); |
| 133 | 133 | if ( $img && $img->exists() ) { |
| 134 | | - if ( !$img->isDeleted(File::DELETED_FILE) ) { |
| | 134 | + if ( !$img->isDeleted( File::DELETED_FILE ) ) { |
| | 135 | + return $img; // always OK |
| | 136 | + } elseif ( !empty( $options['private'] ) && $img->userCan( File::DELETED_FILE ) ) { |
| 135 | 137 | return $img; |
| 136 | | - } elseif ( !empty( $options['private'] ) && $img->userCan(File::DELETED_FILE) ) { |
| 137 | | - return $img; |
| 138 | 138 | } |
| 139 | 139 | } |
| 140 | 140 | } |
| — | — | @@ -187,30 +187,6 @@ |
| 188 | 188 | } |
| 189 | 189 | |
| 190 | 190 | /** |
| 191 | | - * Create a new File object from the local repository |
| 192 | | - * @param $sha1 Mixed: base 36 SHA-1 hash |
| 193 | | - * @param $time Mixed: time at which the image was uploaded. |
| 194 | | - * If this is specified, the returned object will be an |
| 195 | | - * of the repository's old file class instead of a current |
| 196 | | - * file. Repositories not supporting version control should |
| 197 | | - * return false if this parameter is set. |
| 198 | | - * |
| 199 | | - * @return File |
| 200 | | - */ |
| 201 | | - function newFileFromKey( $sha1, $time = false ) { |
| 202 | | - if ( $time ) { |
| 203 | | - if ( $this->oldFileFactoryKey ) { |
| 204 | | - return call_user_func( $this->oldFileFactoryKey, $sha1, $this, $time ); |
| 205 | | - } |
| 206 | | - } else { |
| 207 | | - if ( $this->fileFactoryKey ) { |
| 208 | | - return call_user_func( $this->fileFactoryKey, $sha1, $this ); |
| 209 | | - } |
| 210 | | - } |
| 211 | | - return false; |
| 212 | | - } |
| 213 | | - |
| 214 | | - /** |
| 215 | 191 | * Find an instance of the file with this key, created at the specified time |
| 216 | 192 | * Returns false if the file does not exist. Repositories not supporting |
| 217 | 193 | * version control should return false if the time is specified. |
| — | — | @@ -221,22 +197,23 @@ |
| 222 | 198 | function findFileFromKey( $sha1, $options = array() ) { |
| 223 | 199 | $time = isset( $options['time'] ) ? $options['time'] : false; |
| 224 | 200 | |
| 225 | | - # First try the current version of the file to see if it precedes the timestamp |
| 226 | | - $img = $this->newFileFromKey( $sha1 ); |
| 227 | | - if ( !$img ) { |
| 228 | | - return false; |
| | 201 | + # First try to find a matching current version of a file... |
| | 202 | + if ( $this->fileFactoryKey ) { |
| | 203 | + $img = call_user_func( $this->fileFactoryKey, $sha1, $this, $time ); |
| | 204 | + } else { |
| | 205 | + return false; // find-by-sha1 not supported |
| 229 | 206 | } |
| 230 | | - if ( $img->exists() && ( !$time || $img->getTimestamp() == $time ) ) { |
| | 207 | + if ( $img && $img->exists() ) { |
| 231 | 208 | return $img; |
| 232 | 209 | } |
| 233 | | - # Now try an old version of the file |
| 234 | | - if ( $time !== false ) { |
| 235 | | - $img = $this->newFileFromKey( $sha1, $time ); |
| | 210 | + # Now try to find a matching old version of a file... |
| | 211 | + if ( $time !== false && $this->oldFileFactoryKey ) { // find-by-sha1 supported? |
| | 212 | + $img = call_user_func( $this->oldFileFactoryKey, $sha1, $this, $time ); |
| 236 | 213 | if ( $img && $img->exists() ) { |
| 237 | | - if ( !$img->isDeleted(File::DELETED_FILE) ) { |
| | 214 | + if ( !$img->isDeleted( File::DELETED_FILE ) ) { |
| | 215 | + return $img; // always OK |
| | 216 | + } elseif ( !empty( $options['private'] ) && $img->userCan( File::DELETED_FILE ) ) { |
| 238 | 217 | return $img; |
| 239 | | - } elseif ( !empty( $options['private'] ) && $img->userCan(File::DELETED_FILE) ) { |
| 240 | | - return $img; |
| 241 | 218 | } |
| 242 | 219 | } |
| 243 | 220 | } |