API Standardise extension hooks (#88)

silverstripe · Aug 27, 2024 · 8581f56 · 8581f56
1 parent 58125b4
commit 8581f56
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/code/AuditHook.php b/code/AuditHook.php
@@ -304,7 +304,7 @@ protected function onAfterRestoreToStage()
     /**
      * Log successful login attempts.
      */
-    public function afterMemberLoggedIn()
+    protected function onAfterMemberLoggedIn()
     {
         $this->getAuditLogger()->info(sprintf(
             '"%s" (ID: %s) successfully logged in',
@@ -328,7 +328,7 @@ protected function memberAutoLoggedIn()
     /**
      * Log failed login attempts.
      */
-    public function authenticationFailed($data)
+    protected function onAuthenticationFailed($data)
     {
         // LDAP authentication uses a "Login" POST field instead of Email.
         $login = isset($data['Login'])
@@ -348,9 +348,9 @@ public function authenticationFailed($data)
     /**
      * Log failed login attempts when the email address doesn't map to an existing member record
      */
-    protected function authenticationFailedUnknownUser($data)
+    protected function onAuthenticationFailedUnknownUser($data)
     {
-        $this->authenticationFailed($data);
+        $this->onAuthenticationFailed($data);
     }
 
     /**
@@ -388,7 +388,7 @@ protected function logPermissionDenied($statusCode, $member)
     /**
      * Log successful logout.
      */
-    public function afterMemberLoggedOut()
+    protected function onAfterMemberLoggedOut()
     {
         $this->getAuditLogger()->info(sprintf(
             '"%s" (ID: %s) successfully logged out',