This repository has been archived by the owner on Jul 5, 2021. It is now read-only.
forked from eriksape/laravel_5_x-pdf-merger
-
Notifications
You must be signed in to change notification settings - Fork 35
PHP 8 Compatibility #30
Comments
Hi, this is not applied yet, right? dhcmega |
Well, I just got it working with PHP 8 and Laravel 8 with no troubles whatsoever. |
Hi PHP Deprecated: Required parameter $currentfont follows optional parameter $str in /home |
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.
The current version will not run on PHP8. This pull request: tecnickcom/TCPDF#244 seems to fix the issue.
The text was updated successfully, but these errors were encountered: