[RESOLVED] best method to get a directory listing by ID

dmacman1962

Hi All,

I have a work order form that our employees will use to order website changes and I let them upload attachments also. I am appending the work order id to the beginning of the files names. IE 123-image.jpg etc.

Here is how I am uploading the files.

$ID = mysql_insert_id(); //get the id of the last to append to the beginning of the filenames

$filename1 = "/path to directory/staff/uploads/" . $ID . "-" . $FILE1_name ;
$filename2 = "/path to directory/staff/uploads/" . $ID . "-" . $FILE2_name ;

@copy("$FILE1" , $filename1);
@copy("$FILE2" , $filename2);

And I can read them back as such. (thanks to Bpat's post)

$list = ftp_nlist($ftp, $dir);

foreach($list as $file)
{
  if(!is_dir($file) && !is_dir($dir.'/'.$list))
    echo '<a href="path to directory/staff/uploads/', $dir, '/', $file, '">', $file, '</a>';
}

But that would list all the files. I want to limit it to the files with the same $ID at the beginning of the name. I would imagine an 'explode' might work, but I am not clear on coding that.

Any suggestions, or has somone seen a post that does that already?

Thanks in advance.
Don

etully

When you list the files, I assume you know the $ID that you want to restrict to? If so, then you use regular expressions to check to make sure that the filename starts with the desired ID by wrapping the echo statement in an if statement like this:

if (preg_match("/^{$ID/",$file))} {
echo '<a href="path to directory/staff/uploads/', $dir, '/', $file, '">', $file, '</a>';
}

laserlight

You can check if $ID is at the start of the filename with [man]strpos/man, e.g.,

$list = ftp_nlist($ftp, $dir);

foreach($list as $file)
{
  if(strpos($file, $ID) === 0 && !is_dir($file) && !is_dir($dir.'/'.$list))
    echo '<a href="path to directory/staff/uploads/', $dir, '/', $file, '">', $file, '</a>';
}

etully

ereg would also work like this:

if (ereg("^$ID",$file)) {

and since your echo statement is already wrapped in an IF statement, you can simplify the code by adding this test to your existing if statement like this:

foreach($list as $file)
{
if((!is_dir($file) && !is_dir($dir.'/'.$list)) and ereg("^$ID",$file))
echo '<a href="path to directory/staff/uploads/', $dir, '/', $file, '">', $file, '</a>';
}

If I remember correctly, ereg is faster than preg_match (but less powerful). Combining the if statements will also be faster. For your purposes, speed won't be an issue but it's good to learn to be fast so that your code will work well when you have a site that performs 100,000 of these statements every second.

laserlight

If I remember correctly, ereg is faster than preg_match (but less powerful). Combining the if statements will also be faster. For your purposes, speed won't be an issue but it's good to learn to be fast so that your code will work well when you have a site that performs 100,000 of these statements every second.

In this case strpos() would be the fastest, but the PHP Manual actually recommends [man]preg_match/man over [man]ereg/man by stating that "preg_match(), which uses a Perl-compatible regular expression syntax, is often a faster alternative to ereg()".

Oh, and please use the php forum bbcode tag for posting PHP code snippets.

dmacman1962

Thanks to both of you. Here is what I ended up with...

$ftp_host = "x";
$ftp_user = "y";
$ftp_password = "z";

//Connect
$connftp = ftp_connect($ftp_host);
$login = ftp_login($connftp, $ftp_user, $ftp_password);

//
//Enable PASV ( Note: must be done after ftp_login() )
//
$mode = ftp_pasv($connftp, TRUE);

//Login OK ?
if ((!$connftp) || (!$login) || (!$mode)) {
   die("FTP connection has failed !");
} else {
echo '<br /><b>Attached files</b><br />---------------------------<br />';
//
//Now run ftp_nlist()
//
$path = "http://www.site/";
$file_list = ftp_nlist($connftp, "/public_html/staff/uploads/");
foreach ($file_list as $file)
{
  echo "<a href=".$path.$file.">".$file."</a>";
}
echo '<br />---------------------------<br />';
}
//close
ftp_close($connftp);

Works great.

Thanks!!!
Don