feature #9360 [Finder] Fix finder date constraints and tests (ruian)

fabpot · fabpot · commit f132197df618 · 2013-12-27T22:51:59.000+01:00
This PR was merged into the 2.5-dev branch. Discussion ---------- [Finder] Fix finder date constraints and tests Description: When you search some files with date constraints, all folders are given, even if they are not in the date scope. | Q | A | ------------- | --- | Bug fix? | yes | New feature? |no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #9303 | License | MIT Commits ------- 4ccafa6 Fix finder date constraints and tests
diff --git a/src/Symfony/Component/Finder/Iterator/DateRangeFilterIterator.php b/src/Symfony/Component/Finder/Iterator/DateRangeFilterIterator.php
@@ -44,10 +44,6 @@ public function accept()
     {
         $fileinfo = $this->current();
 
-        if (!$fileinfo->isFile()) {
-            return true;
-        }
-
         $filedate = $fileinfo->getMTime();
         foreach ($this->comparators as $compare) {
             if (!$compare->test($filedate)) {
diff --git a/src/Symfony/Component/Finder/Tests/Iterator/DateRangeFilterIteratorTest.php b/src/Symfony/Component/Finder/Tests/Iterator/DateRangeFilterIteratorTest.php
@@ -57,12 +57,8 @@ public function getAcceptData()
         );
 
         $untilLastMonth = array(
-            '.git',
-            'foo',
             'foo/bar.tmp',
             'test.php',
-            'toto',
-            '.foo',
         );
 
         return array(
