diff --git a/Admin/Install/Navigation.install.json b/Admin/Install/Navigation.install.json index 834334b..ad9ed4f 100755 --- a/Admin/Install/Navigation.install.json +++ b/Admin/Install/Navigation.install.json @@ -19,7 +19,7 @@ "type": 2, "subtype": 1, "name": "Groups", - "uri": "{/base}/admin/group/list?{?}", + "uri": "{/base}/admin/group/list", "target": "self", "icon": null, "order": 2, @@ -33,7 +33,7 @@ "type": 3, "subtype": 1, "name": "List", - "uri": "{/base}/admin/group/list?{?}", + "uri": "{/base}/admin/group/list", "target": "self", "icon": null, "order": 1, @@ -48,7 +48,7 @@ "type": 3, "subtype": 1, "name": "Create", - "uri": "{/base}/admin/group/create?{?}", + "uri": "{/base}/admin/group/create", "target": "self", "icon": null, "order": 5, @@ -65,7 +65,7 @@ "type": 2, "subtype": 1, "name": "Accounts", - "uri": "{/base}/admin/account/list?{?}", + "uri": "{/base}/admin/account/list", "target": "self", "icon": null, "order": 4, @@ -79,7 +79,7 @@ "type": 3, "subtype": 1, "name": "List", - "uri": "{/base}/admin/account/list?{?}", + "uri": "{/base}/admin/account/list", "target": "self", "icon": null, "order": 1, @@ -94,7 +94,7 @@ "type": 3, "subtype": 1, "name": "Create", - "uri": "{/base}/admin/account/create?{?}", + "uri": "{/base}/admin/account/create", "target": "self", "icon": null, "order": 5, @@ -111,7 +111,7 @@ "type": 2, "subtype": 1, "name": "Modules", - "uri": "{/base}/admin/module/list?{?}", + "uri": "{/base}/admin/module/list", "target": "self", "icon": null, "order": 6, @@ -125,7 +125,7 @@ "type": 3, "subtype": 1, "name": "Info", - "uri": "{/base}/admin/module/info?{?}", + "uri": "{/base}/admin/module/info", "target": "self", "icon": null, "order": 1, @@ -140,7 +140,7 @@ "type": 3, "subtype": 1, "name": "Settings", - "uri": "{/base}/admin/module/settings?{?}", + "uri": "{/base}/admin/module/settings", "target": "self", "icon": null, "order": 5, @@ -155,7 +155,7 @@ "type": 3, "subtype": 1, "name": "Routes", - "uri": "{/base}/admin/module/route/list?{?}", + "uri": "{/base}/admin/module/route/list", "target": "self", "icon": null, "order": 15, @@ -170,7 +170,7 @@ "type": 3, "subtype": 1, "name": "Hooks", - "uri": "{/base}/admin/module/hook/list?{?}", + "uri": "{/base}/admin/module/hook/list", "target": "self", "icon": null, "order": 15, @@ -185,7 +185,7 @@ "type": 3, "subtype": 1, "name": "Log", - "uri": "{/base}/admin/module/log?{?}", + "uri": "{/base}/admin/module/log", "target": "self", "icon": null, "order": 20, diff --git a/Controller/ApiController.php b/Controller/ApiController.php index 69794fd..c038f27 100755 --- a/Controller/ApiController.php +++ b/Controller/ApiController.php @@ -1045,14 +1045,12 @@ final class ApiController extends Controller */ public function apiSettingsDesignSet(RequestAbstract $request, ResponseAbstract $response, array $data = []) : void { - $uploadedFiles = $request->files; - - if (!empty($uploadedFiles)) { + if (!empty($request->files)) { $upload = new UploadFile(); $upload->preserveFileName = false; $upload->outputDir = __DIR__ . '/../../../Web/Backend/img'; - $status = $upload->upload($uploadedFiles, ['logo.png'], true); + $status = $upload->upload($request->files, ['logo.png'], true); } $this->createStandardUpdateResponse($request, $response, []); diff --git a/Controller/BackendController.php b/Controller/BackendController.php index 77699da..b02af0b 100755 --- a/Controller/BackendController.php +++ b/Controller/BackendController.php @@ -24,7 +24,6 @@ use Modules\Admin\Models\ModuleMapper; use Modules\Admin\Models\SettingsEnum; use Modules\Auditor\Models\AuditMapper; use Modules\Organization\Models\UnitMapper; -use phpOMS\Asset\AssetType; use phpOMS\Autoloader; use phpOMS\Contract\RenderableInterface; use phpOMS\DataStorage\Database\Query\OrderType; diff --git a/Theme/Backend/modules-info.tpl.php b/Theme/Backend/modules-info.tpl.php index 3bb27db..f878a05 100755 --- a/Theme/Backend/modules-info.tpl.php +++ b/Theme/Backend/modules-info.tpl.php @@ -13,6 +13,7 @@ declare(strict_types=1); use Modules\Admin\Models\ModuleStatusUpdateType; +use phpOMS\Module\ModuleInfo; use phpOMS\Uri\UriFactory; /** @@ -23,31 +24,35 @@ $active = $this->data['active']; $installed = $this->data['installed']; $id = $this->data['id']; +$module = $modules[$id] ?? new ModuleInfo(''); + if (isset($installed[$id])) { echo $this->data['nav']->render(); } + +// @todo If no id is specified in the url this page looks horrible. Either clean up or return 404 page or something similar. ?>
| = $this->getHtml('Name'); ?> - | = $this->printHtml($modules[$id]->getExternalName()); ?> + | = $this->printHtml($module->getExternalName()); ?> |
| = $this->getHtml('Version'); ?> - | = $this->printHtml($modules[$id]->getVersion()); ?> + | = $this->printHtml($module->getVersion()); ?> |
| = $this->getHtml('CreatedBy'); ?> - | = $this->printHtml($modules[$id]->get()['creator']['name']); ?> + | = $this->printHtml($module->get()['creator']['name'] ?? ''); ?> |
| = $this->getHtml('Website'); ?> - | = $this->printHtml($modules[$id]->get()['creator']['website']); ?> + | = $this->printHtml($module->get()['creator']['website'] ?? ''); ?> |