I am having a problem checking the existance of a file. I have ran the code used to check for the file in a seperate test file and it works fine. For some reason when present in this script it doesn't execute the needed code, which makes me assume that the -e if returning 0. In my test file I simulated the code as exactly as possible with regards to nested ifs with a foreach and the method in which the filename is described (${save_dir}${filename}). Anyone see what I'm missing?

#!/usr/bin/perl -w
use strict;
use HTML::SimpleLinkExtor;
use LWP::Simple;
use Image::Grab;
 
my $save_dir = ">/home/dir/";

my $page = 'http://website.com/webpage.html';

my $link;
my $image;
my $counter = 0;
my $pic = Image::Grab->new();
my $html = get($page);
my $extor = HTML::SimpleLinkExtor->new();
$extor->parse($html);
my @page_links = $extor->links;

my @temp = split(/\//, $page);
pop(@temp);
my $page_dir = join('/', @temp) . "/";
@temp ="";
print "$page_dir\n";

foreach $link (@page_links) {
    if ($link =~ /jpg$/) {    
        next if $link =~ /small.jpg$/;                    
        print $link . "\n";
 $counter++;
        my @temp = split(/\//, $link);
 my $filename = pop(@temp);       
 
 #Here is the suspect code
 if (-e "${save_dir}${filename}") {       
     print "$filename exists, skipping file";
     next;
 }
 print "Save Dir = ${save_dir}${filename}\n";
 print "Image URL = ${page_dir}${link}\n";
 $pic->url($page_dir . $link);
 $pic->grab;
 open(IMAGE, "${save_dir}${filename}") || die "$!";
 print IMAGE $pic->image;
 close IMAGE;
 print "${counter}: Saved $link \n\n";
    }
}
print "Finished!\n";
[download]

----
Nothing in the world can take the place of persistence. Talent will not; nothing is more common than unsuccessful men with talent. Genius will not; unrewarded genius is almost a proverb. Education will not; the world is full of educated failures. Persistence and determination alone are omnipotent. --Calvin Coolidge (1872-1933)