API Set extension hook implementation visibility to protected

silverstripe · May 16, 2024 · 32e7ebf · 32e7ebf
1 parent a1b113a
commit 32e7ebf
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/src/Extensions/ContentReviewDefaultSettings.php b/src/Extensions/ContentReviewDefaultSettings.php
@@ -110,7 +110,7 @@ public function OwnerUsers()
     /**
      * @param FieldList $fields
      */
-    public function updateCMSFields(FieldList $fields)
+    protected function updateCMSFields(FieldList $fields)
     {
         $helpText = LiteralField::create(
             'ContentReviewHelp',

diff --git a/src/Extensions/ContentReviewLeftAndMainExtension.php b/src/Extensions/ContentReviewLeftAndMainExtension.php
@@ -15,7 +15,7 @@ class ContentReviewLeftAndMainExtension extends LeftAndMainExtension
      *
      * @param array &$clientConfig
      */
-    public function updateClientConfig(&$clientConfig)
+    protected function updateClientConfig(&$clientConfig)
     {
         $clientConfig['form']['ReviewContentForm'] = [
             'schemaUrl' => $this->owner->Link('schema/ReviewContentForm')

diff --git a/src/Extensions/ContentReviewOwner.php b/src/Extensions/ContentReviewOwner.php
@@ -23,7 +23,7 @@ class ContentReviewOwner extends DataExtension
         "SiteTreeContentReview" => SiteTree::class,
     ];
 
-    public function updateCMSFields(FieldList $fields)
+    protected function updateCMSFields(FieldList $fields)
     {
         // Remove automatically scaffolded GridField in Member CMS fields
         $fields->removeByName('SiteTreeContentReview');

diff --git a/src/Extensions/SiteTreeContentReview.php b/src/Extensions/SiteTreeContentReview.php
@@ -161,7 +161,7 @@ public static function merge_owners(SS_List $groups, SS_List $members)
     /**
      * @param FieldList $actions
      */
-    public function updateCMSActions(FieldList $actions)
+    protected function updateCMSActions(FieldList $actions)
     {
         if (!$this->canBeReviewedBy(Security::getCurrentUser())) {
             return;
@@ -326,7 +326,7 @@ public function OwnerUsers()
     /**
      * @param FieldList $fields
      */
-    public function updateSettingsFields(FieldList $fields)
+    protected function updateSettingsFields(FieldList $fields)
     {
         if ($this->owner->hasMethod('displayContentReview') && !$this->owner->displayContentReview()) {
             return;