Apps\Controller\Admin\Feedback\ActionTurn::turn()

Complexity

Conditions	7
Paths	6

Size

Total Lines	35
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function turn(string $direction, string $id): void
    {
        if (!Any::isInt($id) || $id < 1) {
            throw new NotFoundException('Bad is format');
        }
        // try to find record
        $record = FeedbackPost::find($id);
        if (!$record) {
            throw new NotFoundException(__('Feedback request with id %id% is not found', ['id' => $id]));
        }

        // switch operation direction to what we must change
        switch ($direction) {
            case 'open':
                $record->closed = 0;

The property $closed was declared of type boolean, but 0 is of type integer. Maybe add a type cast?

                $record->save();
                break;
            case 'close':
                $record->closed = 1;
                $record->save();
                break;
            case 'read':
                $record->readed = 1;

The property $readed was declared of type boolean, but 1 is of type integer. Maybe add a type cast?

                $record->save();
                break;
            default:
                throw new NotFoundException(__('Hack attention'));
                break;
        }

        // add notification of successful changes
        App::$Session->getFlashBag()->add('success', __('Feedback request is changed!'));

        // redirect to feedback post read
        $this->response->redirect('feedback/read/' . $id);
    }