Core23\LastFm\Service\AuthService

Complexity

Total Complexity

6

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	createSession()	0	12	2
A	createToken()	0	10	2
A	getAuthUrl()	0	4	1

<?php

/*
 * This file is part of the ni-ju-san CMS.
 *
 * (c) Christian Gripp <mail@core23.de>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Core23\LastFm\Service;

use Core23\LastFm\Connection\ConnectionInterface;
use Core23\LastFm\Connection\Session;
use Core23\LastFm\Connection\SessionInterface;
use Core23\LastFm\Exception\ApiException;
use Core23\LastFm\Exception\NotFoundException;

final class AuthService extends AbstractService
{
    /**
     * @var string
     */
    private $authUrl;

    /**
     * AuthService constructor.
     *
     * @param ConnectionInterface $connection
     * @param string              $authUrl
     */
    public function __construct(ConnectionInterface $connection, $authUrl = 'http://www.last.fm/api/auth/')
    {
        parent::__construct($connection);

        $this->authUrl = $authUrl;
    }


    /**
     * Creates a new session from a token.
     *
     * @param string $token
     *
     * @return SessionInterface|null
     *
     * @throws ApiException
     * @throws NotFoundException
     */
    public function createSession($token)
    {
        $response = $this->signedCall('auth.getSession', array(
            'token' => $token,
        ));

        if ($response) {

The expression $response of type array 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 new Session($response['session']['name'], $response['session']['key'], $response['session']['subscriber']);
        }

        return;
    }

    /**
     * Creates a new api token.
     *
     * @return string|false
     *
     * @throws ApiException
     * @throws NotFoundException
     */
    public function createToken()
    {
        $response = $this->signedCall('auth.getToken');

        if ($response) {

The expression $response of type array 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 $response['token'];
        }

        return false;
    }

    /**
     * Return the auth url.
     *
     * @param string $callbackUrl
     *
     * @return string
     */
    public function getAuthUrl($callbackUrl)
    {
        return $this->authUrl . '?api_key=' . $this->connection->getApiKey() . '&cb=' . $callbackUrl;
    }
}