Zemit\Mvc\Controller\Rest\Actions\SaveAction::saveAction()

Complexity

Conditions	3
Paths	3

Size

Total Lines	17
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function saveAction(?int $id = null): ResponseInterface
    {
        $ret = $this->save($id);

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

        $this->view->setVars($ret);

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

        $saved = $this->saveResultHasKey($ret, 'saved');
        $messages = $this->saveResultHasKey($ret, 'messages');
        
        if (!$saved) {
            if (!$messages) {
                $this->response->setStatusCode(422, 'Unprocessable Entity');

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

            }
            else {
                $this->response->setStatusCode(400, 'Bad Request');
            }
        }
        
        return $this->setRestResponse($saved);
    }