LaraCrafts\GeoRoutes\DeterminesGeoAccess

Complexity

Total Complexity

3

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	shouldHaveAccess()	0	16	3

<?php

namespace LaraCrafts\GeoRoutes;

use Illuminate\Http\Request;
use Stevebauman\Location\Facades\Location;

trait DeterminesGeoAccess
{
    /**
     * Get the countries.
     *
     * @return string[]
     */
    abstract public function getCountries();

    /**
     * Get the strategy.
     *
     * @return string
     */
    abstract public function getStrategy();

    /**
     * Determine if the request should be allowed through.
     *
     * @param \Illuminate\Http\Request $request
     * @return bool
     */
    public function shouldHaveAccess(Request $request)
    {
        $countries = $this->getCountries();
        $strategy = $this->getStrategy();

        if (!$countries) {

The expression $countries of type string[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return $strategy !== 'allow';
        }

        $requestCountry = Location::get($request->ip())->countryCode;

        if ($strategy === 'allow') {
            return in_array($requestCountry, $countries);
        }

        return !in_array($requestCountry, $countries);
    }
}