cerbero90 / Auth

src/Http/Controllers/AuthController.php

Doc Comments +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 	 * Set the dependencies.
 	 *
 	 * @author	Andrea Marco Sartori
-	 * @param	Illuminate\Contracts\Bus\Dispatcher	$bus
+	 * @param	Dispatcher	$bus
 	 * @return	void
 	 */
 	public function __construct(Dispatcher $bus)

Indentation +181 added lines, -181 removed lines

@@ -14,187 +14,187 @@
 
 class AuthController extends Controller {
 
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		Illuminate\Contracts\Bus\Dispatcher	$bus	Bus dispatcher.
-	 */
-	protected $bus;
-
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		array	$loginPipes	List of pipes for the login process.
-	 */
-	protected $loginPipes = ['DispatchEvent', 'Throttle'];
-
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		array	$logoutPipes	List of pipes for the logout process.
-	 */
-	protected $logoutPipes = ['DispatchEvent'];
-
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		array	$registerPipes	List of pipes for the register process.
-	 */
-	protected $registerPipes = ['DispatchEvent', 'Login', 'Notify', 'Hash'];
-
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		array	$recoverPipes	List of pipes for the password recover process.
-	 */
-	protected $recoverPipes = ['DispatchEvent', 'Notify', 'Store'];
-
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		array	$resetPipes	List of pipes for the password reset process.
-	 */
-	protected $resetPipes = ['DispatchEvent'];
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		Illuminate\Contracts\Bus\Dispatcher	$bus	Bus dispatcher.
+     */
+    protected $bus;
+
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		array	$loginPipes	List of pipes for the login process.
+     */
+    protected $loginPipes = ['DispatchEvent', 'Throttle'];
+
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		array	$logoutPipes	List of pipes for the logout process.
+     */
+    protected $logoutPipes = ['DispatchEvent'];
+
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		array	$registerPipes	List of pipes for the register process.
+     */
+    protected $registerPipes = ['DispatchEvent', 'Login', 'Notify', 'Hash'];
+
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		array	$recoverPipes	List of pipes for the password recover process.
+     */
+    protected $recoverPipes = ['DispatchEvent', 'Notify', 'Store'];
+
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		array	$resetPipes	List of pipes for the password reset process.
+     */
+    protected $resetPipes = ['DispatchEvent'];
 	
-	/**
-	 * Set the dependencies.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	Illuminate\Contracts\Bus\Dispatcher	$bus
-	 * @return	void
-	 */
-	public function __construct(Dispatcher $bus)
-	{
-		$this->bus = $bus;
-	}
-
-	/**
-	 * Display the login page.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	Illuminate\Http\Response
-	 */
-	public function showLogin()
-	{
-		$login = config('_auth.login.view');
-
-		return view($login);
-	}
-
-	/**
-	 * Log the user in.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	Illuminate\Http\RedirectResponse
-	 */
-	public function login(LoginRequest $request)
-	{
-		$this->bus->pipeThrough($this->pipesOf('login'))->dispatchFrom(LoginJob::class, $request);
-
-		return redirect()->route(config('_auth.login.redirect'));
-	}
-
-	/**
-	 * Retrieve the pipes of a given process.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	string	$process
-	 * @return	array
-	 */
-	protected function pipesOf($process)
-	{
-		$Process = ucfirst($process);
-
-		return array_map(function($pipe) use($Process)
-		{
-			return "Cerbero\Auth\Pipes\\{$Process}\\{$pipe}";
-
-		}, $this->{"{$process}Pipes"});
-	}
-
-	/**
-	 * Log the user out.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	Illuminate\Http\RedirectResponse
-	 */
-	public function logout()
-	{
-		$this->bus->pipeThrough($this->pipesOf('logout'))->dispatchNow(new LogoutJob);
-
-		return redirect()->route(config('_auth.logout.redirect'));
-	}
-
-	/**
-	 * Display the registration page.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	Illuminate\Http\Response
-	 */
-	public function showRegister()
-	{
-		$register = config('_auth.register.view');
-
-		return view($register);
-	}
-
-	/**
-	 * Register the user.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	Illuminate\Http\RedirectResponse
-	 */
-	public function register(RegisterRequest $request)
-	{
-		$this->bus->pipeThrough($this->pipesOf('register'))->dispatchFrom(RegisterJob::class, $request);
-
-		return redirect()->route(config('_auth.register.redirect'))->withSuccess(trans('auth::register.success'));
-	}
-
-	/**
-	 * Display the recover page.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	Illuminate\Http\Response
-	 */
-	public function showRecover()
-	{
-		$recover = config('_auth.recover.view');
-
-		return view($recover);
-	}
-
-	/**
-	 * Remember the user password.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	Illuminate\Http\RedirectResponse
-	 */
-	public function recover(RecoverRequest $request)
-	{
-		$this->bus->pipeThrough($this->pipesOf('recover'))->dispatchFrom(RecoverJob::class, $request);
-
-		return back()->withSuccess(trans('auth::recover.success'));
-	}
-
-	/**
-	 * Display the reset page.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	Illuminate\Http\Response
-	 */
-	public function showReset($token)
-	{
-		$reset = config('_auth.reset.view');
-
-		return view($reset);
-	}
-
-	/**
-	 * Reset the user password.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	Illuminate\Http\RedirectResponse
-	 */
-	public function reset(ResetRequest $request, $token)
-	{
-		$this->bus->pipeThrough($this->pipesOf('reset'))->dispatchFrom(ResetJob::class, $request, compact('token'));
-
-		return redirect()->route('login.index')->withSuccess(trans('auth::reset.success'));
-	}
+    /**
+     * Set the dependencies.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	Illuminate\Contracts\Bus\Dispatcher	$bus
+     * @return	void
+     */
+    public function __construct(Dispatcher $bus)
+    {
+        $this->bus = $bus;
+    }
+
+    /**
+     * Display the login page.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	Illuminate\Http\Response
+     */
+    public function showLogin()
+    {
+        $login = config('_auth.login.view');
+
+        return view($login);
+    }
+
+    /**
+     * Log the user in.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	Illuminate\Http\RedirectResponse
+     */
+    public function login(LoginRequest $request)
+    {
+        $this->bus->pipeThrough($this->pipesOf('login'))->dispatchFrom(LoginJob::class, $request);
+
+        return redirect()->route(config('_auth.login.redirect'));
+    }
+
+    /**
+     * Retrieve the pipes of a given process.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	string	$process
+     * @return	array
+     */
+    protected function pipesOf($process)
+    {
+        $Process = ucfirst($process);
+
+        return array_map(function($pipe) use($Process)
+        {
+            return "Cerbero\Auth\Pipes\\{$Process}\\{$pipe}";
+
+        }, $this->{"{$process}Pipes"});
+    }
+
+    /**
+     * Log the user out.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	Illuminate\Http\RedirectResponse
+     */
+    public function logout()
+    {
+        $this->bus->pipeThrough($this->pipesOf('logout'))->dispatchNow(new LogoutJob);
+
+        return redirect()->route(config('_auth.logout.redirect'));
+    }
+
+    /**
+     * Display the registration page.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	Illuminate\Http\Response
+     */
+    public function showRegister()
+    {
+        $register = config('_auth.register.view');
+
+        return view($register);
+    }
+
+    /**
+     * Register the user.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	Illuminate\Http\RedirectResponse
+     */
+    public function register(RegisterRequest $request)
+    {
+        $this->bus->pipeThrough($this->pipesOf('register'))->dispatchFrom(RegisterJob::class, $request);
+
+        return redirect()->route(config('_auth.register.redirect'))->withSuccess(trans('auth::register.success'));
+    }
+
+    /**
+     * Display the recover page.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	Illuminate\Http\Response
+     */
+    public function showRecover()
+    {
+        $recover = config('_auth.recover.view');
+
+        return view($recover);
+    }
+
+    /**
+     * Remember the user password.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	Illuminate\Http\RedirectResponse
+     */
+    public function recover(RecoverRequest $request)
+    {
+        $this->bus->pipeThrough($this->pipesOf('recover'))->dispatchFrom(RecoverJob::class, $request);
+
+        return back()->withSuccess(trans('auth::recover.success'));
+    }
+
+    /**
+     * Display the reset page.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	Illuminate\Http\Response
+     */
+    public function showReset($token)
+    {
+        $reset = config('_auth.reset.view');
+
+        return view($reset);
+    }
+
+    /**
+     * Reset the user password.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	Illuminate\Http\RedirectResponse
+     */
+    public function reset(ResetRequest $request, $token)
+    {
+        $this->bus->pipeThrough($this->pipesOf('reset'))->dispatchFrom(ResetJob::class, $request, compact('token'));
+
+        return redirect()->route('login.index')->withSuccess(trans('auth::reset.success'));
+    }
 
 }

src/Jobs/RecoverJob.php

Doc Comments +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 	/**
 	 * Execute the job.
 	 *
-	 * @return void
+	 * @return string
 	 */
 	public function handle()
 	{

Indentation +23 added lines, -23 removed lines

@@ -4,30 +4,30 @@
 
 class RecoverJob implements SelfHandling {
 
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		string	$email	Email input.
-	 */
-	public $email;
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		string	$email	Email input.
+     */
+    public $email;
 
-	/**
-	 * Create a new job instance.
-	 *
-	 * @return void
-	 */
-	public function __construct($email)
-	{
-		$this->email = $email;
-	}
+    /**
+     * Create a new job instance.
+     *
+     * @return void
+     */
+    public function __construct($email)
+    {
+        $this->email = $email;
+    }
 
-	/**
-	 * Execute the job.
-	 *
-	 * @return void
-	 */
-	public function handle()
-	{
-		return str_random(10);
-	}
+    /**
+     * Execute the job.
+     *
+     * @return void
+     */
+    public function handle()
+    {
+        return str_random(10);
+    }
 
 }

src/Pipes/AbstractEventDispatcherPipe.php

Doc Comments +2 added lines, -2 removed lines

@@ -20,8 +20,8 @@
 	 * Set the dependencies.
 	 *
 	 * @author	Andrea Marco Sartori
-	 * @param	Illuminate\Contracts\Container\Container	$container
-	 * @param	Illuminate\Contracts\Events\Dispatcher	$dispatcher
+	 * @param	Container	$container
+	 * @param	Dispatcher	$dispatcher
 	 * @return	void
 	 */
 	public function __construct(Container $container, Dispatcher $dispatcher)

Indentation +64 added lines, -64 removed lines

@@ -10,78 +10,78 @@
  */
 abstract class AbstractEventDispatcherPipe extends AbstractPipe {
 
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		Illuminate\Contracts\Events\Dispatcher	$dispatcher	Event dispatcher.
-	 */
-	protected $dispatcher;
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		Illuminate\Contracts\Events\Dispatcher	$dispatcher	Event dispatcher.
+     */
+    protected $dispatcher;
 	
-	/**
-	 * Set the dependencies.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	Illuminate\Contracts\Container\Container	$container
-	 * @param	Illuminate\Contracts\Events\Dispatcher	$dispatcher
-	 * @return	void
-	 */
-	public function __construct(Container $container, Dispatcher $dispatcher)
-	{
-		parent::__construct($container);
+    /**
+     * Set the dependencies.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	Illuminate\Contracts\Container\Container	$container
+     * @param	Illuminate\Contracts\Events\Dispatcher	$dispatcher
+     * @return	void
+     */
+    public function __construct(Container $container, Dispatcher $dispatcher)
+    {
+        parent::__construct($container);
 
-		$this->dispatcher = $dispatcher;
-	}
+        $this->dispatcher = $dispatcher;
+    }
 
-	/**
-	 * Run before the job is handled.
-	 *
-	 * @param	mixed	$job
-	 * @return	mixed
-	 */
-	public function before($job)
-	{
-		$this->fireEventOn('start', $job);
-	}
+    /**
+     * Run before the job is handled.
+     *
+     * @param	mixed	$job
+     * @return	mixed
+     */
+    public function before($job)
+    {
+        $this->fireEventOn('start', $job);
+    }
 
-	/**
-	 * Fire an event at some point.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	string	$action
-	 * @param	mixed	$payload
-	 * @return	void
-	 */
-	private function fireEventOn($action, $payload)
-	{
-		$event = $this->getEventName();
+    /**
+     * Fire an event at some point.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	string	$action
+     * @param	mixed	$payload
+     * @return	void
+     */
+    private function fireEventOn($action, $payload)
+    {
+        $event = $this->getEventName();
 
-		$this->dispatcher->fire("auth.{$event}.{$action}", $payload);
-	}
+        $this->dispatcher->fire("auth.{$event}.{$action}", $payload);
+    }
 
-	/**
-	 * Retrieve the event name.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	string
-	 */
-	protected function getEventName()
-	{
-		$chunks = explode('\\', get_class($this));
+    /**
+     * Retrieve the event name.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	string
+     */
+    protected function getEventName()
+    {
+        $chunks = explode('\\', get_class($this));
 
-		$name = $chunks[count($chunks) - 2];
+        $name = $chunks[count($chunks) - 2];
 
-		return strtolower($name);
-	}
+        return strtolower($name);
+    }
 
-	/**
-	 * Run after the handled job.
-	 *
-	 * @param	mixed	$handled
-	 * @param	mixed	$job
-	 * @return	mixed
-	 */
-	public function after($handled, $job)
-	{
-		$this->fireEventOn('end', [$handled, $job]);
-	}
+    /**
+     * Run after the handled job.
+     *
+     * @param	mixed	$handled
+     * @param	mixed	$job
+     * @return	mixed
+     */
+    public function after($handled, $job)
+    {
+        $this->fireEventOn('end', [$handled, $job]);
+    }
 
 }

src/Pipes/AbstractPipe.php

Doc Comments +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 	 * Set the dependencies.
 	 *
 	 * @author	Andrea Marco Sartori
-	 * @param	Illuminate\Contracts\Container\Container	$container
+	 * @param	Container	$container
 	 * @return	void
 	 */
 	public function __construct(Container $container)

Indentation +100 added lines, -100 removed lines

@@ -10,117 +10,117 @@
  */
 abstract class AbstractPipe {
 
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		Illuminate\Contracts\Container\Container	$container	Service container.
-	 */
-	protected $container;
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		Illuminate\Contracts\Container\Container	$container	Service container.
+     */
+    protected $container;
 	
-	/**
-	 * Set the dependencies.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	Illuminate\Contracts\Container\Container	$container
-	 * @return	void
-	 */
-	public function __construct(Container $container)
-	{
-		$this->container = $container;
-	}
+    /**
+     * Set the dependencies.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	Illuminate\Contracts\Container\Container	$container
+     * @return	void
+     */
+    public function __construct(Container $container)
+    {
+        $this->container = $container;
+    }
 
-	/**
-	 * Handle the given job.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	mixed	$job
-	 * @param	Closure	$next
-	 * @return	mixed
-	 */
-	public function handle($job, Closure $next)
-	{
-		$this->callBefore($job);
+    /**
+     * Handle the given job.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	mixed	$job
+     * @param	Closure	$next
+     * @return	mixed
+     */
+    public function handle($job, Closure $next)
+    {
+        $this->callBefore($job);
 
-		$handled = $next($job);
+        $handled = $next($job);
 
-		$this->callAfter($handled, $job);
+        $this->callAfter($handled, $job);
 
-		return $handled;
-	}
+        return $handled;
+    }
 
-	/**
-	 * Call the before method.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	Cerbero\Jobs\Job	$job
-	 * @return	void
-	 */
-	protected function callBefore($job)
-	{
-		$this->callIfExistsAndEnabled('before', [$job]);
-	}
+    /**
+     * Call the before method.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	Cerbero\Jobs\Job	$job
+     * @return	void
+     */
+    protected function callBefore($job)
+    {
+        $this->callIfExistsAndEnabled('before', [$job]);
+    }
 
-	/**
-	 * Call and resolve depepndencies of a method if enabled.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	string	$method
-	 * @param	array	$parameters
-	 * @return	void
-	 */
-	private function callIfExistsAndEnabled($method, array $parameters = [])
-	{
-		if( ! $this->isEnabled()) return;
+    /**
+     * Call and resolve depepndencies of a method if enabled.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	string	$method
+     * @param	array	$parameters
+     * @return	void
+     */
+    private function callIfExistsAndEnabled($method, array $parameters = [])
+    {
+        if( ! $this->isEnabled()) return;
 
-		if(method_exists($this, $method) && $this->{"{$method}IsEnabled"}())
-		{
-			$this->container->call([$this, $method], $parameters);
-		}
-	}
+        if(method_exists($this, $method) && $this->{"{$method}IsEnabled"}())
+        {
+            $this->container->call([$this, $method], $parameters);
+        }
+    }
 
-	/**
-	 * Determine whether the whole pipe has to be processed.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	boolean
-	 */
-	protected function isEnabled()
-	{
-		return true;
-	}
+    /**
+     * Determine whether the whole pipe has to be processed.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	boolean
+     */
+    protected function isEnabled()
+    {
+        return true;
+    }
 
-	/**
-	 * Call the after method.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	mixed	$handled
-	 * @param	Cerbero\Jobs\Job	$job
-	 * @return	void
-	 */
-	protected function callAfter($handled, $job)
-	{
-		$this->callIfExistsAndEnabled('after', [$handled, $job]);
-	}
+    /**
+     * Call the after method.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	mixed	$handled
+     * @param	Cerbero\Jobs\Job	$job
+     * @return	void
+     */
+    protected function callAfter($handled, $job)
+    {
+        $this->callIfExistsAndEnabled('after', [$handled, $job]);
+    }
 
-	/**
-	 * Determine whether the before method has to be processed.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	boolean
-	 */
-	protected function beforeIsEnabled()
-	{
-		return true;
-	}
+    /**
+     * Determine whether the before method has to be processed.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	boolean
+     */
+    protected function beforeIsEnabled()
+    {
+        return true;
+    }
 
-	/**
-	 * Determine whether the after method has to be processed.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	boolean
-	 */
-	protected function afterIsEnabled()
-	{
-		return true;
-	}
+    /**
+     * Determine whether the after method has to be processed.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	boolean
+     */
+    protected function afterIsEnabled()
+    {
+        return true;
+    }
 
 }

Spacing +2 added lines, -2 removed lines

@@ -69,9 +69,9 @@
 	 */
 	private function callIfExistsAndEnabled($method, array $parameters = [])
 	{
-		if( ! $this->isEnabled()) return;
+		if (!$this->isEnabled()) return;
 
-		if(method_exists($this, $method) && $this->{"{$method}IsEnabled"}())
+		if (method_exists($this, $method) && $this->{"{$method}IsEnabled"}())
 		{
 			$this->container->call([$this, $method], $parameters);
 		}

Braces +3 added lines, -1 removed lines

@@ -69,7 +69,9 @@
 	 */
 	private function callIfExistsAndEnabled($method, array $parameters = [])
 	{
-		if( ! $this->isEnabled()) return;
+		if( ! $this->isEnabled()) {
+		    return;
+		}
 
 		if(method_exists($this, $method) && $this->{"{$method}IsEnabled"}())
 		{

src/Pipes/Login/Throttle.php

Doc Comments +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@ 
 	/**
 	 * Run before the job is handled.
 	 *
-	 * @param	Cerbero\Auth\Services\Throttling\ThrottlerInterface	$throttler
+	 * @param	ThrottlerInterface	$throttler
 	 * @param	Cerbero\Auth\Jobs\LoginJob	$job
 	 * @return	mixed
 	 */
@@ -46,7 +46,7 @@ 
 	/**
 	 * Run after the handled job.
 	 *
-	 * @param	Cerbero\Auth\Services\Throttling\ThrottlerInterface	$throttler
+	 * @param	ThrottlerInterface	$throttler
 	 * @param	mixed	$handled
 	 * @param	Cerbero\Auth\Jobs\LoginJob	$job
 	 * @return	mixed

Indentation +52 added lines, -52 removed lines

@@ -6,65 +6,65 @@
 
 class Throttle extends AbstractPipe {
 
-	/**
-	 * Run before the job is handled.
-	 *
-	 * @param	Cerbero\Auth\Services\Throttling\ThrottlerInterface	$throttler
-	 * @param	Cerbero\Auth\Jobs\LoginJob	$job
-	 * @return	mixed
-	 */
-	public function before(ThrottlerInterface $throttler, $job)
-	{
-		$throttler->setSource($this->getSourceByJob($job));
+    /**
+     * Run before the job is handled.
+     *
+     * @param	Cerbero\Auth\Services\Throttling\ThrottlerInterface	$throttler
+     * @param	Cerbero\Auth\Jobs\LoginJob	$job
+     * @return	mixed
+     */
+    public function before(ThrottlerInterface $throttler, $job)
+    {
+        $throttler->setSource($this->getSourceByJob($job));
 
-		$throttler->incrementAttempts();
+        $throttler->incrementAttempts();
 
-		if($throttler->tooManyAttempts() || $throttler->lockedOut())
-		{
-			$throttler->lockOut();
+        if($throttler->tooManyAttempts() || $throttler->lockedOut())
+        {
+            $throttler->lockOut();
 
-			$seconds = $throttler->getRemainingSeconds();
+            $seconds = $throttler->getRemainingSeconds();
 
-			throw new DisplayException('auth::throttling.error', compact('seconds'));
-		}
-	}
+            throw new DisplayException('auth::throttling.error', compact('seconds'));
+        }
+    }
 
-	/**
-	 * Create source by using the given job.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	type	$job
-	 * @return	string
-	 */
-	private function getSourceByJob($job)
-	{
-		$login = head(array_except($job->credentials, 'password'));
+    /**
+     * Create source by using the given job.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	type	$job
+     * @return	string
+     */
+    private function getSourceByJob($job)
+    {
+        $login = head(array_except($job->credentials, 'password'));
 
-		return $login . app('request')->ip();
-	}
+        return $login . app('request')->ip();
+    }
 
-	/**
-	 * Run after the handled job.
-	 *
-	 * @param	Cerbero\Auth\Services\Throttling\ThrottlerInterface	$throttler
-	 * @param	mixed	$handled
-	 * @param	Cerbero\Auth\Jobs\LoginJob	$job
-	 * @return	mixed
-	 */
-	public function after(ThrottlerInterface $throttler, $handled, $job)
-	{
-		$throttler->resetAttempts();
-	}
+    /**
+     * Run after the handled job.
+     *
+     * @param	Cerbero\Auth\Services\Throttling\ThrottlerInterface	$throttler
+     * @param	mixed	$handled
+     * @param	Cerbero\Auth\Jobs\LoginJob	$job
+     * @return	mixed
+     */
+    public function after(ThrottlerInterface $throttler, $handled, $job)
+    {
+        $throttler->resetAttempts();
+    }
 
-	/**
-	 * Determine whether the whole pipe has to be processed.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	boolean
-	 */
-	protected function isEnabled()
-	{
-		return config('_auth.login.throttling.enabled');
-	}
+    /**
+     * Determine whether the whole pipe has to be processed.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	boolean
+     */
+    protected function isEnabled()
+    {
+        return config('_auth.login.throttling.enabled');
+    }
 
 }

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 
 		$throttler->incrementAttempts();
 
-		if($throttler->tooManyAttempts() || $throttler->lockedOut())
+		if ($throttler->tooManyAttempts() || $throttler->lockedOut())
 		{
 			$throttler->lockOut();
 

src/Pipes/Register/Login.php

Doc Comments +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 	/**
 	 * Run after the handled job.
 	 *
-	 * @param	Illuminate\Contracts\Auth\Guard	$auth
+	 * @param	Guard	$auth
 	 * @param	mixed	$handled
 	 * @param	Cerbero\Auth\Jobs\LoginJob	$job
 	 * @return	mixed

Indentation +22 added lines, -22 removed lines

@@ -5,28 +5,28 @@
 
 class Login extends AbstractPipe {
 
-	/**
-	 * Run after the handled job.
-	 *
-	 * @param	Illuminate\Contracts\Auth\Guard	$auth
-	 * @param	mixed	$handled
-	 * @param	Cerbero\Auth\Jobs\LoginJob	$job
-	 * @return	mixed
-	 */
-	public function after(Guard $auth, $handled, $job)
-	{
-		$auth->login($handled);
-	}
+    /**
+     * Run after the handled job.
+     *
+     * @param	Illuminate\Contracts\Auth\Guard	$auth
+     * @param	mixed	$handled
+     * @param	Cerbero\Auth\Jobs\LoginJob	$job
+     * @return	mixed
+     */
+    public function after(Guard $auth, $handled, $job)
+    {
+        $auth->login($handled);
+    }
 
-	/**
-	 * Determine whether the after method has to be processed.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	boolean
-	 */
-	protected function afterIsEnabled()
-	{
-		return config('_auth.register.login_after_registering');
-	}
+    /**
+     * Determine whether the after method has to be processed.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	boolean
+     */
+    protected function afterIsEnabled()
+    {
+        return config('_auth.register.login_after_registering');
+    }
 
 }

src/Pipes/Register/Notify.php

Doc Comments +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
 	/**
 	 * Run after the handled job.
 	 *
-	 * @param	Illuminate\Contracts\Mail\Mailer	$mailer
+	 * @param	Mailer	$mailer
 	 * @param	mixed	$handled
 	 * @param	Cerbero\Auth\Jobs\RegisterJob	$job
 	 * @return	mixed

Indentation +54 added lines, -54 removed lines

@@ -5,67 +5,67 @@
 
 class Notify extends AbstractPipe {
 
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		string	$password	Generated password.
-	 */
-	protected $password = null;
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		string	$password	Generated password.
+     */
+    protected $password = null;
 
-	/**
-	 * Run before the job is handled.
-	 *
-	 * @param	Cerbero\Auth\Jobs\RegisterJob	$job
-	 * @return	mixed
-	 */
-	public function before($job)
-	{
-		$this->password = str_random(8);
+    /**
+     * Run before the job is handled.
+     *
+     * @param	Cerbero\Auth\Jobs\RegisterJob	$job
+     * @return	mixed
+     */
+    public function before($job)
+    {
+        $this->password = str_random(8);
 
-		$job->attributes['password'] = $this->password;
-	}
+        $job->attributes['password'] = $this->password;
+    }
 
-	/**
-	 * Determine whether the before method has to be processed.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	boolean
-	 */
-	protected function beforeIsEnabled()
-	{
-		return config('_auth.register.email.generate_password_for_user');
-	}
+    /**
+     * Determine whether the before method has to be processed.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	boolean
+     */
+    protected function beforeIsEnabled()
+    {
+        return config('_auth.register.email.generate_password_for_user');
+    }
 
-	/**
-	 * Run after the handled job.
-	 *
-	 * @param	Illuminate\Contracts\Mail\Mailer	$mailer
-	 * @param	mixed	$handled
-	 * @param	Cerbero\Auth\Jobs\RegisterJob	$job
-	 * @return	mixed
-	 */
-	public function after(Mailer $mailer, $handled, $job)
-	{
-		$email = $handled->email;
+    /**
+     * Run after the handled job.
+     *
+     * @param	Illuminate\Contracts\Mail\Mailer	$mailer
+     * @param	mixed	$handled
+     * @param	Cerbero\Auth\Jobs\RegisterJob	$job
+     * @return	mixed
+     */
+    public function after(Mailer $mailer, $handled, $job)
+    {
+        $email = $handled->email;
 
-		$payload = ['user' => $handled, 'password' => $this->password];
+        $payload = ['user' => $handled, 'password' => $this->password];
 
-		$method = config('_auth.register.email.queue') ? 'queue' : 'send';
+        $method = config('_auth.register.email.queue') ? 'queue' : 'send';
 
-		$mailer->$method(config('_auth.register.email.view'), $payload, function($message) use($email)
-		{
-			$message->to($email)->subject(trans('auth::register.email_subject'));
-		});
-	}
+        $mailer->$method(config('_auth.register.email.view'), $payload, function($message) use($email)
+        {
+            $message->to($email)->subject(trans('auth::register.email_subject'));
+        });
+    }
 
-	/**
-	 * Determine whether the after method has to be processed.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	boolean
-	 */
-	protected function afterIsEnabled()
-	{
-		return config('_auth.register.email.send');
-	}
+    /**
+     * Determine whether the after method has to be processed.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	boolean
+     */
+    protected function afterIsEnabled()
+    {
+        return config('_auth.register.email.send');
+    }
 
 }

src/Repositories/EloquentUserRepository.php

Doc Comments +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
 	 * @author	Andrea Marco Sartori
 	 * @param	User	$user
 	 * @param	string	$password
-	 * @return	boolean
+	 * @return	boolean|null
 	 */
 	public function resetPassword($user, $password)
 	{

Indentation +66 added lines, -66 removed lines

@@ -7,81 +7,81 @@
  */
 class EloquentUserRepository implements UserRepositoryInterface {
 
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		User	$user	User model.
-	 */
-	protected $user;
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		User	$user	User model.
+     */
+    protected $user;
 	
-	/**
-	 * Set the dependencies.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	void
-	 */
-	public function __construct()
-	{
-		$user = config('auth.model');
+    /**
+     * Set the dependencies.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	void
+     */
+    public function __construct()
+    {
+        $user = config('auth.model');
 
-		$this->user = new $user;
-	}
+        $this->user = new $user;
+    }
 
-	/**
-	 * Register a new user.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	array	$attributes
-	 * @return	User
-	 */
-	public function register(array $attributes)
-	{
-		return $this->user->create($attributes);
-	}
+    /**
+     * Register a new user.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	array	$attributes
+     * @return	User
+     */
+    public function register(array $attributes)
+    {
+        return $this->user->create($attributes);
+    }
 
-	/**
-	 * Assign a token to reset the password of the user with the given email.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	string	$token
-	 * @param	string	$email
-	 * @return	void
-	 */
-	public function assignResetToken($token, $email)
-	{
-		$user = $this->user->whereEmail($email)->first();
+    /**
+     * Assign a token to reset the password of the user with the given email.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	string	$token
+     * @param	string	$email
+     * @return	void
+     */
+    public function assignResetToken($token, $email)
+    {
+        $user = $this->user->whereEmail($email)->first();
 
-		$user->reset_token = $token;
+        $user->reset_token = $token;
 
-		$user->save();
-	}
+        $user->save();
+    }
 
-	/**
-	 * Retrieve the user with the given reset token.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	string	$token
-	 * @return	User|null
-	 */
-	public function findByResetToken($token)
-	{
-		return $this->user->whereResetToken($token)->first();
-	}
+    /**
+     * Retrieve the user with the given reset token.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	string	$token
+     * @return	User|null
+     */
+    public function findByResetToken($token)
+    {
+        return $this->user->whereResetToken($token)->first();
+    }
 
-	/**
-	 * Reset the password of the given user.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	User	$user
-	 * @param	string	$password
-	 * @return	boolean
-	 */
-	public function resetPassword($user, $password)
-	{
-		$user->password = $password;
+    /**
+     * Reset the password of the given user.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	User	$user
+     * @param	string	$password
+     * @return	boolean
+     */
+    public function resetPassword($user, $password)
+    {
+        $user->password = $password;
 
-		$user->reset_token = null;
+        $user->reset_token = null;
 
-		$user->save();
-	}
+        $user->save();
+    }
 
 }

src/Services/Throttling/CachingThrottler.php

Doc Comments +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
 	 * Set the dependencies.
 	 *
 	 * @author	Andrea Marco Sartori
-	 * @param	Illuminate\Contracts\Cache\Repository	$cache
+	 * @param	Cache	$cache
 	 * @return	void
 	 */
 	public function __construct(Cache $cache)
@@ -46,7 +46,7 @@ 
 	 * Set univocal source where attempts come from.
 	 *
 	 * @author	Andrea Marco Sartori
-	 * @return	boolean
+	 * @return	boolean|null
 	 */
 	public function setSource($source)
 	{

Indentation +135 added lines, -135 removed lines

@@ -12,141 +12,141 @@
 class CachingThrottler implements ThrottlerInterface
 {
 
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		Illuminate\Contracts\Cache\Repository	$cache	Cache repository.
-	 */
-	protected $cache;
-
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		string	$key	Source where attempts come from.
-	 */
-	protected $key;
-
-	/**
-	 * @author	Andrea Marco Sartori
-	 * @var		string	$lockOutKey	Key to store lock out time in.
-	 */
-	protected $lockOutKey;
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		Illuminate\Contracts\Cache\Repository	$cache	Cache repository.
+     */
+    protected $cache;
+
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		string	$key	Source where attempts come from.
+     */
+    protected $key;
+
+    /**
+     * @author	Andrea Marco Sartori
+     * @var		string	$lockOutKey	Key to store lock out time in.
+     */
+    protected $lockOutKey;
 	
-	/**
-	 * Set the dependencies.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @param	Illuminate\Contracts\Cache\Repository	$cache
-	 * @return	void
-	 */
-	public function __construct(Cache $cache)
-	{
-		$this->cache = $cache;
-	}
-
-	/**
-	 * Set univocal source where attempts come from.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	boolean
-	 */
-	public function setSource($source)
-	{
-		$this->key = $source;
-
-		$this->lockOutKey = "{$source}:lockout";
-	}
-
-	/**
-	 * Determine whether the user has been locked out.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	boolean
-	 */
-	public function lockedOut()
-	{
-		return $this->cache->has($this->lockOutKey);
-	}
-
-	/**
-	 * Retrieve the number of remaining seconds before the next attempt.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	integer
-	 */
-	public function getRemainingSeconds()
-	{
-		return $this->cache->get($this->lockOutKey) - time();
-	}
-
-	/**
-	 * Increment the number of failed attempts.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	void
-	 */
-	public function incrementAttempts()
-	{
-		$this->cache->add($this->key, 0, $this->getExpiry());
-
-		$this->cache->increment($this->key);
-	}
-
-	/**
-	 * Retrieve the minutes after which keys expire.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	integer
-	 */
-	private function getExpiry()
-	{
-		return (int) $this->getDelay() / 60;
-	}
-
-	/**
-	 * Retrieve the seconds to wait before the next attempt.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	integer
-	 */
-	private function getDelay()
-	{
-		return config('_auth.login.throttling.delay');
-	}
-
-	/**
-	 * Determine whether a user has performed too many attempts.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	boolean
-	 */
-	public function tooManyAttempts()
-	{
-		$maximum = config('_auth.login.throttling.max_attempts');
-
-		return $this->cache->get($this->key, 0) > $maximum;
-	}
-
-	/**
-	 * Lock a user out.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	void
-	 */
-	public function lockOut()
-	{
-		$this->resetAttempts();
-
-		$this->cache->add($this->lockOutKey, $this->getDelay() + time(), $this->getExpiry());
-	}
-
-	/**
-	 * Reset the attempts counter.
-	 *
-	 * @author	Andrea Marco Sartori
-	 * @return	void
-	 */
-	public function resetAttempts()
-	{
-		$this->cache->forget($this->key);
-	}
+    /**
+     * Set the dependencies.
+     *
+     * @author	Andrea Marco Sartori
+     * @param	Illuminate\Contracts\Cache\Repository	$cache
+     * @return	void
+     */
+    public function __construct(Cache $cache)
+    {
+        $this->cache = $cache;
+    }
+
+    /**
+     * Set univocal source where attempts come from.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	boolean
+     */
+    public function setSource($source)
+    {
+        $this->key = $source;
+
+        $this->lockOutKey = "{$source}:lockout";
+    }
+
+    /**
+     * Determine whether the user has been locked out.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	boolean
+     */
+    public function lockedOut()
+    {
+        return $this->cache->has($this->lockOutKey);
+    }
+
+    /**
+     * Retrieve the number of remaining seconds before the next attempt.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	integer
+     */
+    public function getRemainingSeconds()
+    {
+        return $this->cache->get($this->lockOutKey) - time();
+    }
+
+    /**
+     * Increment the number of failed attempts.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	void
+     */
+    public function incrementAttempts()
+    {
+        $this->cache->add($this->key, 0, $this->getExpiry());
+
+        $this->cache->increment($this->key);
+    }
+
+    /**
+     * Retrieve the minutes after which keys expire.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	integer
+     */
+    private function getExpiry()
+    {
+        return (int) $this->getDelay() / 60;
+    }
+
+    /**
+     * Retrieve the seconds to wait before the next attempt.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	integer
+     */
+    private function getDelay()
+    {
+        return config('_auth.login.throttling.delay');
+    }
+
+    /**
+     * Determine whether a user has performed too many attempts.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	boolean
+     */
+    public function tooManyAttempts()
+    {
+        $maximum = config('_auth.login.throttling.max_attempts');
+
+        return $this->cache->get($this->key, 0) > $maximum;
+    }
+
+    /**
+     * Lock a user out.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	void
+     */
+    public function lockOut()
+    {
+        $this->resetAttempts();
+
+        $this->cache->add($this->lockOutKey, $this->getDelay() + time(), $this->getExpiry());
+    }
+
+    /**
+     * Reset the attempts counter.
+     *
+     * @author	Andrea Marco Sartori
+     * @return	void
+     */
+    public function resetAttempts()
+    {
+        $this->cache->forget($this->key);
+    }
 
 }