WITR\ViewComposers\AuthViewComposer

Complexity

Total Complexity

2

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	compose()	0	6	1

<?php

namespace WITR\ViewComposers;

use Illuminate\Http\Request;
use Illuminate\View\View;
use Illuminate\Contracts\Auth\Guard;
use WITR\Services\Whitelist;

class AuthViewComposer {

	protected $auth;
	protected $whitelist;
	protected $request;

	public function __construct(Guard $auth, Whitelist $whitelist, Request $request)

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

	{
		$this->auth = $auth;
		$this->whitelist = $whitelist;
		$this->request = $request;
	}

	/**
	 * Bind data to the view.
	 *
	 * @param  View  $view
	 * @return void
	 */
	public function compose(View $view)
	{
		$view->with('user', $this->auth->user())
			 ->with('authenticated', $this->auth->check())
			 ->with('in_station', $this->whitelist->inRange($this->request));
	}
}