Apps\Controller\Admin\Profile\ActionFieldUpdate::profileFieldUpdate()

Complexity

Conditions	4
Paths	3

Size

Total Lines	17
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function profileFieldUpdate($id = null)
    {
        // get current record or new and init form DI
        $record = ProfileField::findOrNew($id);
        $model = new FormFieldUpdate($record);

It seems like $record can also be of type null; however, parameter $record of Apps\Model\Admin\Profile\FormFieldUpdate::__construct() does only seem to accept Apps\ActiveRecord\ProfileField, maybe add an additional type check?

        // check if form is submitted
        if ($model->send() && $model->validate()) {
            $model->save();
            if ($record->id) {
                $this->response->redirect('profile/fieldlist');
            }
            App::$Session->getFlashBag()->add('success', __('Profile field was successful updated'));
        }

        return $this->view->render('profile/field_update', [
            'model' => $model
        ]);