[TYPO3-core] RFC #13283: Thumbnail generation broken for PDF files
Xavier Perseguers
typo3 at perseguers.ch
Sat Feb 20 10:05:05 CET 2010
Hi Benni,
> Solution:
> Make it right by adding the frame parameter by checking if it is "numeric".
- You assign "false" as default value. I thought we'd to write FALSE
- php doc states that parameter is string, I guess it should be mixed. phpdoc says either '' or a number. Why assigning a boolean as default value?
- missing spaces after commas in method signature (already in original code but we'd say we take advantage of patch to enhance the code)
BTW, what is the output of GD of a negative frame number is given?
Cheers
--
Xavier Perseguers
http://xavier.perseguers.ch/en
More information about the TYPO3-team-core
mailing list