ConfigRetriever::isAdditionalConfig() - Code Metrics - Inspection of "change spoofedConfig visibility" - SocialiteProviders/Manager - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( ac108b...8f997c )

by Brian

created 2017-11-20 08:41 UTC

ConfigRetriever::isAdditionalConfig() A

↳ Parent: ConfigRetriever

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	4
ccs	0
cts	0
cp	0
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	1
crap	2

<?php

namespace SocialiteProviders\Manager\Helpers;

use SocialiteProviders\Manager\Config;
use SocialiteProviders\Manager\SocialiteWasCalled;
use SocialiteProviders\Manager\Exception\MissingConfigException;
use SocialiteProviders\Manager\Contracts\Helpers\ConfigRetrieverInterface;

class ConfigRetriever implements ConfigRetrieverInterface
{
    /**
     * @var string
     */
    protected $providerName;

    /**
     * @var string
     */
    protected $providerIdentifier;

    /**
     * @var array
     */
    protected $servicesArray;

    /**
     * @var array
     */
    protected $additionalConfigKeys;

    /**
     * @param string $providerIdentifier
     * @param array  $additionalConfigKeys
     *
     * @throws \SocialiteProviders\Manager\Exception\MissingConfigException
     *
     * @return \SocialiteProviders\Manager\Contracts\ConfigInterface
     */
    public function fromEnv($providerIdentifier, array $additionalConfigKeys = [])
    {
        $this->providerIdentifier = $providerIdentifier;
        $this->additionalConfigKeys = $additionalConfigKeys;

        return new Config(
            $this->getFromEnv('KEY'),
            $this->getFromEnv('SECRET'),
            $this->getFromEnv('REDIRECT_URI'),
            $this->getConfigItems($additionalConfigKeys, function ($key) {
                return $this->getFromEnv(strtoupper($key));
            }));
    }

    /**
     * @param string $providerName
     * @param array  $additionalConfigKeys
     *
     * @throws \SocialiteProviders\Manager\Exception\MissingConfigException
     *
     * @return \SocialiteProviders\Manager\Contracts\ConfigInterface
     */
    public function fromServices($providerName, array $additionalConfigKeys = [])
    {
        $this->providerName = $providerName;
        $this->getConfigFromServicesArray($providerName);

        $this->additionalConfigKeys = $additionalConfigKeys;

        return new Config(
            $this->getFromServices('client_id'),
            $this->getFromServices('client_secret'),
            $this->getFromServices('redirect'),
            $this->getConfigItems($additionalConfigKeys, function ($key) {
                return $this->getFromServices(strtolower($key));
            }));
    }

    /**
     * @param array    $configKeys
     * @param \Closure $keyRetrievalClosure
     *
     * @return array
     */
    protected function getConfigItems(array $configKeys, \Closure $keyRetrievalClosure)
    {
        if (count($configKeys) < 1) {
            return [];
        }

        return $this->retrieveItemsFromConfig($configKeys, $keyRetrievalClosure);
    }

    /**
     * @param array    $keys
     * @param \Closure $keyRetrievalClosure
     *
     * @return array
     */
    protected function retrieveItemsFromConfig(array $keys, \Closure $keyRetrievalClosure)
    {
        $out = [];

        foreach ($keys as $key) {
            $out[$key] = $keyRetrievalClosure($key);
        }

        return $out;
    }

    /**
     * @param string $key
     *
     * @throws \SocialiteProviders\Manager\Exception\MissingConfigException
     *
     * @return string
     */
    protected function getFromServices($key)
    {
        $keyExists = array_key_exists($key, $this->servicesArray);

        // ADDITIONAL value is empty
        if (! $keyExists && $this->isAdditionalConfig($key)) {
            return;
        }

        // REQUIRED value is empty
        if (! $keyExists) {
            throw new MissingConfigException("Missing services entry for {$this->providerName}.$key");
        }

        return $this->servicesArray[$key];
    }

    /**
     * @param string $key
     *
     * @throws \SocialiteProviders\Manager\Exception\MissingConfigException
     *
     * @return string
     */
    protected function getFromEnv($key)
    {
        $providerKey = "{$this->providerIdentifier}_{$key}";
        $item = env($providerKey);

        // ADDITIONAL value is empty
        if (empty($item) && $this->isAdditionalConfig($key)) {
            return;
        }

        // REQUIRED value is empty
        if (empty($item)) {
            // If we are running in console we should spoof values to make Socialite happy...
            if (! app()->runningInConsole()) {
                throw new MissingConfigException("Configuration for $providerKey is missing.");
            }
            $item = $providerKey;
        }

        return $item;
    }

    /**
     * @param string $providerName
     *
     * @throws \SocialiteProviders\Manager\Exception\MissingConfigException
     *
     * @return array
     */
    protected function getConfigFromServicesArray($providerName)
    {
        /** @var array $configArray */
        $configArray = config("services.$providerName");

        if (empty($configArray)) {
            // If we are running in console we should spoof values to make Socialite happy...
            if (app()->runningInConsole()) {
                $configArray = [
                    'client_id' => "{$this->providerIdentifier}_KEY",
                    'client_secret' => "{$this->providerIdentifier}_SECRET",
                    'redirect' => "{$this->providerIdentifier}_REDIRECT_URI",
                ];
            } else {
                throw new MissingConfigException("There is no services entry for $providerName");
            }
        }

        $this->servicesArray = $configArray;

        return $this->servicesArray;
    }

    /**
     * @param string $key
     *
     * @return bool
     */
    protected function isAdditionalConfig($key)
    {
        return in_array(strtolower($key), $this->additionalConfigKeys);
    }
}


1		<?php
2
3		namespace SocialiteProviders\Manager\Helpers;
4
5		use SocialiteProviders\Manager\Config;
6		use SocialiteProviders\Manager\SocialiteWasCalled;
7		use SocialiteProviders\Manager\Exception\MissingConfigException;
8		use SocialiteProviders\Manager\Contracts\Helpers\ConfigRetrieverInterface;
9
10		class ConfigRetriever implements ConfigRetrieverInterface
11		{
12		/**
13		* @var string
14		*/
15		protected $providerName;
16
17		/**
18		* @var string
19		*/
20		protected $providerIdentifier;
21
22		/**
23		* @var array
24		*/
25		protected $servicesArray;
26
27		/**
28		* @var array
29		*/
30		protected $additionalConfigKeys;
31
32		/**
33	2	* @param string $providerIdentifier
34		* @param array $additionalConfigKeys
35	2	*
36		* @throws \SocialiteProviders\Manager\Exception\MissingConfigException
37	2	*
38	2	* @return \SocialiteProviders\Manager\Contracts\ConfigInterface
39	1	*/
40	1	public function fromEnv($providerIdentifier, array $additionalConfigKeys = [])
41		{
42	1	$this->providerIdentifier = $providerIdentifier;
43	1	$this->additionalConfigKeys = $additionalConfigKeys;
44
45		return new Config(
46		$this->getFromEnv('KEY'),
47		$this->getFromEnv('SECRET'),
48		$this->getFromEnv('REDIRECT_URI'),
49		$this->getConfigItems($additionalConfigKeys, function ($key) {
50		return $this->getFromEnv(strtoupper($key));
51		}));
52		}
53	2
54		/**
55	2	* @param string $providerName
56	2	* @param array $additionalConfigKeys
57		*
58	1	* @throws \SocialiteProviders\Manager\Exception\MissingConfigException
59	1	*
60	1	* @return \SocialiteProviders\Manager\Contracts\ConfigInterface
61	1	*/
62	1	public function fromServices($providerName, array $additionalConfigKeys = [])
63	1	{
64	1	$this->providerName = $providerName;
65		$this->getConfigFromServicesArray($providerName);
66
67		$this->additionalConfigKeys = $additionalConfigKeys;
68
69		return new Config(
70		$this->getFromServices('client_id'),
71		$this->getFromServices('client_secret'),
72		$this->getFromServices('redirect'),
73	2	$this->getConfigItems($additionalConfigKeys, function ($key) {
74		return $this->getFromServices(strtolower($key));
75	2	}));
76		}
77
78		/**
79	2	* @param array $configKeys
80		* @param \Closure $keyRetrievalClosure
81		*
82		* @return array
83		*/
84		protected function getConfigItems(array $configKeys, \Closure $keyRetrievalClosure)
85		{
86		if (count($configKeys) < 1) {
87		return [];
88	2	}
89
90	2	return $this->retrieveItemsFromConfig($configKeys, $keyRetrievalClosure);
91		}
92	2
93	2	/**
94	2	* @param array $keys
95		* @param \Closure $keyRetrievalClosure
96	2	*
97		* @return array
98		*/
99		protected function retrieveItemsFromConfig(array $keys, \Closure $keyRetrievalClosure)
100		{
101		$out = [];
102
103		foreach ($keys as $key) {
104		$out[$key] = $keyRetrievalClosure($key);
105	1	}
106
107	1	return $out;
108		}
109
110		/**
111	1	* @param string $key
112		*
113		* @throws \SocialiteProviders\Manager\Exception\MissingConfigException
114		*
115		* @return string
116		*/
117		protected function getFromServices($key)
118		{
119		$keyExists = array_key_exists($key, $this->servicesArray);
120	2
121		// ADDITIONAL value is empty
122	2	if (! $keyExists && $this->isAdditionalConfig($key)) {
123	2	return;
124		}
125	2
126	1	// REQUIRED value is empty
127		if (! $keyExists) {
128		throw new MissingConfigException("Missing services entry for {$this->providerName}.$key");
129	1	}
130
131		return $this->servicesArray[$key];
132		}
133
134		/**
135		* @param string $key
136		*
137		* @throws \SocialiteProviders\Manager\Exception\MissingConfigException
138	2	*
139		* @return string
140	2	*/
141	2	protected function getFromEnv($key)
142		{
143	2	$providerKey = "{$this->providerIdentifier}_{$key}";
144	1	$item = env($providerKey);
145
146	1	// ADDITIONAL value is empty
147		if (empty($item) && $this->isAdditionalConfig($key)) {
148	1	return;
149		}
150
151		// REQUIRED value is empty
152		if (empty($item)) {
153		// If we are running in console we should spoof values to make Socialite happy...
154		if (! app()->runningInConsole()) {
155		throw new MissingConfigException("Configuration for $providerKey is missing.");
156		}
157		$item = $providerKey;
158		}
159
160		return $item;
161		}
162
163		/**
164		* @param string $providerName
165		*
166		* @throws \SocialiteProviders\Manager\Exception\MissingConfigException
167		*
168		* @return array
169		*/
170		protected function getConfigFromServicesArray($providerName)
171		{
172		/** @var array $configArray */
173		$configArray = config("services.$providerName");
174
175		if (empty($configArray)) {
176		// If we are running in console we should spoof values to make Socialite happy...
177		if (app()->runningInConsole()) {
178		$configArray = [
179		'client_id' => "{$this->providerIdentifier}_KEY",
180		'client_secret' => "{$this->providerIdentifier}_SECRET",
181		'redirect' => "{$this->providerIdentifier}_REDIRECT_URI",
182		];
183		} else {
184		throw new MissingConfigException("There is no services entry for $providerName");
185		}
186		}
187
188		$this->servicesArray = $configArray;
189
190		return $this->servicesArray;
191		}
192
193		/**
194		* @param string $key
195		*
196		* @return bool
197		*/
198		protected function isAdditionalConfig($key)
199		{
200		return in_array(strtolower($key), $this->additionalConfigKeys);
201		}
202		}
203

SocialiteProviders / Manager

Push — master ( ac108b...8f997c )

ConfigRetriever::isAdditionalConfig() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like