diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

use proper keys to not override appended files

Browse Source
This commit is contained in:
Christian Flothmann 2021-01-28 12:34:38 +01:00
parent 8533ea223e
commit 036c8d71fd
2 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/Symfony/Component/Finder/Finder.php
View File

@ -669,7 +669,8 @@ class Finder implements \IteratorAggregate, \Countable
} elseif ($iterator instanceof \Traversable || \is_array($iterator)) {
$it = new \ArrayIterator();
foreach ($iterator as $file) {
$it->append($file instanceof \SplFileInfo ? $file : new \SplFileInfo($file));
$file = $file instanceof \SplFileInfo ? $file : new \SplFileInfo($file);
$it[$file->getPathname()] = $file;
}
$this->iterators[] = $it;
} else {

11
src/Symfony/Component/Finder/Tests/FinderTest.php
View File

@ -1119,6 +1119,17 @@ class FinderTest extends Iterator\RealIteratorTestCase
$this->assertIterator(iterator_to_array($finder->getIterator()), $finder1->getIterator());
}
public function testMultipleAppendCallsWithSorting()
{
$finder = $this->buildFinder()
->sortByName()
->append([self::$tmpDir.\DIRECTORY_SEPARATOR.'qux_1000_1.php'])
->append([self::$tmpDir.\DIRECTORY_SEPARATOR.'qux_1002_0.php'])
;
$this->assertOrderedIterator($this->toAbsolute(['qux_1000_1.php', 'qux_1002_0.php']), $finder->getIterator());
}
public function testCountDirectories()
{
$directory = Finder::create()->directories()->in(self::$tmpDir);