Skip to content
This repository has been archived by the owner on Jul 5, 2021. It is now read-only.

PHP 8 Compatibility #30

Open
kaptk2 opened this issue Apr 27, 2021 · 4 comments
Open

PHP 8 Compatibility #30

kaptk2 opened this issue Apr 27, 2021 · 4 comments

Comments

@kaptk2
Copy link

kaptk2 commented Apr 27, 2021

The current version will not run on PHP8. This pull request: tecnickcom/TCPDF#244 seems to fix the issue.

@dhcmega
Copy link

dhcmega commented Jun 7, 2021

Hi, this is not applied yet, right?
Do I have to rollback to php 7.4 in order to be able to use this package?
Thanks!

dhcmega

@basepack
Copy link

Well, I just got it working with PHP 8 and Laravel 8 with no troubles whatsoever.

@dhcmega
Copy link

dhcmega commented Jun 10, 2021

Hi
It works, the problem starts when there is something regarding fonts:

PHP Deprecated: Required parameter $currentfont follows optional parameter $str in /home
/vpssigesedev/sigesedev.huayra.com.ar/vendor/daltcore/lara-pdf-merger/src/LynX39/LaraPdfMerger/tcpdf/include/tcpdf_fonts.php on line
2072

@basepack
Copy link

basepack commented Jun 10, 2021

I only use it to merge PDF files, nothing special really. Maybe that is why it worked at once fore me.

Edit: currently also getting this error! For some reason not always when merging pdfs.

Edit2: fixed in TCPDF 6.4.1: tecnickcom/TCPDF#222

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants