Marek\Toggable\Factory\Authentication\AuthenticationFactory

Complexity

Total Complexity

6

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	2
A	build()	0	19	4

<?php

namespace Marek\Toggable\Factory\Authentication;

use Marek\Toggable\API\Security\ApiToken;
use Marek\Toggable\API\Security\UsernameAndPasswordToken;
use Marek\Toggable\Factory\FactoryInterface;
use InvalidArgumentException;

/**
 * Class AuthenticationFactory
 * @package Marek\Toggable\Factory\Authentication
 */
class AuthenticationFactory implements FactoryInterface
{
    /**
     * @var array
     */
    private $config;

    /**
     * AuthenticationFactory constructor.
     *
     * @param array $config
     */
    public function __construct(array $config)
    {
        if (empty($config['marek_toggable']['security'])) {
            throw new InvalidArgumentException('Please provide security configuration.');
        }
        $this->config = $config;
    }

    /**
     * @inheritdoc
     */
    public function build()
    {
        if (!empty($config['marek_toggable']['security']['token'])) {

            return new ApiToken($config['marek_toggable']['security']['token']);

        } else if (!empty($config['marek_toggable']['security']['username']) && !empty($config['marek_toggable']['security']['password'])) {

The variable $config seems to never exist, and therefore empty should always return true. Did you maybe rename this variable?

            return new UsernameAndPasswordToken(
                $config['marek_toggable']['security']['username'],
                $config['marek_toggable']['security']['password']
            );

        } else {

            throw new InvalidArgumentException('Please provide valid security configuration.');

        }
    }
}