API Standardise extension hooks

silverstripe · Aug 25, 2024 · ccdfc27 · ccdfc27
1 parent e38593f
commit ccdfc27
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/Extensions/ElementalAreasExtension.php b/src/Extensions/ElementalAreasExtension.php
@@ -298,7 +298,7 @@ public function ensureElementalAreasExist($elementalAreaRelations)
      *
      * @return void
      */
-    public function requireDefaultRecords()
+    protected function onRequireDefaultRecords()
     {
         if (!$this->supportsElemental()) {
             return;

diff --git a/src/Extensions/ElementalPageExtension.php b/src/Extensions/ElementalPageExtension.php
@@ -119,7 +119,7 @@ protected function updateAnchorsOnPage(array &$anchors): void
         });
     }
 
-    public function MetaTags(&$tags)
+    protected function updateMetaTags(&$tags)
     {
         if (!Controller::has_curr()) {
             return;

diff --git a/tests/Src/ValidationFailedExtension.php b/tests/Src/ValidationFailedExtension.php
@@ -11,7 +11,7 @@ class ValidationFailedExtension extends Extension implements TestOnly
 {
     public const INVALID_TITLE_MESSAGE = '%s is invalid';
 
-    public function validate(ValidationResult $result)
+    protected function updateValidate(ValidationResult $result)
     {
         $result->addFieldError('Title', sprintf(static::INVALID_TITLE_MESSAGE, ClassInfo::shortName($this->owner)));
     }
-Original file line number
+Diff line change
@@ Expand Up @@
             });
         }
-        public function MetaTags(&$tags)
+        protected function updateMetaTags(&$tags)
         {
             if (!Controller::has_curr()) {
                 return;
@@ Expand Down @@