Callback Functionality

Author: IvanGrigorov

README.md

@@ -116,6 +116,7 @@ In what order to validate the classes (methods or properties first),  and what t
  - [x] Custom property and method names for the exceptions
  - [x] Severity levels
  - [x] Debug explorer
+ - [x] Callback execution*
 
 
 ### *The custom validation should be declared as static in a validation class
@@ -149,6 +150,13 @@ You can pass additional arguments to use in the validation function, but the fir
 
 ```
 
+### *Callback execution
+
+Call backs are executed when the validation does not trigger and Exception (based on SEVERITY_LEVEL)
+
+```$forceCallback``` can be used and has an effect only with EXCEPTION_SEVERITY_LEVEL to force callback after an EXCEPTION trigger
+
+
 ### Severity
 Add option to make the failed validation throw NOTICE|WARNING|ERROR
 ```

RMValidator/Validators/MasterValidator.php

@@ -18,25 +18,40 @@
 
 final class MasterValidator {
 
-    public static function validate(object $target, OptionsModel $passedOptionsModel = null) {
+    public static function validate(object $target, OptionsModel $passedOptionsModel = null, callable $callback = null, $forceCallback = false) {
         $optionsModel = (empty($passedOptionsModel)) ? new OptionsModel() : $passedOptionsModel;
+        if (count($optionsModel->getOrAttributes()) != count(array_unique($optionsModel->getOrAttributes()))) {
+            throw new Exception("There is a duplicate OR attribute, the array of the OR attributes must be unique");
+        }
         $className = get_class($target);
         $reflectionClass = new ReflectionClass($className);
         $methods = $reflectionClass->getMethods();
         $properties = $reflectionClass->getProperties();
         $constants = $reflectionClass->getConstants();
-        foreach($optionsModel->getOrderOfValidation() as $order) {
-            switch($order) {
-                case ValidationOrderEnum::METHODS:
-                    MasterValidator::validateMethods($methods, $className, $target, $optionsModel->getExcludedMethods(), $passedOptionsModel->getOrAttributes(), $passedOptionsModel->getGlobalSeverityLevel());
-                    break;
-                case ValidationOrderEnum::PROPERTIES:
-                    MasterValidator::validateProperties($properties, $className, $target, $optionsModel->getExcludedProperties(), $passedOptionsModel->getOrAttributes(), $passedOptionsModel->getGlobalSeverityLevel());
-                    break;
-                case ValidationOrderEnum::CONSTANTS:
-                    MasterValidator::validateConstants($constants, $className, $target, $optionsModel->getExcludedProperties(), $passedOptionsModel->getOrAttributes(), $passedOptionsModel->getGlobalSeverityLevel());
-                    break;
-
+        try {
+            foreach($optionsModel->getOrderOfValidation() as $order) {
+                switch($order) {
+                    case ValidationOrderEnum::METHODS:
+                        MasterValidator::validateMethods($methods, $className, $target, $optionsModel->getExcludedMethods(), $optionsModel->getOrAttributes(), $optionsModel->getGlobalSeverityLevel());
+                        break;
+                    case ValidationOrderEnum::PROPERTIES:
+                        MasterValidator::validateProperties($properties, $className, $target, $optionsModel->getExcludedProperties(), $optionsModel->getOrAttributes(), $optionsModel->getGlobalSeverityLevel());
+                        break;
+                    case ValidationOrderEnum::CONSTANTS:
+                        MasterValidator::validateConstants($constants, $className, $target, $optionsModel->getExcludedProperties(), $optionsModel->getOrAttributes(), $optionsModel->getGlobalSeverityLevel());
+                        break;
+                }
+            }
+            if (!$forceCallback && !is_null($callback)) {
+                $callback();
+            }
+        }
+        catch(Exception $e) {
+            throw $e;
+        }
+        finally {
+            if ($forceCallback && !is_null($callback)) {
+                $callback();
             }
         }
 

tests/unit/Tests/Callbacks/ValidationCallbacksTest.php

@@ -0,0 +1,65 @@
+<?php
+
+
+use PHPUnit\Framework\TestCase;
+use RMValidator\Attributes\PropertyAttributes\Numbers\RangeAttribute;
+use RMValidator\Validators\MasterValidator;
+
+class ValidationCallbacksTest extends TestCase {
+
+     /**
+    * @doesNotPerformAssertions
+    */
+    public function testMasterValidator_shouldExecuteCallbackOnSuccessfullValidation()
+    {
+        $controll = 0;
+        MasterValidator::validate(new SuccessfullValidityTest(), null, function() use (&$controll) {
+            $controll++;
+        });
+        $this->assertEquals(1, $controll);
+    }
+
+    public function testMasterValidator_shouldNotExecuteCallbackOnUnuccessfullValidation()
+    {
+        $controll = 0;
+        try {
+            MasterValidator::validate(new UnsuccessfullValidityTest(), null, function() use (&$controll) {
+                $controll++;
+            });
+        }
+        catch(Exception $e) {
+            $this->assertEquals(0, $controll);
+        }
+        
+    }
+
+    public function testMasterValidator_shouldExecuteCallbackOnUnuccessfullValidationButForce()
+    {
+        $controll = 0;
+        try {
+            MasterValidator::validate(new UnsuccessfullValidityTest(), null, function() use (&$controll) {
+                $controll++;
+            }, true);
+        }
+        catch(Exception $e) {
+            $this->assertEquals(1, $controll);
+        }
+        
+    }
+}
+
+class SuccessfullValidityTest {
+
+    #[RangeAttribute(from:10, to:50)]
+    public static function successfulMethod() {
+        return 40;
+    }
+}
+
+class UnsuccessfullValidityTest {
+
+    #[RangeAttribute(from:10, to:50)]
+    public static function unSuccessfulMethod() {
+        return 60;
+    }
+}