Zemit\Mvc\Controller\Rest\Actions\IndexAction::restForwarding()

Complexity

Conditions	7
Paths	5

Size

Total Lines	22
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	56

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function restForwarding(): bool
    {
        $id = $this->getParam('id', [Filter::FILTER_ALNUM]);

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

        if ($this->request->isPost() || $this->request->isPut() || $this->request->isPatch()) {

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

            $this->dispatcher->forward(['action' => 'save']);

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

            return true;
        }
        else if ($this->request->isDelete()) {
            $this->dispatcher->forward(['action' => 'delete']);
            return true;
        }
        else if ($this->request->isGet()) {
            if (is_null($id)) {
                $this->dispatcher->forward(['action' => 'getList']);
                return true;
            }
            else {
                $this->dispatcher->forward(['action' => 'get']);
                return true;
            }
        }
        return false;
    }