Merge pull request #80 from enumag/events

Fixed onError event dispatching

Author: dg

src/Forms/Form.php

@@ -409,19 +409,20 @@ public function fireEvents()
 			}
 		}
 
-		if ($this->onSuccess) {
+		if (!$this->isValid()) {
+			$this->onError($this);
+		} elseif ($this->onSuccess) {
 			foreach ($this->onSuccess as $handler) {
+				$params = Nette\Utils\Callback::toReflection($handler)->getParameters();
+				$values = isset($params[1]) ? $this->getValues($params[1]->isArray()) : NULL;
+				Nette\Utils\Callback::invoke($handler, $this, $values);
 				if (!$this->isValid()) {
 					$this->onError($this);
 					break;
 				}
-				$params = Nette\Utils\Callback::toReflection($handler)->getParameters();
-				$values = isset($params[1]) ? $this->getValues($params[1]->isArray()) : NULL;
-				Nette\Utils\Callback::invoke($handler, $this, $values);
 			}
-		} elseif (!$this->isValid()) {
-			$this->onError($this);
 		}
+
 		$this->onSubmit($this);
 	}
 

tests/Forms/Forms.onSuccess.phpt

@@ -14,7 +14,6 @@ require __DIR__ . '/../bootstrap.php';
 $_SERVER['REQUEST_METHOD'] = 'POST';
 
 $called = array();
-
 $form = new Form;
 $form->addText('name');
 $form->addSubmit('submit');
@@ -34,8 +33,23 @@ $form->onSuccess[] = function() use (& $called) {
 $form->onError[] = function() use (& $called) {
 	$called[] = 'err';
 };
+$form->fireEvents();
+Assert::same(array(1, 2, 'err'), $called);
 
 
+$called = array();
+$form = new Form;
+$form->addText('name');
+$form->addSubmit('submit');
+$form->onSuccess[] = function() use (& $called) {
+	$called[] = 1;
+};
+$form->onSuccess[] = function($form) use (& $called) {
+	$called[] = 2;
+	$form['name']->addError('error');
+};
+$form->onError[] = function() use (& $called) {
+	$called[] = 'err';
+};
 $form->fireEvents();
-
 Assert::same(array(1, 2, 'err'), $called);