MNT Remove TODO comments

client/src/bundles/PagesDueForReview.js

@@ -1,8 +1,5 @@
 import jQuery from 'jquery';
 
-/**
- * @todo Re-validate this with Subsites
- */
 jQuery.entwine('ss', ($) => {
   // Hide all owner dropdowns except the one for the current subsite
   function showCorrectSubsiteIDDropdown(value) {

src/Extensions/ContentReviewCMSExtension.php

@@ -129,8 +129,6 @@ protected function findRecord($data)
      * Check if the current request has a X-Formschema-Request header set.
      * Used by conditional logic that responds to validation results
      *
-     * @todo Remove duplication. See https://github.com/silverstripe/silverstripe-admin/issues/240
-     *
      * @return bool
      */
     protected function getSchemaRequested()
@@ -142,8 +140,6 @@ protected function getSchemaRequested()
     /**
      * Generate schema for the given form based on the X-Formschema-Request header value
      *
-     * @todo Remove duplication. See https://github.com/silverstripe/silverstripe-admin/issues/240
-     *
      * @param string $schemaID ID for this schema. Required.
      * @param Form $form Required for 'state' or 'schema' response
      * @param ValidationResult $errors Required for 'error' response

tests/php/ContentReviewBaseTest.php

@@ -4,8 +4,6 @@
 
 use SilverStripe\Dev\FunctionalTest;
 use SilverStripe\CMS\Model\SiteTree;
-// @todo add translatable namespace
-use Translatable;
 
 /**
  * Extend this class when writing unit tests which are compatible with other modules.
@@ -21,32 +19,10 @@ abstract class ContentReviewBaseTest extends FunctionalTest
     protected function setUp(): void
     {
         parent::setUp();
-
-        /*
-         *  We set the locale for pages explicitly, because if we don't, then we get into a situation
-         *  where the page takes on the tester's (your) locale, and any calls to simulate subsequent requests
-         *  (e.g. $this->post()) do not seem to get passed the tester's locale,
-         *  but instead fallback to the default locale.
-         *
-         *  So we set the pages locale to be the default locale, which will then match any subsequent requests.
-         *
-         *  If creating pages in your unit tests (rather than reading from the fixtures file), you must explicitly call
-         *  self::compat() on the page, for the same reasons as above.
-         */
-        if (class_exists(Translatable::class)) {
-            $this->translatableEnabledBefore = SiteTree::has_extension(Translatable::class);
-            SiteTree::remove_extension(Translatable::class);
-        }
     }
 
     protected function tearDown(): void
     {
-        if (class_exists(Translatable::class)) {
-            if ($this->translatableEnabledBefore) {
-                SiteTree::add_extension(Translatable::class);
-            }
-        }
-
         parent::tearDown();
     }
 }