Brian Wolff has uploaded a new change for review.
https://gerrit.wikimedia.org/r/59531
Change subject: Fix warning if pdfinfo fails but pdftext succedes.
......................................................................
Fix warning if pdfinfo fails but pdftext succedes.
Discovered when fixing bug 41281
Change-Id: I8c956da326e5dc339893a010370d399e97e204fd
---
M PdfHandler_body.php
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/PdfHandler
refs/changes/31/59531/1
diff --git a/PdfHandler_body.php b/PdfHandler_body.php
index 73ce1b5..8fa6719 100644
--- a/PdfHandler_body.php
+++ b/PdfHandler_body.php
@@ -315,7 +315,7 @@
*/
function pageCount( $image ) {
$data = $this->getMetaArray( $image );
- if ( !$data ) {
+ if ( !$data || !isset( $data['Pages'] ) ) {
return false;
}
return intval( $data['Pages'] );
--
To view, visit https://gerrit.wikimedia.org/r/59531
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8c956da326e5dc339893a010370d399e97e204fd
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PdfHandler
Gerrit-Branch: master
Gerrit-Owner: Brian Wolff <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
