API Standardise extension hooks

silverstripe · Aug 19, 2024 · b001050 · b001050
1 parent 523cc2c
commit b001050
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/Extensions/LDAPMemberExtension.php b/src/Extensions/LDAPMemberExtension.php
@@ -185,7 +185,7 @@ protected function updateCMSFields(FieldList $fields)
      * @param  ValidationResult
      * @throws ValidationException
      */
-    public function validate(ValidationResult $validationResult)
+    protected function updateValidate(ValidationResult $validationResult)
     {
         // We allow empty Username for registration purposes, as we need to
         // create Member records with empty Username temporarily. Forms should explicitly
@@ -295,7 +295,7 @@ public function sync()
      *
      * @throws Exception When failures are not acceptable via configuration
      */
-    public function afterMemberLoggedIn()
+    protected function onAfterMemberLoggedIn()
     {
         if ($this->owner->GUID) {
             try {