-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
How to support php 7.1 #22
Comments
Theoretically, it is already supported. If you are experiencing issues, feel free to report here (if it is related to this library) or report to mpdf repo (in case it is related to mpdf directly) |
No, PHP 7 is not actually supported. It is required to update mpdf first, but then yii2-pdf is not compatible with mpdf v7. To make it compatible one need to change class name from mPDF to Mpdf and Mpdf accepts single input parameter that is actually array of mPDF parameters. See post: https://harrybailey.com/2018/08/yii-1-php-7-1-yii-pdf-and-mpdf/ I would like to update in a clean way yii2-pdf and mpdf v7. Is there an easy way to do it? Shall I fork yii2-pdf and make it work for mpdf v7? |
Feel free to submit a PR, or to become a contributor :-) |
I am not big git master, but I think I managed to send two pull requests with changes that might do the job. And you can mark it as new version after merge... |
@robregonm i have made a PR for this issue #29 |
How to support php 7.1
The text was updated successfully, but these errors were encountered: