Zemit\Mvc\Controller\Rest\Response::setRestResponse()

Complexity

Conditions	11
Paths	99

Size

Total Lines	57
Code Lines	39

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	132

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function setRestResponse(mixed $response = null, int $code = null, string $status = null, int $jsonOptions = 0, int $depth = 512): ResponseInterface
    {
        $debug = $this->isDebugEnabled();

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

        
        // keep forced status code or set our own
        $statusCode = $this->response->getStatusCode();

The property response does not exist on Zemit\Mvc\Controller\Rest\Response. Did you mean response;?

        $reasonPhrase = $this->response->getReasonPhrase();
        $code ??= (int)$statusCode ?: 200;
        $status ??= $reasonPhrase ?: StatusCode::getMessage($code);
        
        $view = $this->view->getParamsToView();

The property view does not exist on Zemit\Mvc\Controller\Rest\Response. Did you mean view;?

        $hash = hash('sha512', json_encode($view)); // @todo store hash in cache layer with response content
        
        // set response status code
        $this->response->setStatusCode($code, $code . ' ' . $status);
        
        // @todo handle this correctly
        // @todo private vs public cache type
        $cache = $this->getCache();

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

        if (!empty($cache['lifetime'])) {
            if ($this->response->getStatusCode() === 200) {
                $this->response->setCache($cache['lifetime']);
                $this->response->setEtag($hash);
            }
        }
        else {
            $this->response->setCache(0);
            $this->response->setHeader('Cache-Control', 'no-cache, max-age=0');
        }
        
        $ret = [];
        $ret['api'] = [];
        $ret['api']['version'] = ['0.1']; // @todo
        $ret['timestamp'] = date('c');
        $ret['hash'] = $hash;
        $ret['status'] = $status;
        $ret['code'] = $code;
        $ret['response'] = $response;
        $ret['view'] = $view;
        
        if ($debug) {
            $ret['api']['php'] = phpversion();
            $ret['api']['phalcon'] = $this->config->path('phalcon.version');
            $ret['api']['zemit'] = $this->config->path('core.version');
            $ret['api']['core'] = $this->config->path('core.name');
            $ret['api']['app'] = $this->config->path('app.version');
            $ret['api']['name'] = $this->config->path('app.name');
            
            $ret['identity'] = $this->identity ? $this->identity->getIdentity() : null;

The property identity does not exist on Zemit\Mvc\Controller\Rest\Response. Did you mean identity;?

            $ret['profiler'] = $this->profiler ? $this->profiler->toArray() : null;
            $ret['request'] = $this->request ? $this->request->toArray() : null;

The property request does not exist on Zemit\Mvc\Controller\Rest\Response. Did you mean request;?

            $ret['dispatcher'] = $this->dispatcher ? $this->dispatcher->toArray() : null;

The property dispatcher does not exist on Zemit\Mvc\Controller\Rest\Response. Did you mean dispatcher;?

            $ret['router'] = $this->router ? $this->router->toArray() : null;

The property router does not exist on Zemit\Mvc\Controller\Rest\Response. Did you mean router;?

            $ret['memory'] = Utils::getMemoryUsage();
        }
        
        return $this->response->setJsonContent($ret, $jsonOptions, $depth);
    }