Fix that no form values are stored for the LdapDiscoveryPage

refs #7163
2014-10-17 16:00:52 +02:00 · 2014-10-17 16:00:52 +02:00 · 7f453be1cc
parent e72f2cac24
commit 7f453be1cc
1 changed files with 3 additions and 4 deletions
--- a/application/forms/Setup/LdapDiscoveryPage.php
+++ b/application/forms/Setup/LdapDiscoveryPage.php
@ -69,15 +69,14 @@ class LdapDiscoveryPage extends Form
     */
    public function isValid($data)
    {
-        if ($data['skip_validation'] === '1') {
-            return true;
-        }
        if (false === parent::isValid($data)) {
            return false;
        }
-        if (false === $this->discoveryForm->isValid($data)) {
+
+        if (! $data['skip_validation'] && false === $this->discoveryForm->isValid($data)) {
            return false;
        }
+
        return true;
    }