VariableResolver::__construct() - Code Metrics - Inspection of "Hotfix/1.1.17" - 24Slides/auth-connector - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#6)

unknown

created 2020-03-10 13:51 UTC

VariableResolver::__construct() A

↳ Parent: VariableResolver

Complexity

Conditions	2
Paths	2

Size

Total Lines	6
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	1
Bugs	0	Features	0

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

<?php

namespace Slides\Connector\Auth\Clients\Mandrill;

use Exception;
use Illuminate\Support\Collection;
use Illuminate\Support\Str;

/**
 * Class VariableResolver
 *
 * @package Slides\Connector\Auth\Clients\Mandrill
 */
class VariableResolver
{
    /**
     * @var Collection
     */
    protected $emails;

    /**
     * VariableResolver constructor.
     *
     * @param Collection $emails
     */
    final public function __construct(Collection $emails)
    {
        $this->emails = $emails;

        if (method_exists($this, 'boot')) {
            app()->call([$this, 'boot']);
        }
    }

    /**
     * Resolver given variable.
     *
     * @param string $variable
     * @param string $email
     *
     * @return array
     *
     * @throws Exception
     */
    final public function resolve(string $variable, string $email): array
    {
        $name = Str::before($variable, ':');

        $arguments[] = $email;


        if (Str::contains($variable, ':')) {
            $arguments = array_merge($arguments, explode(',', Str::after($variable, $name . ':')));
        }

        $method = 'get' . ucfirst($name) . 'Variable';

        if (!method_exists($this, $method)) {
            throw new Exception('Method ' . $method . ' should be defined.');
        }

        return [
            'name' => $name,
            'content' => call_user_func([$this, $method], ...$arguments)
        ];
    }
}

1			<?php
2
3			namespace Slides\Connector\Auth\Clients\Mandrill;
4
5			use Exception;
6			use Illuminate\Support\Collection;
7			use Illuminate\Support\Str;
8
9			/**
10			* Class VariableResolver
11			*
12			* @package Slides\Connector\Auth\Clients\Mandrill
13			*/
14			class VariableResolver
15			{
16			/**
17			* @var Collection
18			*/
19			protected $emails;
20
21			/**
22			* VariableResolver constructor.
23			*
24			* @param Collection $emails
25			*/
26			final public function __construct(Collection $emails)
27			{
28			$this->emails = $emails;
29
30			if (method_exists($this, 'boot')) {
31			app()->call([$this, 'boot']);
32			}
33			}
34
35			/**
36			* Resolver given variable.
37			*
38			* @param string $variable
39			* @param string $email
40			*
41			* @return array
42			*
43			* @throws Exception
44			*/
45			final public function resolve(string $variable, string $email): array
46			{
47			$name = Str::before($variable, ':');
48
49			$arguments[] = $email;
			0 ignored issues – show Comprehensibility Best Practice introduced 2020-03-10 13:55 UTC by Report Bug Copy Issue Report `$arguments` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$arguments = array();` before regardless. Loading history...
50
51			if (Str::contains($variable, ':')) {
52			$arguments = array_merge($arguments, explode(',', Str::after($variable, $name . ':')));
53			}
54
55			$method = 'get' . ucfirst($name) . 'Variable';
56
57			if (!method_exists($this, $method)) {
58			throw new Exception('Method ' . $method . ' should be defined.');
59			}
60
61			return [
62			'name' => $name,
63			'content' => call_user_func([$this, $method], ...$arguments)
64			];
65			}
66			}

24Slides / auth-connector

Pull Request — master (#6)

VariableResolver::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like