WyriHaximus\Travis\ApiSettings

Complexity

Total Complexity

2

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getOptions()	0	20	2

<?php declare(strict_types=1);

namespace WyriHaximus\Travis;

use ApiClients\Foundation\Hydrator\Options as HydratorOptions;
use ApiClients\Foundation\Options as FoundationOptions;
use ApiClients\Foundation\Transport\Middleware\JsonDecodeMiddleware;
use ApiClients\Foundation\Transport\Middleware\JsonEncodeMiddleware;
use ApiClients\Foundation\Transport\Options as TransportOptions;
use ApiClients\Foundation\Transport\UserAgentStrategies;
use WyriHaximus\Travis\Middleware\TokenAuthorizationHeaderMiddleware;

class ApiSettings
{
    /**
     * Travis' Pusher App ID as found on: https://docs.travis-ci.com/api?http#external-apis
     * Will automate the retrieval of that key later in the PR
     */
    const PUSHER_KEY = '5df8ac576dcccf4fd076';

    const NAMESPACE = 'WyriHaximus\\Travis\\Resource';

    const TRANSPORT_OPTIONS = [
        FoundationOptions::HYDRATOR_OPTIONS => [
            HydratorOptions::NAMESPACE => self::NAMESPACE,
            HydratorOptions::NAMESPACE_DIR => __DIR__ .
                DIRECTORY_SEPARATOR .
                'Resource' .
                DIRECTORY_SEPARATOR,
        ],
        FoundationOptions::TRANSPORT_OPTIONS => [
            TransportOptions::HOST => 'api.travis-ci.org',
            TransportOptions::HEADERS => [
                'Accept' => 'application/vnd.travis-ci.2+json',
            ],
            TransportOptions::USER_AGENT_STRATEGY => UserAgentStrategies::PACKAGE_VERSION,
            TransportOptions::PACKAGE => 'wyrihaximus/travis-client',
            TransportOptions::MIDDLEWARE => [
                JsonDecodeMiddleware::class,
                JsonEncodeMiddleware::class,
            ],
        ],
    ];

    /**
     * @param string $token
     * @param string $suffix
     * @return array
     */
    public static function getOptions(
        string $token,
        string $suffix
    ): array {
        $options = self::TRANSPORT_OPTIONS;
        $transportOptions[HydratorOptions::NAMESPACE_SUFFIX] = $suffix;

$transportOptions was never initialized. Although not strictly required by PHP, it is generally a good practice to add $transportOptions = array(); before regardless.

        if (!empty($token)) {
            $transportOptions[TransportOptions::MIDDLEWARE][] = TokenAuthorizationHeaderMiddleware::class;
            $transportOptions[TransportOptions::DEFAULT_REQUEST_OPTIONS] = [
                TokenAuthorizationHeaderMiddleware::class => [
                    Options::TOKEN => $token,
                ],
            ];
        }

        $options[FoundationOptions::HYDRATOR_OPTIONS] = $transportOptions;

        return $options;
    }
}