Aaron Schulz has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/208304

Change subject: Removed DjVu pageCount() check and let it surface in the shell 
command
......................................................................

Removed DjVu pageCount() check and let it surface in the shell command

* This avoids loading the bloated metadata field

Bug: T96360
Change-Id: I2f1b59e81829d5752857816536688cd596b65edf
---
M includes/media/DjVu.php
1 file changed, 0 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/04/208304/1

diff --git a/includes/media/DjVu.php b/includes/media/DjVu.php
index 5f6b965..011fb2a 100644
--- a/includes/media/DjVu.php
+++ b/includes/media/DjVu.php
@@ -143,14 +143,6 @@
                $width = $params['width'];
                $height = $params['height'];
                $page = $params['page'];
-               if ( $page > $this->pageCount( $image ) ) {
-                       return new MediaTransformError(
-                               'thumbnail_error',
-                               $width,
-                               $height,
-                               wfMessage( 'djvu_page_error' )->text()
-                       );
-               }
 
                if ( $flags & self::TRANSFORM_LATER ) {
                        $params = array(

-- 
To view, visit https://gerrit.wikimedia.org/r/208304
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2f1b59e81829d5752857816536688cd596b65edf
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <asch...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to