Need help using file_exists for different extensions...

[Kinsbane]

Hey guys,

So I've got this code that forces a download when a user clicks a link. It basically looks at all the variables passed, knows which folder to look in, and sends the file to the browser. Easy.

Until I realized that for certain files, I have some that end in .zip and some that end in .sit (stuffit).

Right now it only checks for .zip files, and basically I need my function to output the proper response if either the .sit or the .zip file is there...

The premise behind the function is to check to see if a file is there, and if so, it displays a link to download the file - otherwise it says simply, "Not Available"...

Here's the function I'm working with:

function wheres_src ($type, $control, $cat) {
  	$sheet_base_dir = "/home/designcenter/public_html/datasheets/";
  	switch ($type) {
  		case "sourcel":
  		$ex = ".zip";
  		$ex2 = ".sit";
  		$fold = "/source/";
  		break;

	case "sourcea4":
	$ex = "_A4.zip";
	$ex2 = "_A4.sit";
	$fold = "/source/";
	break;
}

$filezip = $sheet_base_dir.$cat.$fold.$control.$ex;
$filesit = $sheet_base_dir.$cat.$fold.$control.$ex2;
if (file_exists($filezip) || file_exists($filesit)) {
	return true;
} else {
	return false;
}
  }

But, that doesn't work... it's still looking at only .zip stuff and it's only printing the link if it finds a zip.. but I need it to print a link if it's EITHER .sit OR .zip

Thanks in advance

[Installer]

Have you checked the value of $type, e.g.

echo $type . ',' . strlen($type);

to make sure it's right?

[Kinsbane]

Originally posted by Installer
Have you checked the value of $type, e.g.

echo $type . ',' . strlen($type);

to make sure it's right? [/B]

Yeah, $type isn't being passed dynamically when I call the variable... basically when I come to the link, I do:

if (wheres_file("sourcel", $ctrl, $cat) == true) {
echo "<link>";
} else {
echo "Not Available";
}

[Kinsbane]

Anyone care to take a stab?

[Installer]

I've set up the directories and files, and tested it a bunch, with the same results as you. I can't figure out why one works and the other doesn't. So, yeah, I guess a fresh pair of eyes is what's needed.

[Installer]

I guess not exactly the same results as you. For me, it doesn't trigger the "source1" case at all, but it does the other one ok (no matter which order I put them in).

[Kinsbane]

Originally posted by Installer
I guess not exactly the same results as you. For me, it doesn't trigger the "source1" case at all, but it does the other one ok (no matter which order I put them in).

Maybe yours WILL work after I let you know the actually "source1" is really "sourceL" - the L is lowercase.

not logical, i know, I just need to get this working before I make neat & clean.

[Installer]

I knew I must have been doing something stupid, 'cause the code looked good. So now it works perfectly for me. FWIW, here's my testing code:

$ctrl = 'test';
$cat = 'cat';
if (wheres_src("sourcea4", $ctrl, $cat) == true) { 
    echo "Found" . '<br />'; 
} else { 
    echo "Not Available" . '<br />'; 
}
echo '<br />';
if (wheres_src("sourcel", $ctrl, $cat) == true) { 
    echo "Found" . '<br />'; 
} else { 
    echo "Not Available" . '<br />'; 
}
echo '<br />';

function wheres_src ($type, $control, $cat) { 
    echo $type . '<br />';
    echo $control . '<br />';
    echo $cat . '<br />';
    $sheet_base_dir = "./test/"; 
    $ex = '';
    $ex2 = '';
    $fold = '';
    switch ($type) { 
        case "sourcea4":
            echo 'case: ' . $type . '<br />'; 
            $ex = "_A4.zip"; 
            $ex2 = "_A4.sit"; 
            $fold = "/source/"; 
            break; 
        case "sourcel": 
            echo 'case: ' . $type . '<br />'; 
            $ex = ".zip"; 
            $ex2 = ".sit"; 
            $fold = "/source/"; 
            break; 
    } 

$filezip = $sheet_base_dir.$cat.$fold.$control.$ex; 
echo $filezip . '<br />';
$filesit = $sheet_base_dir.$cat.$fold.$control.$ex2;
echo $filesit . '<br />';
if (file_exists($filezip) || file_exists($filesit)) { 
    return true; 
} else { 
    return false; 
} 
}

/** Displays:

sourcea4
test
cat
case: sourcea4
./test/cat/source/test_A4.zip
./test/cat/source/test_A4.sit
Found

sourcel
test
cat
case: sourcel
./test/cat/source/test.zip
./test/cat/source/test.sit
Found

**/