1 | <?php |
||||||
2 | |||||||
3 | namespace Apps\Controller\Admin\Feedback; |
||||||
4 | |||||||
5 | use Apps\Model\Admin\Feedback\FormSettings; |
||||||
6 | use Ffcms\Core\App; |
||||||
7 | use Ffcms\Core\Arch\View; |
||||||
8 | use Ffcms\Core\Network\Request; |
||||||
9 | use Ffcms\Core\Network\Response; |
||||||
10 | |||||||
11 | /** |
||||||
12 | * Trait ActionSettings |
||||||
13 | * @package Apps\Controller\Admin\Feedback |
||||||
14 | * @property Request $request |
||||||
15 | * @property Response $response |
||||||
16 | * @property View $view |
||||||
17 | */ |
||||||
18 | trait ActionSettings |
||||||
19 | { |
||||||
20 | /** |
||||||
21 | * @return string|null |
||||||
22 | * @throws \Ffcms\Core\Exception\SyntaxException |
||||||
23 | */ |
||||||
24 | public function settings(): ?string |
||||||
25 | { |
||||||
26 | // initialize model and pass configs |
||||||
27 | $model = new FormSettings($this->getConfigs()); |
||||||
0 ignored issues
–
show
Bug
introduced
by
Loading history...
|
|||||||
28 | |||||||
29 | // check if form is submited to save data |
||||||
30 | if ($model->send()) { |
||||||
31 | // is validation passed? |
||||||
32 | if ($model->validate()) { |
||||||
33 | // save properties |
||||||
34 | $this->setConfigs($model->getAllProperties()); |
||||||
0 ignored issues
–
show
It seems like
setConfigs() must be provided by classes using this trait. How about adding it as abstract method to this trait?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the
Loading history...
|
|||||||
35 | App::$Session->getFlashBag()->add('success', __('Settings is successful updated')); |
||||||
36 | $this->response->redirect('feedback/index'); |
||||||
37 | } else { |
||||||
38 | App::$Session->getFlashBag()->add('error', __('Form validation is failed')); |
||||||
39 | } |
||||||
40 | } |
||||||
41 | |||||||
42 | // render view |
||||||
43 | return $this->view->render('feedback/settings', [ |
||||||
44 | 'model' => $model |
||||||
45 | ]); |
||||||
46 | } |
||||||
47 | } |
||||||
48 |