Merge pull request #7815 from kenjis/fix-validation-rule-instances

fix: instances of Validation rules are incremented each time `run()` is executed

Author: kenjis

system/Validation/Validation.php

@@ -107,6 +107,8 @@ public function __construct($config, RendererInterface $view)
         $this->config = $config;
 
         $this->view = $view;
+
+        $this->loadRuleSets();
     }
 
     /**
@@ -127,7 +129,6 @@ public function run(?array $data = null, ?string $group = null, ?string $dbGroup
         // `DBGroup` is a reserved name. For is_unique and is_not_unique
         $data['DBGroup'] = $dbGroup;
 
-        $this->loadRuleSets();
         $this->loadRuleGroup($group);
 
         // If no rules exist, we return false to ensure

tests/system/Validation/ValidationTest.php

@@ -240,6 +240,22 @@ public function testRunReturnsFalseWithNothingToDo(): void
         $this->assertFalse($this->validation->run([]));
     }
 
+    public function testRuleClassesInstantiatedOnce(): void
+    {
+        $this->validation->setRules([]);
+        $this->validation->run([]);
+        $count1 = count(
+            $this->getPrivateProperty($this->validation, 'ruleSetInstances')
+        );
+
+        $this->validation->run([]);
+        $count2 = count(
+            $this->getPrivateProperty($this->validation, 'ruleSetInstances')
+        );
+
+        $this->assertSame($count1, $count2);
+    }
+
     public function testRunDoesTheBasics(): void
     {
         $data = ['foo' => 'notanumber'];