search

kartik-v / mpdf

Fork of the mPDF latest DEV library (unofficial) with composer and packagist support.
GNU General Public License v2.0
40 stars 55 forks source link

Fix deprecated class constructor #5

Closed vyants closed 8 years ago

vyants commented 8 years ago

Library throw PHP Deprecated Warning at PHP7.

danielhjames commented 8 years ago

Hi @vyants, hi @kartik-v, there is a new community repo for mPDF where PHP 7 support is being worked on. You might want to check out issues such as https://github.com/mpdf/mpdf/issues/68 and the development branch https://github.com/mpdf/mpdf/commits/development Cheers!

kartik-v commented 8 years ago

@danielhjames thanks. Will check and update ...

hscstudio commented 8 years ago

thank awesome.. running well php7 #yii2

MassimilianoMancini commented 8 years ago

Hi @kartik-v, any chance to merge the pull request of @vyants?

It is needed to fix warning constructor https://github.com/kartik-v/yii2-mpdf/issues/27

kartik-v commented 8 years ago

Will merge this... however am thinking of phasing this repo out slowly and directly using the mpdf repo suggested by @danielhjames for the yii2-mpdf extension.

MassimilianoMancini commented 8 years ago

Just tested and it works like a charm.

however am thinking of phasing this repo out slowly and directly using the mpdf repo suggested by @danielhjames for the yii2-mpdf extension.

Yes, it will be the right choice. I'm looking forward.

Thanks!

Jeff-P commented 8 years ago

Have you guys noticed an increase on speed when doing multiple page pdfs?