crnr / timthumb

Automatically exported from code.google.com/p/timthumb
0 stars 0 forks source link

file exists works better with an absolute url #269

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. create a symbolic link to timthumb.php in the place where you would use it
2. create timthumb-config.php in the same location as timthumb.php

I have this setup since many of the themes use this one script and I would like 
to synchronize upgrades.

What is the expected output? What do you see instead?

TimThumb does not load the config file

What version of the product are you using? On what operating system?

Latest from svn today

I have attached a patch that solves this issue for me. Another solution is to 
search for timthumb-config.php in the include patch, ie. do a try...catch 
around the include rather than using file_exists altogether.

Original issue reported on code.google.com by hnass...@gmail.com on 29 Sep 2011 at 4:52

Attachments:

GoogleCodeExporter commented 8 years ago
thanks for the tip. I have included the patch.

I notice you removed the is404 method as well. Is that intentional?

Original comment by BinaryMoon on 12 Oct 2011 at 8:28

GoogleCodeExporter commented 8 years ago
No the patch only affects one line, there is a note in the end about that one 
of the files has a newline in the end where the other doesn't that can be 
ignored, its just the one line in the patch.

Original comment by hnass...@gmail.com on 12 Oct 2011 at 8:41

GoogleCodeExporter commented 8 years ago
in that case it's been commited. Thanks again :)

Original comment by BinaryMoon on 13 Oct 2011 at 8:37