[Finder] Added a way to inverse a previous sorting

lyrixx · lyrixx · commit 27bc298c175c · 2018-07-19T10:21:04.000+02:00
Sometimes, it's useful to inverse the previous sorting.
For exemple when you want to display the most recent uploaded files
diff --git a/src/Symfony/Component/Finder/CHANGELOG.md b/src/Symfony/Component/Finder/CHANGELOG.md
@@ -5,6 +5,7 @@ CHANGELOG
 -----
 
  * added $useNaturalSort option to Finder::sortByName() method
+ * added `Finder::reverseSorting` to reverse the sorting
 
 4.0.0
 -----
diff --git a/src/Symfony/Component/Finder/Finder.php b/src/Symfony/Component/Finder/Finder.php
@@ -48,6 +48,7 @@ class Finder implements \IteratorAggregate, \Countable
     private $depths = array();
     private $sizes = array();
     private $followLinks = false;
+    private $reverseSorting = false;
     private $sort = false;
     private $ignore = 0;
     private $dirs = array();
@@ -460,6 +461,18 @@ public function sortByAccessedTime()
         return $this;
     }
 
+    /**
+     * Reverses the sorting.
+     *
+     * @return $this
+     */
+    public function reverseSorting()
+    {
+        $this->reverseSorting = true;
+
+        return $this;
+    }
+
     /**
      * Sorts files and directories by the last inode changed time.
      *
@@ -739,6 +752,10 @@ private function searchInDirectory(string $dir): \Iterator
             $iterator = $iteratorAggregate->getIterator();
         }
 
+        if ($this->reverseSorting) {
+            $iterator = new Iterator\ReverseSortingIterator($iterator);
+        }
+
         return $iterator;
     }
 
diff --git a/src/Symfony/Component/Finder/Iterator/ReverseSortingIterator.php b/src/Symfony/Component/Finder/Iterator/ReverseSortingIterator.php
@@ -0,0 +1,25 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\Finder\Iterator;
+
+/**
+ * Reverse the order of a previous iterator.
+ *
+ * @author Grégoire Pineau <lyrixx@lyrixx.info>
+ */
+class ReverseSortingIterator extends \ArrayIterator
+{
+    public function __construct(\Traversable $iterator)
+    {
+        parent::__construct(array_reverse(iterator_to_array($iterator, true)));
+    }
+}
diff --git a/src/Symfony/Component/Finder/Tests/FinderTest.php b/src/Symfony/Component/Finder/Tests/FinderTest.php
@@ -611,6 +611,30 @@ public function testSortByModifiedTime()
         )), $finder->in(self::$tmpDir)->getIterator());
     }
 
+    public function testReverseSorting()
+    {
+        $finder = $this->buildFinder();
+        $this->assertSame($finder, $finder->sortByName());
+        $this->assertSame($finder, $finder->reverseSorting());
+        $this->assertOrderedIteratorInForeach($this->toAbsolute(array(
+            'toto',
+            'test.py',
+            'test.php',
+            'qux_2_0.php',
+            'qux_12_0.php',
+            'qux_10_2.php',
+            'qux_1002_0.php',
+            'qux_1000_1.php',
+            'qux_0_1.php',
+            'qux/baz_1_2.py',
+            'qux/baz_100_1.py',
+            'qux',
+            'foo/bar.tmp',
+            'foo bar',
+            'foo',
+        )), $finder->in(self::$tmpDir)->getIterator());
+    }
+
     public function testSortByNameNatural()
     {
         $finder = $this->buildFinder();
diff --git a/src/Symfony/Component/Finder/Tests/Iterator/ReverseSortingIteratorTest.php b/src/Symfony/Component/Finder/Tests/Iterator/ReverseSortingIteratorTest.php
@@ -0,0 +1,32 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\Finder\Tests\Iterator;
+
+use Symfony\Component\Finder\Iterator\ReverseSortingIterator;
+
+class ReverseSortingIteratorTest extends IteratorTestCase
+{
+    public function test()
+    {
+        $iterator = new ReverseSortingIterator(new MockFileListIterator(array(
+            'a.txt',
+            'b.yaml',
+            'c.php',
+        )));
+
+        $result = iterator_to_array($iterator);
+        $this->assertCount(3, $iterator);
+        $this->assertSame('c.php', $result[0]->getFilename());
+        $this->assertSame('b.yaml', $result[1]->getFilename());
+        $this->assertSame('a.txt', $result[2]->getFilename());
+    }
+}
