Skip to content
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

Do not allow tecnickcom/tcpdf 6.3.x #10

Closed
wants to merge 1 commit into from
Closed

Conversation

fritzmg
Copy link
Contributor

@fritzmg fritzmg commented Jan 21, 2020

Version 6.3.0 through 6.3.2 (the latest version) of tecnickcom/tcpdf (used by the contao/tcpdf-bundle of course) has a serious issue where files used in the PDF get deleted after generation: tecnickcom/TCPDF#158

@Total-Reality
Copy link

@fritzmg
Copy link
Contributor Author

fritzmg commented Jan 21, 2020

Although, this does not happen under regular circumstances within Contao (i.e. printing an article as PDF). Only if you use code generating a custom PDF and using $pdf->Image(…) yourself.

@fritzmg fritzmg closed this Jan 21, 2020
@Total-Reality
Copy link

Wen interessiert das? Contao ist doch nicht nur darauf ausgelegt, dass alle Core-Funktionen korrekt funktionieren, sondern es versteht sich als CMS mit vielen Erweiterungen, die ebenfalls funktionieren sollten. Es ist grundsätzlich ein Bug.

@aschempp
Copy link
Member

@Total-Reality wenn deine Erweiterung oder deine Applikation Bilder verwendet, dann solltest du die entsprechende tcpdf-Bibliothek selber (zusätzlich) requiren. Damit stellst du sicher dass sie immer da ist, auch wenn Contao sie z.B. nicht mehr benötigen würde. Und damit stellst du auch sicher dass du eine Version bekommst welche funktioniert (in diesem Fall die ohne diesen Bug).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants