@@ -32,78 +32,78 @@ |
||
| 32 | 32 | |
| 33 | 33 | class SettingsController extends Controller { |
| 34 | 34 | |
| 35 | - private $userId; |
|
| 36 | - private $appConfig; |
|
| 37 | - private $navigationManager; |
|
| 38 | - private $util; |
|
| 35 | + private $userId; |
|
| 36 | + private $appConfig; |
|
| 37 | + private $navigationManager; |
|
| 38 | + private $util; |
|
| 39 | 39 | |
| 40 | - public function __construct($appName, IRequest $request, ConfigService $appConfig, INavigationManager $urlGenerator, Util $util, $userId) { |
|
| 41 | - parent::__construct($appName, $request); |
|
| 42 | - $this->userId = $userId; |
|
| 43 | - $this->appConfig = $appConfig; |
|
| 44 | - $this->navigationManager = $urlGenerator; |
|
| 45 | - $this->util = $util; |
|
| 46 | - } |
|
| 40 | + public function __construct($appName, IRequest $request, ConfigService $appConfig, INavigationManager $urlGenerator, Util $util, $userId) { |
|
| 41 | + parent::__construct($appName, $request); |
|
| 42 | + $this->userId = $userId; |
|
| 43 | + $this->appConfig = $appConfig; |
|
| 44 | + $this->navigationManager = $urlGenerator; |
|
| 45 | + $this->util = $util; |
|
| 46 | + } |
|
| 47 | 47 | |
| 48 | - /** |
|
| 49 | - * Admin: render admin page |
|
| 50 | - * FIXME: Move to dedicated class |
|
| 51 | - * |
|
| 52 | - * @return TemplateResponse |
|
| 53 | - */ |
|
| 54 | - public function adminIndex() { |
|
| 55 | - // Private API call |
|
| 56 | - $navigation = $this->navigationManager->getAll(); |
|
| 57 | - $order = json_decode($this->appConfig->getAppValue('order')); |
|
| 58 | - $nav = $this->util->matchOrder($navigation, $order); |
|
| 59 | - return new TemplateResponse( |
|
| 60 | - $this->appName, |
|
| 61 | - 'admin', |
|
| 62 | - ["nav" => $nav], |
|
| 63 | - 'blank' |
|
| 64 | - ); |
|
| 65 | - } |
|
| 48 | + /** |
|
| 49 | + * Admin: render admin page |
|
| 50 | + * FIXME: Move to dedicated class |
|
| 51 | + * |
|
| 52 | + * @return TemplateResponse |
|
| 53 | + */ |
|
| 54 | + public function adminIndex() { |
|
| 55 | + // Private API call |
|
| 56 | + $navigation = $this->navigationManager->getAll(); |
|
| 57 | + $order = json_decode($this->appConfig->getAppValue('order')); |
|
| 58 | + $nav = $this->util->matchOrder($navigation, $order); |
|
| 59 | + return new TemplateResponse( |
|
| 60 | + $this->appName, |
|
| 61 | + 'admin', |
|
| 62 | + ["nav" => $nav], |
|
| 63 | + 'blank' |
|
| 64 | + ); |
|
| 65 | + } |
|
| 66 | 66 | |
| 67 | - /** |
|
| 68 | - * Admin: save default order |
|
| 69 | - * |
|
| 70 | - * @param $order |
|
| 71 | - * @return array response |
|
| 72 | - */ |
|
| 73 | - public function saveDefaultOrder($order) { |
|
| 74 | - if (!is_null($order)) { |
|
| 75 | - $this->appConfig->setAppValue('order', $order); |
|
| 76 | - } |
|
| 77 | - return array('status' => 'success', 'order' => $order); |
|
| 78 | - } |
|
| 67 | + /** |
|
| 68 | + * Admin: save default order |
|
| 69 | + * |
|
| 70 | + * @param $order |
|
| 71 | + * @return array response |
|
| 72 | + */ |
|
| 73 | + public function saveDefaultOrder($order) { |
|
| 74 | + if (!is_null($order)) { |
|
| 75 | + $this->appConfig->setAppValue('order', $order); |
|
| 76 | + } |
|
| 77 | + return array('status' => 'success', 'order' => $order); |
|
| 78 | + } |
|
| 79 | 79 | |
| 80 | - /** |
|
| 81 | - * Return order for current user |
|
| 82 | - * |
|
| 83 | - * @NoAdminRequired |
|
| 84 | - * @return array response |
|
| 85 | - */ |
|
| 86 | - public function getOrder() { |
|
| 87 | - $order = $this->util->getAppOrder(); |
|
| 88 | - return array('status' => 'success', 'order' => $order); |
|
| 89 | - } |
|
| 80 | + /** |
|
| 81 | + * Return order for current user |
|
| 82 | + * |
|
| 83 | + * @NoAdminRequired |
|
| 84 | + * @return array response |
|
| 85 | + */ |
|
| 86 | + public function getOrder() { |
|
| 87 | + $order = $this->util->getAppOrder(); |
|
| 88 | + return array('status' => 'success', 'order' => $order); |
|
| 89 | + } |
|
| 90 | 90 | |
| 91 | - /** |
|
| 92 | - * Save order for current user |
|
| 93 | - * |
|
| 94 | - * @NoAdminRequired |
|
| 95 | - * @param $order string |
|
| 96 | - * @return array response |
|
| 97 | - */ |
|
| 98 | - public function savePersonal($order) { |
|
| 99 | - $this->appConfig->setUserValue('order', $this->userId, $order); |
|
| 100 | - $response = array( |
|
| 101 | - 'status' => 'success', |
|
| 102 | - 'data' => array('message' => 'User order saved successfully.'), |
|
| 103 | - 'order' => $order |
|
| 104 | - ); |
|
| 105 | - return $response; |
|
| 106 | - } |
|
| 91 | + /** |
|
| 92 | + * Save order for current user |
|
| 93 | + * |
|
| 94 | + * @NoAdminRequired |
|
| 95 | + * @param $order string |
|
| 96 | + * @return array response |
|
| 97 | + */ |
|
| 98 | + public function savePersonal($order) { |
|
| 99 | + $this->appConfig->setUserValue('order', $this->userId, $order); |
|
| 100 | + $response = array( |
|
| 101 | + 'status' => 'success', |
|
| 102 | + 'data' => array('message' => 'User order saved successfully.'), |
|
| 103 | + 'order' => $order |
|
| 104 | + ); |
|
| 105 | + return $response; |
|
| 106 | + } |
|
| 107 | 107 | |
| 108 | 108 | |
| 109 | 109 | } |