Merge pull request #70 from creative-commoners/pulls/3.0/remove-todo

MNT Remove TODO comments
silverstripe · Oct 24, 2023 · 97c4521 · 97c4521
2 parents de1a170 + c1e66e8
commit 97c4521
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 2 deletions.
diff --git a/src/Forms/GridFieldExportReportButton.php b/src/Forms/GridFieldExportReportButton.php
@@ -60,7 +60,6 @@ public function generateExportFileData($gridField)
         // The is the only variation from the parent, using getList() instead of getManipulatedList()
         $items = $gridField->getList();
 
-        // @todo should GridFieldComponents change behaviour based on whether others are available in the config?
         foreach ($gridField->getConfig()->getComponents() as $component) {
             if ($component instanceof GridFieldFilterHeader || $component instanceof GridFieldSortableHeader) {
                 $items = $component->getManipulatedData($gridField, $items);

diff --git a/tests/php/UserSecurityReportTest.php b/tests/php/UserSecurityReportTest.php
@@ -42,7 +42,6 @@ class UserSecurityReportTest extends SapphireTest
      * Utility method for all tests to use.
      *
      * @return ArrayList
-     * @todo pre-fill the report with fixture-defined users
      */
     protected function setUp(): void
     {