I found that print_pdf_mpdf.module doesn't list a minimum version. Given that PHP 5.6 and 7.0 are reaching EOL at the end of 2018, older releases will no longer be considered secure. Therefore it's a good idea to suggest a minimum version for installation.

My suggestion is 7.1.0 for the following reasons:
- Full compatibility for PHP 7.1+ was completed as of 7.0.0
- 7.1.0 is the latest "release", and appears at the top of the Releases page.
- 7.1.0 has been available since 18-May-2018, so it's likely fairly well tested.
- (There are newer tagged versions, but they only appear at the top of the Releases page when you click the nearly-invisible "Show 6 newer tags" link.)

Here's the list of releases so you can make your own decision:
https://github.com/mpdf/mpdf/releases

CommentFileSizeAuthor
#2 print-mpdf_min_version-3011265-2.patch3.35 KBhargobind
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hargobind created an issue. See original summary.

hargobind’s picture

hargobind
he/him
Location Austin, Texas
CreditAttribution: hargobind commented
Status: Active » Needs review
FileSize
print-mpdf_min_version-3011265-2.patch3.35 KB

Attaching a patch to add a minimum version.

This patch also improves version detection in mPDF by getting the VERSION value from the class, something that has been available since mPDF 7.0.0.

collinhaines’s picture

collinhaines CreditAttribution: collinhaines commented

Instructions within patch assisted me with a mPDF upgrade to 7.1.0.

jcnventura’s picture

jcnventura CreditAttribution: jcnventura at 1xINTERNET commented
Status: Needs review » Reviewed & tested by the community

  • jcnventura committed a764232 on 7.x-2.x authored by hargobind 
    Issue #3011265 by hargobind, collinhaines, jcnventura: Minimum version...
jcnventura’s picture

jcnventura CreditAttribution: jcnventura at 1xINTERNET commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.