Recent changes to bugs

#63 Use Imagick php class

James Heinrich — Fri, 05 Sep 2014 22:07:20 -0000

That sounds like a fine feature request (not a bug report). It would be welcome at the proper support locations:
http://support.silisoftware.com/ = forum for discussion, bug reports and feature suggestions)
https://github.com/JamesHeinrich/phpThumb = always the most current version. Please submit any patches here.

Use Imagick php class

Rodionov Dmitry — Fri, 05 Sep 2014 21:16:09 -0000

Functions passthru+system+exec+shell_exec are disabled, but php imagick extension with imagick class is enable. Would be great to use imagick class if all command functions were disable.

Rotation parameter differs between GD and convert

Xemle — Sat, 17 Nov 2012 19:09:55 -0000

The parameter ra is defined as "Rotate by Angle: angle of rotation in degrees positive = counterclockwise, negative = clockwise". This is OK to use positive angle for counterclockwise rotation for GD extension. However, if convert is preferred through config\_prefer\_imagemagick, the rotation angle is passed through as '-rotate' parameter. Convert uses clockwise rotation by default which is the opposite of GD parameter of phpThumb. So it is not possible to use both converter at the same time without patching phpThumb or a separate rotation parameter switch.

AntiOffsiteLinking Error - False positive

Mark Jenkinson — Thu, 13 Sep 2012 21:55:52 -0000

When $PHPTHUMB\_CONFIG\['nooffsitelink\_enabled'\] = true; there are cases where thumb generation throws a AntiOffsiteLinking error. I've realized the error occurs when $\_SERVER\['HTTP\_HOST'\] returns a domain name with the www prefix but the thumbnail is being generated for a user who is accessing the site without the www. So, if a thumbnail generation task is requested from mysite.com it would incorrectly identify www.mysite.com as an unauthorized source for thumbnail linking.

config_allow_src_above_docroot not taking custom value

martinsalinas — Fri, 13 Apr 2012 15:20:03 -0000

I'm using phpThum as a plugin for MeioUpload behavior on CakePHP2 On phpThumb.config.php i set: $PHPTHUMB\_CONFIG\['allow\_src\_above\_phpthumb'\] = true; but then on phpthumb.class.php, method ResolveFilenameToAbsolute it checks: if $\!$this->config\_allow\_src\_above\_docroot && .... that was causing me some troubles, so if just above that check I var\_dump\($this->config\_allow\_src\_above\_docroot$, it says it's FALSE. i set it to true on line 142: var $config\_allow\_src\_above\_docroot = true; and now it's working. shouldn't phpthumb be mapping those values overriden in phpThumb.config.php?

External Image Retrieval Failed

Hidayat Febiansyah — Thu, 12 Jan 2012 20:06:03 -0000

Hi, I used phpThumb along with autothumb in as a wordpress plugin. I experienced error in retrieving external image from other sites. The error was: "403 Forbidden" I tried to trace the error source, and found out that the source of the error is because the URL has been altered. In my case, the url didn't have any query or what soever. It's a seems to be a static image, that in the code, it was added with a "?" letter. Finally, I can resolve it by adding a condition that if the URL has no query, I will append nothing. The changed file was phpthumb.functions.php on line 746 for the last beta file I found in this project. I suggest this change: $rawData = phpthumb\_functions::URLreadFsock$@$parsed\_url\['host'\], @$parsed\_url\['path'\].\(@$parsed\_url\['query'\] ? '?'.@$parsed\_url\['query'\] : ''$,, $errstr, true, $@$parsed\_url\['port'\] ? @$parsed\_url\['port'\] : 80$, $timeout\); Since I don't know yet how to do editing on the project source code, I thought of using svn or git, but I couldn't find one, so I include the file in this report. Thank you for making a great code. Regards,

Cache cleanup not working

Arunas Mazeika — Thu, 17 Nov 2011 13:27:03 -0000

It seems that there's a problem with the regexp located at line 695 of phpthumb.class.php. The regexp should not contain the ^ char at the beginning of the pattern as otherwise cached files inside subdirs won't be matched. Line 695 should be changed from: if $preg\_match\('\#^'.preg\_quote\($this->config\_cache\_prefix$.'\#i', $fullfilename\) && file\_exists$$fullfilename$\) \{ to: if $preg\_match\('\#'.preg\_quote\($this->config\_cache\_prefix$.'\#i', $fullfilename\) && file\_exists$$fullfilename$\) \{ phpThumb being used is v1.7.11-201108081344 - August 8, 2011. Cheers, Arunas

ResolveFilenameToAbsolute returns path relative to phpThumb

Anonymous — Thu, 03 Nov 2011 13:29:46 -0000

We use phpThumb 1.7.11 $beta$ inside a folder of 3rd party libraries off root, the ResolveFilenameToAbsolute resolves the sourceFilename relative to the phpThumb folder, NOT the document folder. The simple fix is in phpthumb.class.php uncomment line 1071 and comment out line 1072 to change it back to the code used in 1.7.9.

patch note

Anonymous — Wed, 24 Nov 2010 12:15:40 -0000

Hi, a bug happening while php cash during image processing caused the temp dir to inflate dramaticaly. here is my patch : in class::method : phpthumb::CleanUpCacheDirectory add : if $rand\(0, 500$ < 10\) \{ exec$"find ".escapeshellarg\($this->config\_temp\_directory$." -cmin +5 -maxdepth 1 -type f -delete"\); \} ps thanck for your work\!

no extensions files

Anonymous — Sun, 21 Nov 2010 19:48:13 -0000

in the cache folder phpthumb creates no-extensions files $like for example: "pThumb6C19Ap"$ with"600" permission which never get deleted.This is a big issue, space used is increasing very fast... looks to me that these files are copies of the original images.