jenkins-bot has submitted this change and it was merged.
Change subject: Set isExpensiveToThumbnail for the djvu handler
......................................................................
Set isExpensiveToThumbnail for the djvu handler
* This makes FileRenderExpensive pool logic apply for large files
to cut down on DB load from getMetaTree()
Bug: T96360
Change-Id: If892d77077607ffcaba0510355175a1e4d780ae9
---
M includes/media/DjVu.php
1 file changed, 11 insertions(+), 0 deletions(-)
Approvals:
GWicke: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/media/DjVu.php b/includes/media/DjVu.php
index 011fb2a..5b57952 100644
--- a/includes/media/DjVu.php
+++ b/includes/media/DjVu.php
@@ -27,6 +27,8 @@
* @ingroup Media
*/
class DjVuHandler extends ImageHandler {
+ const EXPENSIVE_SIZE_LIMIT = 10485760; // 10MiB
+
/**
* @return bool
*/
@@ -50,6 +52,15 @@
}
/**
+ * True if creating thumbnails from the file is large or otherwise
resource-intensive.
+ * @param File $file
+ * @return bool
+ */
+ public function isExpensiveToThumbnail( $file ) {
+ return $file->getSize() > static::EXPENSIVE_SIZE_LIMIT;
+ }
+
+ /**
* @param File $file
* @return bool
*/
--
To view, visit https://gerrit.wikimedia.org/r/209984
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If892d77077607ffcaba0510355175a1e4d780ae9
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
Gerrit-Reviewer: GWicke <[email protected]>
Gerrit-Reviewer: Tpt <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
