Non-core contributed modules

Check the functionality against the new TCPDF library using lib/pdflib.php wrapper

Details

  • Type: Task Task
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.0
  • Fix Version/s: 2.0
  • Component/s: Module: Certificate
  • Labels:
    None
  • Affected Branches:
    MOODLE_20_STABLE
  • Fixed Branches:
    MOODLE_20_STABLE

Description

TCPDF shipped with Moodle was upgraded to a new version tcpdf_php5 TCPDF 4.0.015. All Moodle code should use the wrapper lib/pdflib.php. I used to have some small issues during an upgrade from tcpdf 1.53 (previsou version).

Issue Links

Activity

Hide
Chardelle Busch added a comment -

If we are going to use pdflib.php, wouldn't it make sense to go ahead and set up the config variables in this file instead of in lib/tcpdf?

We could also set some other params here, e.g. RTL lang, as well as working on the cache--see this discussion:

http://moodle.org/mod/forum/discuss.php?d=104383

Show
Chardelle Busch added a comment - If we are going to use pdflib.php, wouldn't it make sense to go ahead and set up the config variables in this file instead of in lib/tcpdf? We could also set some other params here, e.g. RTL lang, as well as working on the cache--see this discussion: http://moodle.org/mod/forum/discuss.php?d=104383
Hide
Chardelle Busch added a comment -

I've got the certificate for 2.0 running solely off of tcpdf using pdflib--so it is safe to delete fpdf.

I have added the config variables to pdflib (see attached). This will make it easier to upgrade tcpdf without having to hack any files--we can just plop it in.

Show
Chardelle Busch added a comment - I've got the certificate for 2.0 running solely off of tcpdf using pdflib--so it is safe to delete fpdf. I have added the config variables to pdflib (see attached). This will make it easier to upgrade tcpdf without having to hack any files--we can just plop it in.
Hide
David Mudrak added a comment -

Chardelle, maybe you can mark this issue as resolved

Show
David Mudrak added a comment - Chardelle, maybe you can mark this issue as resolved

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: