From a30f34a026a825aae24d13f1cfac30265274c246 Mon Sep 17 00:00:00 2001 From: Thomas Gelf Date: Wed, 14 Dec 2016 20:59:26 +0100 Subject: [PATCH] DirectorObjectForm: be careful when continuing... ...after an exception --- library/Director/Web/Form/DirectorObjectForm.php | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/library/Director/Web/Form/DirectorObjectForm.php b/library/Director/Web/Form/DirectorObjectForm.php index accae484..f514fb5a 100644 --- a/library/Director/Web/Form/DirectorObjectForm.php +++ b/library/Director/Web/Form/DirectorObjectForm.php @@ -131,13 +131,15 @@ abstract class DirectorObjectForm extends QuickForm if ($el = $this->getElement($key)) { if (array_key_exists($key, $post)) { $this->populate(array($key => $post[$key])); - + $old = null; try { $old = $object->get($key); $object->set($key, $el->getValue()); $object->resolveUnresolvedRelatedProperties(); } catch (Exception $e) { - $object->set($key, $old); + if ($old !== null) { + $object->set($key, $old); + } $this->addException($e, $key); } }