Revert "LDAP: better check for emptiness, fixes #3815"

This reverts commit 4797f9d7db8b48c977e34303a9c80eaea704af19.

diff --git a/apps/user_ldap/lib/connection.php b/apps/user_ldap/lib/connection.php
index cf39329f150f93c7c40b755be3d582a76559f2d9..46bbbb17a4601b61910c09c82394928115430082 100644 (file)
--- a/apps/user_ldap/lib/connection.php
+++ b/apps/user_ldap/lib/connection.php
@@ -421,21 +421,11 @@ class Connection {
        private function validateConfiguration() {
                // first step: "soft" checks: settings that are not really
                // necessary, but advisable. If left empty, give an info message
-               if(empty($this->config['ldapBaseUsers']) ||
-                       // this part handles return of preg_split in readConfiguration
-                       (count($this->config['ldapBaseUsers'][0]) === 1
-                               && isset($this->config['ldapBaseUsers'][0])
-                               && empty($this->config['ldapBaseUsers'][0]))
-               ) {
+               if(empty($this->config['ldapBaseUsers'])) {
                        \OCP\Util::writeLog('user_ldap', 'Base tree for Users is empty, using Base DN', \OCP\Util::INFO);
                        $this->config['ldapBaseUsers'] = $this->config['ldapBase'];
                }
-               if(empty($this->config['ldapBaseGroups']) ||
-                       // this part handles return of preg_split in readConfiguration
-                       (count($this->config['ldapBaseGroups'][0]) === 1
-                               && isset($this->config['ldapBaseGroups'][0])
-                               && empty($this->config['ldapBaseGroups'][0]))
-               ) {
+               if(empty($this->config['ldapBaseGroups'])) {
                        \OCP\Util::writeLog('user_ldap', 'Base tree for Groups is empty, using Base DN', \OCP\Util::INFO);
                        $this->config['ldapBaseGroups'] = $this->config['ldapBase'];
                }