Apps\Controller\Admin\User\ActionUpdate::update()

Complexity

Conditions	3
Paths	3

Size

Total Lines	19
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function update(?string $id = null): ?string
    {
        $user = UserRecord::findOrNew($id);
        // generate model data based on user object
        $model = new FormUserUpdate($user);

It seems like $user can also be of type null; however, parameter $user of Apps\Model\Admin\User\FormUserUpdate::__construct() does only seem to accept Ffcms\Core\Interfaces\iUser, maybe add an additional type check?

        // check is form is sended
        if ($model->send()) {
            if ($model->validate()) { // check validation
                $model->save();
                App::$Session->getFlashBag()->add('success', __('Data was successful updated'));
            } else {
                App::$Session->getFlashBag()->add('error', __('Form validation is failed'));
            }
        }

        // render viewer
        return $this->view->render('user/user_update', [
            'model' => $model
        ]);