Merge pull request #194 from creative-commoners/pulls/3/protect-hooks

API Set extension hook implementation visibility to protected
silverstripe · May 21, 2024 · 3e2f376 · 3e2f376
2 parents 45381a4 + e049ace
commit 3e2f376
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Extensions/MemberExtension.php b/src/Extensions/MemberExtension.php
@@ -36,7 +36,7 @@ class MemberExtension extends DataExtension
     /**
      * @param FieldList $fields
      */
-    public function updateCMSFields(FieldList $fields)
+    protected function updateCMSFields(FieldList $fields)
     {
         $fields->removeByName('LoginSessions');
 

diff --git a/src/Extensions/RememberLoginHashExtension.php b/src/Extensions/RememberLoginHashExtension.php
@@ -26,7 +26,7 @@ class RememberLoginHashExtension extends Extension
     /**
      * @return void
      */
-    public function onAfterGenerateToken(): void
+    protected function onAfterGenerateToken(): void
     {
         $loginHandler = Injector::inst()->get(LogInAuthenticationHandler::class);
         $loginHandler->setRememberLoginHash($this->owner);
@@ -35,7 +35,7 @@ public function onAfterGenerateToken(): void
     /**
      * @return void
      */
-    public function onAfterRenewToken(): void
+    protected function onAfterRenewToken(): void
     {
         $loginHandler = Injector::inst()->get(LogInAuthenticationHandler::class);
         $request = Injector::inst()->get(HTTPRequest::class);