simplesamlphp / simplesamlphp-module-ldap

lib/Auth/Ldap.php

Braces +6 added lines, -2 removed lines

@@ -886,7 +886,9 @@ 
 
       // Attempt to set attributes
       $result = @ldap_mod_replace($this->ldap, $dn, $attributes);
-      if ($result === false) throw $this->makeException('Library - LDAP setAttributes(): Failed to set attributes for DN \''.$dn.'\'. Bind necessary?');
+      if ($result === false) {
+          throw $this->makeException('Library - LDAP setAttributes(): Failed to set attributes for DN \''.$dn.'\'. Bind necessary?');
+      }
 
       return $result;
     }
@@ -911,7 +913,9 @@ 
 
        // Attempt to add attributes
        $result = @ldap_mod_add($this->ldap, $dn, $attributes);
-       if ($result === false) throw $this->makeException('Library - LDAP addAttributes(): Failed to set attributes for DN \''.$dn.'\'. Bind necessary?');
+       if ($result === false) {
+           throw $this->makeException('Library - LDAP addAttributes(): Failed to set attributes for DN \''.$dn.'\'. Bind necessary?');
+       }
 
        return $result;
      }    

Indentation +30 added lines, -30 removed lines

@@ -877,42 +877,42 @@
      * Result of operation
      */
     public function setAttributes($dn, $attributes) {
-      if (!is_array($attributes)) {
+        if (!is_array($attributes)) {
         Logger::warning('Library - LDAP setAttributes(): No array of attributes given for DN \''.$dn.'\'');
         return false;
-      } else {
+        } else {
         Logger::debug('Library - LDAP setAttributes(): Received arraydata:'.print_r($attributes,true));
-      }
+        }
 
-      // Attempt to set attributes
-      $result = @ldap_mod_replace($this->ldap, $dn, $attributes);
-      if ($result === false) throw $this->makeException('Library - LDAP setAttributes(): Failed to set attributes for DN \''.$dn.'\'. Bind necessary?');
+        // Attempt to set attributes
+        $result = @ldap_mod_replace($this->ldap, $dn, $attributes);
+        if ($result === false) throw $this->makeException('Library - LDAP setAttributes(): Failed to set attributes for DN \''.$dn.'\'. Bind necessary?');
 
-      return $result;
+        return $result;
     }
 
     /**
-      * Adds for a given DN attributes
-      *
-      * @param string $dn
-      * The DN of an element.
-      * @param array $attributes
-      * The names and value of the attribute(s) to set using ldap_mod_add structure;
-      * @return bool
-      * Result of operation
-      */
-     public function addAttributes($dn, $attributes) {
-       if (!is_array($attributes)) {
-         Logger::warning('Library - LDAP addAttributes(): No array of attributes given for DN \''.$dn.'\'');
-         return false;
-       } else {
-         Logger::debug('Library - LDAP addAttributes(): Received arraydata:'.print_r($attributes,true));
-       }
-
-       // Attempt to add attributes
-       $result = @ldap_mod_add($this->ldap, $dn, $attributes);
-       if ($result === false) throw $this->makeException('Library - LDAP addAttributes(): Failed to set attributes for DN \''.$dn.'\'. Bind necessary?');
-
-       return $result;
-     }    
+     * Adds for a given DN attributes
+     *
+     * @param string $dn
+     * The DN of an element.
+     * @param array $attributes
+     * The names and value of the attribute(s) to set using ldap_mod_add structure;
+     * @return bool
+     * Result of operation
+     */
+        public function addAttributes($dn, $attributes) {
+        if (!is_array($attributes)) {
+            Logger::warning('Library - LDAP addAttributes(): No array of attributes given for DN \''.$dn.'\'');
+            return false;
+        } else {
+            Logger::debug('Library - LDAP addAttributes(): Received arraydata:'.print_r($attributes,true));
+        }
+
+        // Attempt to add attributes
+        $result = @ldap_mod_add($this->ldap, $dn, $attributes);
+        if ($result === false) throw $this->makeException('Library - LDAP addAttributes(): Failed to set attributes for DN \''.$dn.'\'. Bind necessary?');
+
+        return $result;
+        }    
 }

Spacing +2 added lines, -2 removed lines

@@ -881,7 +881,7 @@ 
         Logger::warning('Library - LDAP setAttributes(): No array of attributes given for DN \''.$dn.'\'');
         return false;
       } else {
-        Logger::debug('Library - LDAP setAttributes(): Received arraydata:'.print_r($attributes,true));
+        Logger::debug('Library - LDAP setAttributes(): Received arraydata:'.print_r($attributes, true));
       }
 
       // Attempt to set attributes
@@ -906,7 +906,7 @@ 
          Logger::warning('Library - LDAP addAttributes(): No array of attributes given for DN \''.$dn.'\'');
          return false;
        } else {
-         Logger::debug('Library - LDAP addAttributes(): Received arraydata:'.print_r($attributes,true));
+         Logger::debug('Library - LDAP addAttributes(): Received arraydata:'.print_r($attributes, true));
        }
 
        // Attempt to add attributes