Bug 965160 - Bundled library: tcpdf in moodle
Bundled library: tcpdf in moodle
Product: Fedora
Classification: Fedora
Component: moodle (Show other bugs)
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Gwyn Ciesla
Fedora Extras Quality Assurance
Depends On: 964412
  Show dependency treegraph
Reported: 2013-05-20 11:00 EDT by Remi Collet
Modified: 2013-05-20 12:58 EDT (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2013-05-20 12:58:54 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Remi Collet 2013-05-20 11:00:07 EDT
Moodle provides a bundled copy of tcpdf while system library is now available.
Comment 1 Gwyn Ciesla 2013-05-20 11:25:30 EDT
This looks like there are some files in moodle's tcpdf that aren't in php-tcpdf, do you have any insight here?

This is moodle/lib/tcpdf from 2.5:

2dbarcodes.php  CHANGELOG.TXT  config          encodings_maps.php  htmlcolors.php  pdf417.php  readme_moodle.txt  spotcolors.php  tcpdf.crt  tcpdf_filters.php  tcpdf_parser.php  unicode_data.php
barcodes.php    composer.json  datamatrix.php  fonts               LICENSE.TXT     qrcode.php  README.TXT         sRGB.icc        tcpdf.fdf  tcpdf.p12          tcpdf.php
Comment 2 Remi Collet 2013-05-20 12:10:17 EDT
After a quick look.

Moodle 2.5 bundles php-tcpdf 5.9.209 (not so old, 2013-03-14).

Using system php-tcpdf shoud work "out of the box", just need to edit the moodle/lib/pdflib.php to fix paths.

The package need to requires:

Most important changes between 5.9 and 6.0 was the "big" tcpdf class being split. So some call to $this->foo() have to be changed to, p.e. tcpdf_static::foo(), but I don't see any of them (very quick look).

In all case, all changes should happen in pdflib.php.

Of course, this must be tried.
Comment 3 Gwyn Ciesla 2013-05-20 12:16:18 EDT
Excellent, thanks.  I'll implement the above in 2.5 in rawhide only, and we'll go from there.

Note You need to log in before you can comment on or make changes to this bug.