juniorb2ss\DeathByCaptcha\Abstracts\HttpHandlerResponseAbstract

Complexity

Total Complexity

10

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
D	handlerHttpResponseIsOk()	0	42	10

<?php

namespace juniorb2ss\DeathByCaptcha\Abstracts;

use Psr\Http\Message\ResponseInterface;
use juniorb2ss\DeathByCaptcha\Exceptions\AccessDeniedException;
use juniorb2ss\DeathByCaptcha\Exceptions\CaptchaNotFoundException;
use juniorb2ss\DeathByCaptcha\Exceptions\InternalServiceException;
use juniorb2ss\DeathByCaptcha\Exceptions\InvalidCaptchaException;
use juniorb2ss\DeathByCaptcha\Exceptions\InvalidServiceResponseException;
use juniorb2ss\DeathByCaptcha\Exceptions\ServiceOverloadException;

abstract class HttpHandlerResponseAbstract
{

    /**
     * Define o content que deverá ser a resposta
     * do serviço
     * @var string
     */
    protected $contentTypeOk = 'application/json; charset=utf8';

    /**
     * Executa verificação da integridade da resposta
     * desde do cabeçalho do response até o conteúdo
     * @return void|ClientException
     */
    protected function handlerHttpResponseIsOk()
    {
        $response = $this->response;

The property response does not exist. Did you maybe forget to declare it?

        $contentType = $response->getHeaderLine('Content-Type');

        $statusCode = $response->getStatusCode();

        // Efetua o tratamento do status code do response
        switch ($statusCode) {
            case 400:
                throw new InvalidCaptchaException;
                break;

break; does not seem to be reachable.

            case 403:
                throw new AccessDeniedException;
                break;

break; does not seem to be reachable.

            case 404:
                throw new CaptchaNotFoundException;
                break;

break; does not seem to be reachable.

            case 413:
                throw new InvalidCaptchaException;
                break;

break; does not seem to be reachable.

            case 500:
                throw new InternalServiceException;
            case 501:
                throw new InvalidCaptchaException;
                break;

break; does not seem to be reachable.

            case 503:
                throw new ServiceOverloadException;
                break;

break; does not seem to be reachable.

        }

        // Uma coisa básica é verificar o tipo do conteúdo
        // da resposta HTTP.
        // Parece a principio algo meio "desnecessário"
        // já que se a resposta for ok e for passado o
        // accept de json, técnicamente deverá retornar
        // o content type como json
        // mas... nunca se sabe, né?
        if (($contentType !== $this->contentTypeOk) or (empty((string)$response->getBody()))) {

Using logical operators such as or instead of || is generally not recommended.

            throw new InvalidServiceResponseException;
        }
    }
}