Skip to content
This repository has been archived by the owner on Nov 25, 2020. It is now read-only.

Thumbnail generation for JPEG images fail with latest PHP versions (5.4-5.6) #645

Closed
mmvsk opened this issue Sep 21, 2014 · 1 comment
Closed

Comments

@mmvsk
Copy link
Contributor

mmvsk commented Sep 21, 2014

After the latest upgrade of PHP (5.6) on one of our client's server, thumbnail generation for JPEG images stopped working.

After some investigation, I found that this bug is caused by an invalid use of the imagejpeg() function in plugins/editor.diaporama/PThumb.lib.php :

According to the PHP documentation of imagejpeg(), the second parameter must be either a valid filename, or null. You use an empty string, but that's invalid (before this argument was silently typecasted).

That breaking change (silent typecast vs. error throwing) was reported here, and was also fixed in another thumbnail generation library.

@mmvsk
Copy link
Contributor Author

mmvsk commented Sep 21, 2014

Issue fixed in PR #646

@cdujeu cdujeu closed this as completed in 73249f9 Sep 25, 2014
cdujeu added a commit that referenced this issue Sep 25, 2014
Fix #645: thumbnail generation for JPEG images fail with latest PHP versions (5.4-5.6)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant