diff --git a/client/src/bundles/PagesDueForReview.js b/client/src/bundles/PagesDueForReview.js index f8cec4e..691d350 100644 --- a/client/src/bundles/PagesDueForReview.js +++ b/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) { diff --git a/src/Extensions/ContentReviewCMSExtension.php b/src/Extensions/ContentReviewCMSExtension.php index 3768819..d37a4ef 100644 --- a/src/Extensions/ContentReviewCMSExtension.php +++ b/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 diff --git a/tests/php/ContentReviewBaseTest.php b/tests/php/ContentReviewBaseTest.php index 717c0d5..bd5ecaf 100644 --- a/tests/php/ContentReviewBaseTest.php +++ b/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(); } }