From ab24194e73b002e798c5ff3d7e16ebaf1d080358 Mon Sep 17 00:00:00 2001
From: Sabina Talipova <sabina.talipova@silverstripe.com>
Date: Wed, 11 Oct 2023 12:02:23 +1300
Subject: [PATCH] MNT Remove TODO

---
 code/Form/UserForm.php                                      | 4 +---
 code/Model/EditableFormField/EditableCheckboxGroupField.php | 1 -
 code/Model/EditableFormField/EditableLiteralField.php       | 1 -
 code/Task/RecoverUploadLocationsHelper.php                  | 2 --
 tests/php/Model/UserDefinedFormTest.php                     | 1 -
 5 files changed, 1 insertion(+), 8 deletions(-)

diff --git a/code/Form/UserForm.php b/code/Form/UserForm.php
index c9ecf5dec..03e7fe952 100644
--- a/code/Form/UserForm.php
+++ b/code/Form/UserForm.php
@@ -134,9 +134,7 @@ public function getFormFields()
      * Generate the form actions for the UserDefinedForm. You
      * can manipulate these by using {@link updateFormActions()} on
      * a decorator.
-     *
-     * @todo Make form actions editable via their own field editor.
-     *
+     * 
      * @return FieldList
      */
     public function getFormActions()
diff --git a/code/Model/EditableFormField/EditableCheckboxGroupField.php b/code/Model/EditableFormField/EditableCheckboxGroupField.php
index 96e7e1321..b041c5ee8 100755
--- a/code/Model/EditableFormField/EditableCheckboxGroupField.php
+++ b/code/Model/EditableFormField/EditableCheckboxGroupField.php
@@ -61,7 +61,6 @@ public function getValueFromData($data)
     public function getSelectorField(EditableCustomRule $rule, $forOnLoad = false)
     {
         // watch out for checkboxs as the inputs don't have values but are 'checked
-        // @todo - Test this
         if ($rule->FieldValue) {
             return "$(\"input[name='{$this->Name}[]'][value='{$rule->FieldValue}']\")";
         } else {
diff --git a/code/Model/EditableFormField/EditableLiteralField.php b/code/Model/EditableFormField/EditableLiteralField.php
index 9d71be9c7..ffd5509ad 100644
--- a/code/Model/EditableFormField/EditableLiteralField.php
+++ b/code/Model/EditableFormField/EditableLiteralField.php
@@ -149,7 +149,6 @@ public function getFormField()
 
         $field = CompositeField::create($content)
             ->setName($this->Name)
-            // ->setID($this->Name) // @todo: https://github.com/silverstripe/silverstripe-framework/issues/7264
             ->setFieldHolderTemplate(__CLASS__ . '_holder');
 
         $this->doUpdateFormField($field);
diff --git a/code/Task/RecoverUploadLocationsHelper.php b/code/Task/RecoverUploadLocationsHelper.php
index 65f23095a..1961c830c 100644
--- a/code/Task/RecoverUploadLocationsHelper.php
+++ b/code/Task/RecoverUploadLocationsHelper.php
@@ -447,8 +447,6 @@ private function chunk($chunkSize = 100)
                     '"SubmittedFileFieldTable"."UploadedFileID" > ?' => $greaterThanID
                 ])->execute();
 
-            // TODO: Versioned::prepopulate_versionnumber_cache
-
             foreach ($chunk as $item) {
                 yield $item;
                 $greaterThanID = $item['UploadedFileID'];
diff --git a/tests/php/Model/UserDefinedFormTest.php b/tests/php/Model/UserDefinedFormTest.php
index 203a1a540..c434201ff 100644
--- a/tests/php/Model/UserDefinedFormTest.php
+++ b/tests/php/Model/UserDefinedFormTest.php
@@ -49,7 +49,6 @@ public function testRollbackToVersion()
             'UserDefinedForm::rollback() has not been implemented completely'
         );
 
-        // @todo
         $this->logInWithPermission('ADMIN');
         /** @var UserDefinedForm|Versioned $form  */
         $form = $this->objFromFixture(UserDefinedForm::class, 'basic-form-page');