Gewaer\Api\Controllers\IndexController::status()

Complexity

Conditions	8
Paths	528

Size

Total Lines	50
Code Lines	34

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	21.3931

Importance

Changes

0

<?php
    public function status() : Response
    {
        $response = [];

        //Try to connect to Redis
        try {
            $this->redis->hSet('htest', 'a', 'x');
            $this->redis->hSet('htest', 'b', 'y');
            $this->redis->hSet('htest', 'c', 'z');
            $this->redis->hSet('htest', 'd', 't');
            $this->redis->hGetAll('htest');

            //$this->redis->ping();
        } catch (RedisException $e) {
            $this->log->error($e->getMessage(), $e->getTrace());

The property log does not exist on Gewaer\Api\Controllers\IndexController. Since you implemented __get, consider adding a @property annotation.

            $response['errors']['redis'] = $e->getMessage();
        } catch (Exception $e) {
            $this->log->error("Redis isn't working. {$e->getMessage()}", $e->getTrace());
            $response['errors']['redis'] = "Redis isn't working.";
        }

        //Try to connect to Beanstalk
        try {
            $this->queue->connect();
        } catch (BeanstalkException $e) {
            $this->log->error($e->getMessage(), $e->getTrace());
            $response['errors']['beanstalk'] = $e->getMessage();
        } catch (Exception $e) {
            $this->log->error("Beanstalk isn't working. {$e->getMessage()}", $e->getTrace());
            $response['errors']['beanstalk'] = "Beanstalk isn't working.";
        } finally {
            $this->queue->disconnect();
        }

        //Try to connect to db
        try {
            $this->db->connect();
        } catch (PDOException $e) {
            $this->log->error($e->getMessage(), $e->getTrace());
            $response['errors']['db'] = $e->getMessage();
        } catch (Exception $e) {
            $this->log->error("The database isn't working. {$e->getMessage()}", $e->getTrace());
            $response['errors']['db'] = "The database isn't working.";
        }

        if (!count($response)) {
            return $this->response(['OK']);

The expression return $this->response(array('OK')) returns the type Phalcon\Http\Response which is incompatible with the documented return type Gewaer\Api\Controllers\Phalcon\Http\Response.

        }

        throw new ServerErrorHttpException(json_encode($response));
    }