WITR\Services\Whitelist

Complexity

Total Complexity

4

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	inRange()	0	9	3

<?php namespace WITR\Services;

use Closure;
use Illuminate\Contracts\Auth\Guard;

class Whitelist {

	protected $fromIp;
	protected $toIp;

	/**
	 * Create a new filter instance.
	 *
	 * @param  Guard  $auth

There is no parameter named $auth. Was it maybe removed?

	 * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

	 */
	public function __construct($fromIp, $toIp)
	{
		$this->fromIp = $fromIp;
		$this->toIp = $toIp;
	}

	/**
	 * Handle an incoming request.
	 *
	 * @param  \Illuminate\Http\Request  $request
	 * @param  \Closure  $next

There is no parameter named $next. Was it maybe removed?

	 * @return mixed
	 */
	public function inRange($request)
	{
		$ip = $request->getClientIp();
		if( (ip2long($this->fromIp) <= ip2long($ip)) && (ip2long($ip) <= ip2long($this->toIp))) {
			return true;
		} else {
			return false;
		}
	}

}