API Standardise extension hooks

creative-commoners · Aug 19, 2024 · cb405cc · cb405cc
1 parent 1bf5db0
commit cb405cc
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 14 deletions.
diff --git a/code/Controller/AssetAdminFieldsExtension.php b/code/Controller/AssetAdminFieldsExtension.php
@@ -11,7 +11,7 @@
  */
 class AssetAdminFieldsExtension extends Extension
 {
-    public function init()
+    protected function onInit()
     {
         Requirements::add_i18n_javascript('silverstripe/asset-admin:client/lang', false);
         Requirements::javascript('silverstripe/asset-admin:client/dist/js/bundle.js');

diff --git a/tests/php/Controller/AssetAdminTest/FileExtension.php b/tests/php/Controller/AssetAdminTest/FileExtension.php
@@ -7,28 +7,28 @@
 
 class FileExtension extends DataExtension implements TestOnly
 {
-    public function canView($member = null)
+    protected function canView($member = null)
     {
         if ($this->owner->Name === 'disallowCanView.txt') {
             return false;
         }
     }
 
-    public function canEdit($member = null)
+    protected function canEdit($member = null)
     {
         if ($this->owner->Name === 'disallowCanEdit.txt') {
             return false;
         }
     }
 
-    public function canDelete($member = null)
+    protected function canDelete($member = null)
     {
         if ($this->owner->Name === 'disallowCanDelete.txt') {
             return false;
         }
     }
 
-    public function canArchive($member = null)
+    protected function canArchive($member = null)
     {
         if ($this->owner->Name === 'disallowCanDelete.txt') {
             return false;
@@ -37,7 +37,7 @@ public function canArchive($member = null)
     }
 
 
-    public function canCreate($member = null, $context = [])
+    protected function canCreate($member = null, $context = [])
     {
         if (isset($context['Parent']) && $context['Parent']->Name === 'disallowCanAddChildren') {
             return false;

diff --git a/tests/php/Controller/AssetAdminTest/FolderExtension.php b/tests/php/Controller/AssetAdminTest/FolderExtension.php
@@ -7,28 +7,28 @@
 
 class FolderExtension extends DataExtension implements TestOnly
 {
-    public function canView($member = null, $context = array())
+    protected function canView($member = null, $context = array())
     {
         if ($this->owner->Name === 'disallowCanView') {
             return false;
         }
     }
 
-    public function canEdit($member = null, $context = array())
+    protected function canEdit($member = null, $context = array())
     {
         if ($this->owner->Name === 'disallowCanEdit') {
             return false;
         }
     }
 
-    public function canDelete($member = null, $context = array())
+    protected function canDelete($member = null, $context = array())
     {
         if ($this->owner->Name === 'disallowCanDelete') {
             return false;
         }
     }
 
-    public function canCreate($member = null, $context = array())
+    protected function canCreate($member = null, $context = array())
     {
         if (isset($context['Name']) && $context['Name'] === 'disallowCanCreate') {
             return false;

diff --git a/tests/php/Forms/FileFormBuilderTest/FileExtension.php b/tests/php/Forms/FileFormBuilderTest/FileExtension.php
@@ -17,22 +17,22 @@ class FileExtension extends DataExtension implements TestOnly
     public static $canUnpublish = true;
     public static $canEdit = true;
 
-    public function canDelete($member)
+    protected function canDelete($member)
     {
         return FileExtension::$canDelete;
     }
 
-    public function canPublish($member = null)
+    protected function canPublish($member = null)
     {
         return FileExtension::$canPublish;
     }
 
-    public function canUnpublish($member = null)
+    protected function canUnpublish($member = null)
     {
         return FileExtension::$canUnpublish;
     }
 
-    public function canEdit($member = null)
+    protected function canEdit($member = null)
     {
         return FileExtension::$canEdit;
     }