phpffcms / ffcms

Apps/Controller/Admin/User/ActionSettings.php

<?php

namespace Apps\Controller\Admin\User;


use Apps\Model\Admin\User\FormUserSettings;
use Ffcms\Core\App;
use Ffcms\Core\Arch\View;
use Ffcms\Core\Network\Request;
use Ffcms\Core\Network\Response;

/**
 * Trait ActionSettings
 * @package Apps\Controller\Admin\User
 * @property Request $request
 * @property Response $response
 * @property View $view
 */
trait ActionSettings
{
    /**
     * User app settings action
     * @return string|null
     * @throws \Ffcms\Core\Exception\SyntaxException
     */
    public function settings(): ?string
    {
        // load model and pass property's as argument
        $model = new FormUserSettings($this->getConfigs());

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

        if ($model->send()) {
            if ($model->validate()) {
                $this->setConfigs($model->getAllProperties());

Are you sure the usage of $model->getAllProperties() targeting Ffcms\Core\Arch\Model::getAllProperties() seems to always return null.

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

                App::$Session->getFlashBag()->add('success', __('Settings is successful updated'));
                $this->response->redirect('user/index');
            } else {
                App::$Session->getFlashBag()->add('error', __('Form validation is failed'));
            }
        }

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