X-Git-Url: http://git.cyclocoop.org//%22javascript:ModifierStyle%28%27%22.%24id.%22%27%29/%22?a=blobdiff_plain;f=includes%2Ffilerepo%2Ffile%2FOldLocalFile.php;h=f5b7d4384cc61e2f91eb524300976a8556987b23;hb=c29909e59fd89d9fc65a299687ad2aa2df995acc;hp=3cdbfc2ef040799010d3526d3a52f99b9fea780e;hpb=a44ae41d5806992c7b5524b8098022856f03f016;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/filerepo/file/OldLocalFile.php b/includes/filerepo/file/OldLocalFile.php index 3cdbfc2ef0..f5b7d4384c 100644 --- a/includes/filerepo/file/OldLocalFile.php +++ b/includes/filerepo/file/OldLocalFile.php @@ -42,7 +42,7 @@ class OldLocalFile extends LocalFile { * @param Title $title * @param FileRepo $repo * @param string|int|null $time - * @return self + * @return static * @throws MWException */ static function newFromTitle( $title, $repo, $time = null ) { @@ -51,27 +51,27 @@ class OldLocalFile extends LocalFile { throw new MWException( __METHOD__ . ' got null for $time parameter' ); } - return new self( $title, $repo, $time, null ); + return new static( $title, $repo, $time, null ); } /** * @param Title $title * @param FileRepo $repo * @param string $archiveName - * @return self + * @return static */ static function newFromArchiveName( $title, $repo, $archiveName ) { - return new self( $title, $repo, null, $archiveName ); + return new static( $title, $repo, null, $archiveName ); } /** * @param stdClass $row * @param FileRepo $repo - * @return self + * @return static */ static function newFromRow( $row, $repo ) { $title = Title::makeTitle( NS_FILE, $row->oi_name ); - $file = new self( $title, $repo, null, $row->oi_archive_name ); + $file = new static( $title, $repo, null, $row->oi_archive_name ); $file->loadFromRow( $row, 'oi_' ); return $file; @@ -95,57 +95,17 @@ class OldLocalFile extends LocalFile { $conds['oi_timestamp'] = $dbr->timestamp( $timestamp ); } - $fileQuery = self::getQueryInfo(); + $fileQuery = static::getQueryInfo(); $row = $dbr->selectRow( $fileQuery['tables'], $fileQuery['fields'], $conds, __METHOD__, [], $fileQuery['joins'] ); if ( $row ) { - return self::newFromRow( $row, $repo ); + return static::newFromRow( $row, $repo ); } else { return false; } } - /** - * Fields in the oldimage table - * @deprecated since 1.31, use self::getQueryInfo() instead. - * @return string[] - */ - static function selectFields() { - global $wgActorTableSchemaMigrationStage; - - wfDeprecated( __METHOD__, '1.31' ); - if ( $wgActorTableSchemaMigrationStage & SCHEMA_COMPAT_READ_NEW ) { - // If code is using this instead of self::getQueryInfo(), there's a - // decent chance it's going to try to directly access - // $row->oi_user or $row->oi_user_text and we can't give it - // useful values here once those aren't being used anymore. - throw new BadMethodCallException( - 'Cannot use ' . __METHOD__ - . ' when $wgActorTableSchemaMigrationStage has SCHEMA_COMPAT_READ_NEW' - ); - } - - return [ - 'oi_name', - 'oi_archive_name', - 'oi_size', - 'oi_width', - 'oi_height', - 'oi_metadata', - 'oi_bits', - 'oi_media_type', - 'oi_major_mime', - 'oi_minor_mime', - 'oi_user', - 'oi_user_text', - 'oi_actor' => 'NULL', - 'oi_timestamp', - 'oi_deleted', - 'oi_sha1', - ] + MediaWikiServices::getInstance()->getCommentStore()->getFields( 'oi_description' ); - } - /** * Return the tables, fields, and join conditions to be selected to create * a new oldlocalfile object.