MNT Remove TODO

silverstripe · Oct 10, 2023 · 6752ee1 · 6752ee1
1 parent 071fba1
commit 6752ee1
Show file tree

Hide file tree

Showing 6 changed files with 2 additions and 9 deletions.
diff --git 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
@@ -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
@@ -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/Model/Recipient/EmailRecipient.php b/code/Model/Recipient/EmailRecipient.php
@@ -177,7 +177,7 @@ protected function getFormParent()
         }
 
         // Revert to checking for a form from the session
-        // LeftAndMain::sessionNamespace is protected. @todo replace this with a non-deprecated equivalent.
+        // LeftAndMain::sessionNamespace is protected.
         $sessionNamespace = $this->config()->get('session_namespace') ?: CMSMain::class;
 
         $formID = Controller::curr()->getRequest()->getSession()->get($sessionNamespace . '.currentPage');

diff --git 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
@@ -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');