locomotivemtl / charcoal-user

src/Charcoal/User/AuthAwareTrait.php

Indentation +5 added lines, -5 removed lines

@@ -121,11 +121,11 @@
         return $this->authorizer;
     }
 
-     /**
-      * @param string[]|string|null $permissions The list of required permissions.
-      * @throws InvalidArgumentException If the permissions are not an array or a comma-separated string.
-      * @return self
-      */
+        /**
+         * @param string[]|string|null $permissions The list of required permissions.
+         * @throws InvalidArgumentException If the permissions are not an array or a comma-separated string.
+         * @return self
+         */
     protected function setRequiredAclPermissions($permissions)
     {
         if ($permissions === null || !$permissions) {

src/Charcoal/User/ServiceProvider/AuthServiceProvider.php

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
              * @param  Container $container The Pimple DI Container.
              * @return Authenticator
              */
-            $container['authenticator'] = function (Container $container) {
+            $container['authenticator'] = function(Container $container) {
                 return new Authenticator([
                     'logger'        => $container['logger'],
                     'user_type'     => User::class,
@@ -47,7 +47,7 @@ 
              * @param  Container $container The Pimple DI container.
              * @return Authorizer
              */
-            $container['authorizer'] = function (Container $container) {
+            $container['authorizer'] = function(Container $container) {
                 return new Authorizer([
                     'logger'    => $container['logger'],
                     'acl'       => $container['authorizer/acl'],
@@ -60,7 +60,7 @@ 
             /**
              * @return Acl
              */
-            $container['authorizer/acl'] = function () {
+            $container['authorizer/acl'] = function() {
                 return new Acl();
             };
         }

src/Charcoal/User/AbstractUser.php

Doc Comments +1 added lines, -1 removed lines

@@ -519,7 +519,7 @@
     /**
      * Validate the username or email address is unique.
      *
-     * @return boolean
+     * @return false|null
      */
     protected function validateLoginUnique()
     {

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use InvalidArgumentException;
 
 // From 'charcoal-factory'
-use Charcoal\Factory\FactoryInterface;
 
 // From 'charcoal-core'
 use Charcoal\Validator\ValidatorInterface;

src/Charcoal/User/Authenticator.php

Doc Comments +1 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
      *
      * @param  AuthenticatableInterface $user     The authenticated user to log in.
      * @param  boolean                  $remember Whether to "remember" the user or not.
-     * @return boolean Success / Failure
+     * @return boolean|null Success / Failure
      */
     public function login(AuthenticatableInterface $user, $remember = false)
     {
@@ -57,7 +57,6 @@ 
      * Updates the user's timestamp for their last log in.
      *
      * @param  AuthenticatableInterface $user     The user to update.
-     * @param  string                   $password The plain-text password to hash.
      * @param  boolean                  $update   Whether to persist changes to storage.
      * @return boolean Returns TRUE if the password was changed, or FALSE otherwise.
      */

src/Charcoal/User/AbstractAuthenticator.php

Indentation +1 added lines, -1 removed lines

@@ -471,7 +471,7 @@
 
         $this->logger->warning(sprintf(
             'Invalid login attempt for user "%s": invalid password.',
-             $identifier
+                $identifier
         ));
 
         return null;