Case issue

Topics: User Forum
May 21, 2008 at 8:51 AM
Hello, 

Shared/PDF.php includes /Shared/PDF/FPDF.php  but the file when I downloaded it was fpdf.php you should update this to include the file using the correct case...

This is really cool by the way,  I am really enjoying using this with my project,  thank you!

Chris
Coordinator
May 21, 2008 at 12:25 PM
Will be fixed!
Developer
Jun 8, 2008 at 8:52 PM
Now I notice this too, but I think it may after all be better to rename fpdf.php to FPDF.php after download and perhaps provide necessary instructions to do this in the documentation. The file should probably have been named FDPF.php in the first place.

I am saying this because the class name is FPDF (in uppercase) as one sees if one opens the file. Then we will be following the usual naming convention which makes it possible to use this autoload function so one does not explicitly have to include classes when they are needed

function __autoload($class) {
    require_once str_replace('_', '/', $class) . '.php';
}


After all, the FPDF class may be useful not only in connection with PHPExcel. Renaming to FPDF.php makes it readily available via the above autoload function.
Coordinator
Jun 9, 2008 at 8:02 AM
A fix will be available in release 1.6.2.