Séparation des actions de migration dans un contrôleur dédié.

soosyze · Feb 13, 2021 · e18aefc · e18aefc
1 parent d07ea7e
commit e18aefc
Show file tree

Hide file tree

Showing 4 changed files with 38 additions and 2 deletions.
diff --git a/core/modules/System/Config/routes.php b/core/modules/System/Config/routes.php
@@ -7,6 +7,9 @@
 R::get('system.module.edit', 'admin/modules', 'ModulesManager@edit');
 R::post('system.module.update', 'admin/modules', 'ModulesManager@update');
 
+R::get('system.migration.check', 'admin/migration/check', 'ModulesMigration@check');
+R::get('system.migration.update', 'admin/migration/update', 'ModulesMigration@update');
+
 R::get('system.theme.index', 'admin/theme', 'Theme@index');
 R::get('system.theme.admin', 'admin/theme/:type', 'Theme@admin', [ ':type' => 'admin|public' ]);
 R::get('system.theme.active', 'admin/theme/:type/active/:name', 'Theme@active', [ ':type' => 'admin|public', ':name' => '\w+' ]);

diff --git a/core/modules/System/Config/services.php b/core/modules/System/Config/services.php
@@ -44,6 +44,8 @@
             'user.permission.module' => 'hookUserPermissionModule',
             'route.system.module.edit' => 'hookModuleManage',
             'route.system.module.update' => 'hookModuleManage',
+            'route.system.migration.check' => 'hookModuleManage',
+            'route.system.migration.update' => 'hookModuleManage',
             'route.system.theme.index' => 'hookThemeManage',
             'route.system.theme.admin' => 'hookThemeManage',
             'route.system.theme.active' => 'hookThemeManage',

diff --git a/core/modules/System/Controller/ModulesManager.php b/core/modules/System/Controller/ModulesManager.php
@@ -96,8 +96,8 @@ public function edit()
                 ->view('page.messages', $messages)
                 ->make('page.content', 'system/content-modules_manager-admin.php', $this->pathViews, [
                     'module_update'      => self::config()->get('settings.module_update'),
-                    'link_module_check'  => self::router()->getRoute('system.module.check'),
-                    'link_module_update' => self::router()->getRoute('system.module.updater'),
+                    'link_module_check'  => self::router()->getRoute('system.migration.check'),
+                    'link_module_update' => self::router()->getRoute('system.migration.update'),
                     'count'             => count($composer),
                     'form'              => $form,
                     'packages'          => $packages

diff --git a/core/modules/System/Controller/ModulesMigration.php b/core/modules/System/Controller/ModulesMigration.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace SoosyzeCore\System\Controller;
+
+use Soosyze\Components\Http\Redirect;
+
+class ModulesMigration extends \Soosyze\Controller
+{
+    public function check()
+    {
+        if (self::migration()->isMigration()) {
+            self::config()->set('settings.module_update', true);
+        } else {
+            $_SESSION[ 'messages' ][ 'success' ] = [ t('Your site is up to date') ];
+        }
+
+        return new Redirect(self::router()->getRoute('system.module.edit'), 302);
+    }
+
+    public function update()
+    {
+        try {
+            self::migration()->migrate();
+            $_SESSION[ 'messages' ][ 'success' ] = [ t('The update is a success') ];
+        } catch (\Exception $e) {
+            $_SESSION[ 'messages' ][ 'error' ] = [ t('An error occurred during the update') ];
+        }
+
+        return new Redirect(self::router()->getRoute('system.module.edit'), 302);
+    }
+}