Question 42092 in PHP Question posted in
Official manual can be found here.

PHP get files in directory and list if they are within the last year (filetime)

I need to output the contents of a directory only if the file has been created within the last year.

<?php
    $dirpath = "Dir/foo/bar/";
    $files = array();
    $files = glob($dirpath . "*");
    rsort($files);
    $today = date("Y-m-d");
    $lastYear = date("Y-m-d", strtotime("-1 years"));
    $todayTimeStamp = strtotime($today);
    $beginningTimeTtamp = strtotime($lastYear);
    foreach($files as $item) {
        if(filetime($item)  > $beginningTimeStamp && < $todayTimeStamp) {
            echo "basename($item)";
         }
     }

This doesn’t return anything – I just want to return $item that is greater than the date a year from today and less than today (not necessary?).

Am I wrong converting the $beginningTimeStamp and $todayTimeStamp to a timestamp to compare filetime? Im not able to convert each $item to its filetime… Is there a better way to do this?

Tags:

2

Answers


  1. 0

    Your code has some typos like filetime instead of filemtime, etc. I corrected it:

    $dirpath = "Dir/foo/bar/";
$files = [];
$files = glob($dirpath . "*");
rsort($files);
$today = date("Y-m-d");
$lastYear = date("Y-m-d", strtotime("-1 years"));
$todayTimeStamp = strtotime($today);
$beginningTimeStamp = strtotime($lastYear);
foreach ($files as $item) {
    print "<br>" . filemtime($item) . '<hr>';
    if (filemtime($item) > $beginningTimeStamp && filemtime($item) < $todayTimeStamp) {
        echo "basename($item)";
    }
}

    I also corrected the if statement and also $beginningTimeTtamp had a typo -> $beginningTimeStamp

    Login or Signup to reply.
  2. 0

    Several issues. You want to use timestamps. There’s no reason to create timestamps, format them and then convert them back to timestamps. Also, there’s an error in the if. This will work:

    foreach($files as $item) {
    $mt = filemtime($item)
    if($mt > strtotime('-1 year') && $mt < time()) {
        echo "basename($item)";
     }
 }

    However, unless something might have a weird time in the future, you just need:

    foreach($files as $item) {
    if(filemtime($item) > strtotime('-1 year') {
        echo "basename($item)";
     }
 }
    Login or Signup to reply.
Please signup or login to give your own answer.

Back To Top
Search