Quote:
|
Originally Posted by Silvy
Not being disrespectful, but this is a case where the last part really is the most important so I removed most of the quote to get to the good stuff  :
Very close! You see, the file_exists function looks for files relative to it's own location. Since brewery.php is in www/brewery/ the above file_exists will look for www/brewery/imgs/brewery_1.jpg
What we need is it to look for 1 directory higher than itself.
This can be done in one of 3 ways:
By using the full path of the file (so it looks for an absolute path, instead of a relative path.) (file_exists("/var/www/localhost/http/whatever/www/img/brewery_1.jpg")
By using a relative path, but to look one dir higher (file_exists("../imgs/brewery_1.jpg")
By using a global directive to point to the file locations.
This one should be relatively easy to grasp so we're going to do this the most elegant way (via the global constant).
Since all images are always located in www/imgs let's have index.php set the image_path constant:
(put this in the top-section of index.php, makes it easier to find later)
PHP Code:
<?php
define ('IMAGE_FS_PATH', '/var/www/img/'); //this should be the full filesystem path to your images
define ('IMAGE_WWW_PATH', 'img/'); //this should be the path relative to the URL. (so it will expand to http://www.arch13.com/img/ etc)
?>
Then, change the $base statement, the first file_exists statement, and the echo statement in brewery.php to use the new paths:
PHP Code:
$base = "brewery_"; //like you said
file_exists(IMAGE_FS_PATH.$base.$id.'jpg'; //this expands into '/var/www/img/brewery_1.jpg
echo "<img src='".IMAGE_WWW_PATH.$base.$id".jpg'>"; this expands into imgs/brewery_1.jpg, which will allow the browser to find it relative to "http://www.arch13.com"
And leave the rest of the code as-is.
Does it work now? |
Nope
I see two problems going on here, and I'm trying to see how we can fix them.
The first is actually straight foreward, but I almost didn't spot it.
We can't have $base= "brewery_"; like I suggested. It needs to be just "brewery" since the base statment is also used for finding the paths. Throwing in the underscore to makes the images work but makes the file paths
not work.
Perhaps
file_exists(IMAGE_FS_PATH.$base._$id.'jpg';
But that screws up the code (I tried it). It parse errors like so:
Parse error: parse error, unexpected T_VARIABLE in /homepages/6/d93661089/htdocs/brewery_test.php on line 52
Line 52 was if(file_exists(IMAGE_FS_PATH.$base._$id.'jpg'; )) { , so that was a bad idea on my part to try.
Honestly, It's simplest for me to just change the image names to brewery1.jpg, since that should make the code work. I'll try it right now before continuing this post...
Nope
Okay, here are lines 49-59 of brewery.php that don't want to co-operate
PHP Code:
<?php
$base = 'brewery_'; //only for images if they are called brewery1.jpg etc.
$id = $_REQUEST['id2'];
if(file_exists(IMAGE_FS_PATH.$base.$id.'jpg')) { //this expands into '/var/www/img/brewery_1.jpg
echo "<img src='".IMAGE_WWW_PATH.$base.$id'.jpg'>"; this expands into imgs/brewery_1.jpg, which will allow the browser to find it relative to "http://www.arch13.com"
} elseif(file_exists($id.".php")) {
include($id.".php");
} else {
echo "<img src='brewerysplash.jpg'>";
}
?>
I set
PHP Code:
<?php
define ('IMAGE_FS_PATH', '/img/'); //this should be the full filesystem path to your images
define ('IMAGE_WWW_PATH', 'img/'); //this should be the path relative to the URL. (so it will expand to http://www.arch13.com/img/ etc)
?>
Before the body html in index.php right under my javascripts, is that an okay position?
The system path is correct, as the php install is supposed to see my / as the root and works in all instances I've used it as such. It's not parse erroring, brewery_test is, so I assume it's working fine.
I have two copies of every image in the /imgs/brewery folder. One of each named brewery_1.jpg and one of each named brewery1.jpg. The file_exists is seeing neither right now.
Any suggestions?
The file I have been playing with is
www.arch13.com/index.php?id=brewery_test