silverstripe / cwp-pdfexport

Adds PDF export capability for CWP pages
BSD 3-Clause "New" or "Revised" License
4 stars 6 forks source link

NEW Import PDF export functionality from BasePage in cwp/cwp #1

Closed robbieaverill closed 6 years ago

robbieaverill commented 6 years ago

Issue: https://github.com/silverstripe/cwp/issues/31

Requires https://github.com/silverstripe/cwp/pull/32

robbieaverill commented 6 years ago

Noted issue: clicking the "Export to PDF" link for a page initially fails with this message:

Fatal error: wkhtmltopdf failed: Exit with code 1 due to network error: ContentNotFoundError in /webroot/vendor/cwp/cwp-pdfexport/src/Extensions/PdfExportControllerExtension.php on line 164

Subsequent requests succeed and open the PDF.

I suspect this could be a macOS issue with piping the wkhtmltopdf output to stdout. If we're happy, I think we should get this merge and raise that as a separate issue. We could write it to the PHP temp dir instead of piping to stdout.

robbieaverill commented 6 years ago

I've raised an issue for the error I noted above here: https://github.com/silverstripe/cwp-pdfexport/issues/2