RiperFr / Security-ActiveDirectoryBundle

Security/User/AdUser.php

Doc Comments +4 added lines

@@ -21,6 +21,10 @@
     /** @var string */
     private $email;
 
+    /**
+     * @param string $username
+     * @param string $password
+     */
     public function __construct($username, $password, array $roles)
     {
         $this->username = $username;

Security/User/AdUserProvider.php

Spacing +3 added lines, -3 removed lines

@@ -78,7 +78,7 @@ 
             $userString = $this->getUsernameFromString($username);
             $user = new AdUser(
                 $this->getUsernameFromString($userString),
-                uniqid(true) . rand(0, 424242),
+                uniqid(true).rand(0, 424242),
                 array()
             );
         } catch (\InvalidArgumentException $e) {
@@ -112,7 +112,7 @@ 
             }
         }
         $username = strtolower($username);
-        $pattern   = $this->getConfig('username_validation_pattern', '/^[a-z0-9-.]+$/i');
+        $pattern = $this->getConfig('username_validation_pattern', '/^[a-z0-9-.]+$/i');
         if (preg_match($pattern, $username)) {
             return $username;
         }
@@ -188,7 +188,7 @@ 
             $sfRolesTemp = array();
             foreach ($groups as $r) {
                 if (in_array($r, $sfRolesTemp) === false) {
-                    $sfRoles[] = 'ROLE_' . strtoupper(str_replace(' ', '_', $r));
+                    $sfRoles[] = 'ROLE_'.strtoupper(str_replace(' ', '_', $r));
                     $sfRolesTemp[] = $r;
                 }
             }

Service/AdldapService.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      */
     public function __construct(array $parameters)
     {
-        $parameters['account_suffix'] = '@' . $parameters['account_suffix'];
+        $parameters['account_suffix'] = '@'.$parameters['account_suffix'];
         $this->parameters = $parameters;
     }
 

Security/Factory/AdAuthFactory.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     {
         $config['firewall_id'] = $id;
 
-        $providerId = 'security.authentication.provider.riper.active_directory.' . $id;
+        $providerId = 'security.authentication.provider.riper.active_directory.'.$id;
         $container
             ->setDefinition(
                 $providerId,