Support PHP 7.4+ (mPDF 8)

Hi,

I posted a comment on your MR: https://git.drupalcode.org/project/pdf_using_mpdf/-/merge_requests/1#not...

Pushed in MR the change Bryan said, it works as it should then.

Hey all, thanks for updating the code. Any idea when this will be available?

Since Drupal 9 is supporting PHP 7.4 there is no point to make this compatible with only PHP 8. Right now stable version of module is getting error due to this.

It's urgent for me to fix the issue. And I generate a patch from pdf_using_mpdf-3205270

Anyway, patch for php7.3 and php7.4

Hello All,

I am using Drupal8.9.18, PHP 7.4.22 and mPdf 8 on my server.

It is generating PDF with text properly while not adding images into PDF.

Can any one help?

Thanks

Hi, updating priority as Acquia will switch soon to PHP 7.4

Hee is a workaround while this is not merged:

1. composer remove drupal/pdf_using_mpdf
2. add this to the "repositories" section of your composer.json:

        {
            "type": "package",
            "package": {
                "name": "drupal_git/pdf_using_mpdf",
                "version": "2.2",
                "type": "drupal-module",
                "source": {
                    "url": "https://git.drupalcode.org/project/pdf_using_mpdf.git",
                    "type": "git",
                    "reference": "8.x-2.x"
                },
                "require": {
                    "mpdf/mpdf": "^8.0.4"
                }
            }
        }

3. composer require drupal_git/pdf_using_mpdf

Not the best solution but I was forced to switch to PHP 7.4 and this was the only thing blocking me.

#15 was the only way I was able to get out of dependency hell. A new release of this module that considers PHP 7.4 would be greatly appreciated.

#15 works, thanks @gilmord
Tested successfully.

+1 for releasing this to help ease d9 uprgades...

Would be good to release a new version with updated mpdf asap.
We have tested that there are no regressions after the update.

+1 for releasing this to help php7.4 uprgades...

#15 worked great, +1 for a new release.

Think it's safe to say it's RTBC'ed :) we're also actively using this patch

# 15 works for me, +1 for new release as soon as possible

+1 works for me
Release it !

Thanks, @all
merged into the latest dev.

Hello, I can't see a dev version available on this module's page and trying to download 2.x from github only redirects me back here, therefore I still can't add this via composer and the only thing I can do is to download the 2.2 version and fix it manually, what am I missing here? Thank you

Hi @honyik looks like some bug (or feature) of git.drupalcode.org
I tried a couple of other projects - download leads to the project page on drupal.org.

You can use git to download the files like this:

git clone https://git.drupalcode.org/project/pdf_using_mpdf.git

After that, you can switch the branch to the latest dev and use the files however you need.

Also, I am not sure you will be able to fix anything manually, as the required library does not work with the packages of the fresh Drupal installation, and the maintainer of the library is not willing to fix that.

Better look at the alternatives like dompdf, also modules like entity_print can work with various alternatives of the mpdf (if I remember correctly).

Is there any way to this feature be merged on a new release?