SMartins\JsonHandler\CieloRequestHandler::cieloRequestException()

Complexity

Conditions	3
Paths	2

Size

Total Lines	21
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function cieloRequestException(CieloRequestException $e)
    {
        $errors = [];
        $code = $e->getCode();
        do {
            $cieloError = $e->getCieloError();
            $error = [
                'status'    => $e->getCode(),
                'code'      => $this->getCode('cielo').$cieloError->getCode(),

It seems like getCode() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                'source'    => ['pointer' => $e->getFile().':'.$e->getLine()],
                'title'     => $e->getMessage(),
                'detail'    => $cieloError->getMessage(),
            ];
            if (! in_array($error, $errors)) {
                array_push($errors, $error);
            }
            $e = $e->getPrevious();
        } while (method_exists($e, 'getPrevious'));

        $this->jsonApiResponse->setStatus($code);

The property jsonApiResponse does not exist on SMartins\JsonHandler\CieloRequestHandler. Did you maybe forget to declare it?

        $this->jsonApiResponse->setErrors($errors);
    }