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.

Sign up for GitHub

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

Jump to bottom

Remove TCPDF from the list of PDF renderers #399

Closed
ghost opened this issue Oct 18, 2014 · 3 comments
Closed

Remove TCPDF from the list of PDF renderers #399

ghost opened this issue Oct 18, 2014 · 3 comments
Labels
HTML Open XML (Word 2007+) Portable Document (PDF)
Milestone
0.13.0

Comments

@ghost
Copy link

ghost commented Oct 18, 2014

Hi, @PHPOffice/phpword-team.

To get PDF from OOXML we use intermediary conversion into HTML (OOXML -> HTML -> PDF). For the second step we offer three options: dompdf, mPDF, TCPDF. As a matter of fact, TCPDF is not intended for HTML-to-PDF conversion. I foresee lot of problems with this fact and suggest to remove TCPDF from the list.
@ghost ghost self-assigned this Oct 18, 2014
@franzholz
Copy link

I do not think that TCPDF should be removed, because dompdf is anyways recommended as the first choice. "TCPDF is not a web browser and is not originally intended to be used as HTML to PDF converter."
This does not mean that TCPDF cannot be used as a HTML to PDF converter.

@rvanlaak
Copy link

What about using wkhtmltopdf, which is used in Google Chrome for converting html into a PDF? It does even supports lot of css3 properties.

Better should be to integrate KnpLabs/snappy which is a wrapper for wkhtmltopdf.

@ghost ghost added this to the v0.13.0 milestone Jul 30, 2016
@ghost ghost mentioned this issue Jul 30, 2016
Open
@ghost
Copy link
Author

ghost commented Jul 30, 2016

Created another issue for Snappy support.

TCPDF becomes deprecated since v0.13.0 due to its limited HTML support.

@ghost ghost closed this as completed Jul 30, 2016
ghost pushed a commit that referenced this issue Jul 30, 2016
https://github.com/PHPOffice/PHPWord/issues/399.
54013a6
ghost pushed a commit that referenced this issue Jul 30, 2016
https://github.com/PHPOffice/PHPWord/issues/399.
7817524
veggiefrog added a commit to ciniki/ciniki that referenced this issue Sep 11, 2016
@veggiefrog
Module Updates:
ea6fc70 
* site/ciniki-lib/PHPWord 38cb04d...0a3f873 (98):
  > Updated change log.
  > Updated change log.
  > Updated change log.
  > Updated copyright.
  > Removed deprecated code.
  > PHPOffice/PHPWord#399.
  > PHPOffice/PHPWord#399.
  > PHPOffice/PHPWord#335.
  > Fixed formatting.
  > Fixed tests.
  > #483. Output escaping for RTF.
  > Fixed build.
  > #483. Output escaping for HTML.
  > Fixed build.
  > #483. Output escaping for ODF.
  > Fixed build.
  > #483. Output escaping for OOXML.
  > #483. Output escaping for OOXML.
  > Merge pull request #807 from githubjeka/patch-1
  > Merge pull request #804 from githubjeka/patch-1
  > Merge pull request #803 from githubjeka/patch-1
  > #618.
  > #548.
  > #704.
  > Reviewed and merged #612.
  > Reviewed and merged #612.
  > Improved coverage for `\PhpOffice\PhpWord\Element\Section` (#665).
  > Improved coverage for `\PhpOffice\PhpWord\Element\Section` (#665).
  > Improved coverage for `\PhpOffice\PhpWord\Element\Section` (#665).
  > Improved coverage for `\PhpOffice\PhpWord\Element\Section` (#665).
  > Add PHP 7.0 into the build script.
  > Reviewed and merged #658.
  > #617.
  > Merge pull request #617 from kazitanvirahsan/develop
  > Copyright update.
  > Copyright update.
  > Copyright update.
  > Removed custom autoloader.
  > Reorganized unit tests.
  > Reorganized unit tests.
  > Reorganized unit tests.
  > Merge branch 'master' into develop
  > Added backward compatibility for deprecated alignment options.
  > Added backward compatibility for deprecated alignment options.
  > Added backward compatibility for deprecated alignment options.
  > Added backward compatibility for deprecated alignment options.
  > Added backward compatibility for deprecated alignment options.
  > Added backward compatibility for deprecated alignment options.
  > Added backward compatibility for deprecated alignment options.
  > Added backward compatibility for deprecated alignment options.
  > Updated change log due to the changes previously committed.
  > Changed init value of the NumberingLevel.alignment property.
  > Fixed build.
  > Fixed build.
  > Next round of refactoring of alignment-related functionality.
  > Released v0.12.1.
  > #513.
  > #532.
  > Removed redundant *.md documentation.
  > Playing with Travis CI.
  > Playing with Travis CI.
  > Playing with Travis CI.
  > Playing with Travis CI.
  > Fix for #532.
  > Fix for #532.
  > Fix for #532.
  > `.travis.yml` cleanup.
  > `.travis.yml` cleanup.
  > `.travis.yml` cleanup.
  > `.travis.yml` clanup.
  > Performance improvement for #513.
  >  Performance improvement for #513.
  > Changed Gitter badge to the Shields version.
  > Added Gitter badge.
  > Shortened names of ``TemplateProcessor`` properties.
  > Merge remote-tracking branch 'origin/develop' into develop
  > Implementation for #371.
  > Reimplemented #498.
  > Performance improvement for #513.
  > Performance improvement for #513.
  > Performance improvement for #513.
  > #495.
  > Removed typo.
  > #495.
  > #495.
  > #495.
  > Fixed broken paragraph alignment for OpenXml.
  > Fixed broken paragraph alignment for OpenXml.
  > Fixed broken paragraph alignment for OpenXml.
  > Fixed broken paragraph alignment for OpenXml.
  > Minor refactoring.
  > Minor refactoring.
  > Minor refactoring.
  > Minor refactoring.
  > #51 (updated tests, refactored a little).
  > Fixed broken paragraph alignment for OpenXml.
  > #51 (updated tests, refactored a little).
  > #51 (updated tests, refactored a little).

* site/ciniki-mods/artcatalog 7358fdb...44107dc (3):
  > Updated searchQuick for new business menu
  > Fixed bug with uiSettings
  > Updated hooks/uiSettings for new business menu

* site/ciniki-mods/artgallery c87caa6...76dfe54 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/artistprofiles 44e5e87...a52178c (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/atdo bf2eafe...02a4b48 (2):
  > Added colour coding to business main menu task search
  > Updated hooks/uiSettings for new business menu

* site/ciniki-mods/blog b2cb912...005d311 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/bugs f741c94...e414fa8 (1):
  > Updated hooks/uiSettings for new business menu

* site/ciniki-mods/businesses 63578da...8510aec (2):
  > Got calendars working
  > Updated business menu to use hooks/uiSettings

* site/ciniki-mods/classes 83dc3f4...117c63f (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/clicktracker 7881cf0...1946204 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/conferences 4cc24fb...fcada2c (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/courses d8e9371...8e69b01 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/customers 2b5ffe1...61cacbf (4):
  > Added colour coding to search results for business main menu customer/member search
  > Fixed up uiSettings for salesreps
  > Merge branch 'master' of git://github.com/ciniki/customers
  > Updated hooks/uiSettings for new business menu

* site/ciniki-mods/directory 758ffbb...38281bd (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/donations adf1396...82a56c8 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/events 3b95f52...249baf4 (3):
  > fixed bug in uiSettings
  > Fixed bugs with uiSettings
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/exhibitions 02b62ef...ad69e62 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/fatt 08514dd...2821c84 (1):
  > Updated hooks/uiSettings for new business menu

* site/ciniki-mods/filedepot 93a94e8...2046353 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/filmschedule 2f86d62...acab8c6 (2):
  > Fixed bugs in uiSettings
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/gallery 86ec5c3...2507c9c (2):
  > Fixed bugs with uiSettings
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/info 16d996e...e48371f (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/jiji 0fb1f9e...520c3b6 (2):
  > Fixed bugs with uiSettings
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/landingpages f699b04...8b7985f (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/library bfc2f3d...b764dc6 (1):
  > Add hooks/uiSettings for new business menu

* site/ciniki-mods/links d308592...114bf79 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/mail 3a4cd44...0fa7cfe (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/marketing 8d6dd06...c915e73 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/marketplaces c093836...6752f7c (2):
  > Fixed bugs with uiSettings
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/materiamedica e855ed7...a77df07 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/membersonly 34f94b5...e3705a4 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/merchandise 1dcf9d8...99fcbb3 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/newsaggregator ddc995f...7e14ba1 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/newsletters 11c5ad2...f94c4b1 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/patents 96ee84e...fd1344b (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/products 32fedab...dc449e6 (1):
  > Updated hooks/uiSettings for new business menu

* site/ciniki-mods/projects b25d15e...91ccbde (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/propertyrentals ee6f945...7dadb83 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/recipes 2418e31...ed23b5b (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/reseller fa83692...0a22d57 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/sapos 5ad7617...32ab6f5 (1):
  > Updated hooks/uiSettings for new business menu

* site/ciniki-mods/sponsors 527d34d...ca7cd3d (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/subscriptions dc6a28a...3d576f6 (4):
  > Fixed bugs with uiSettings
  > Updated uiSettings
  > Fixed bugs in subscriptions
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/surveys 38d521a...7c3fd2b (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/systemdocs ecf0ca0...f4e13df (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/toolbox b55bb2d...016c72e (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/tutorials 5af9074...30cebb6 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/users cbeac1b...d3a7d4c (1):
  > Fixed bug with missing include in auth.php

* site/ciniki-mods/web f3445d1...3eb345b (1):
  > Updated hooks/uiSettings for new business menu

* site/ciniki-mods/wineproduction 92c1732...e3ba047 (1):
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/workshops 075d603...56fd4b1 (2):
  > Fixed bugs with uiSettings
  > Added hooks/uiSettings for new business menu

* site/ciniki-mods/writingcatalog 7537b3b...e635970 (1):
  > Added hooks/uiSettings for new business menu
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
HTML Open XML (Word 2007+) Portable Document (PDF)
Milestone
0.13.0
Development

No branches or pull requests
2 participants
@rvanlaak @franzholz