From d9817264b0bfff4fb6ca28e56f766231399c10f8 Mon Sep 17 00:00:00 2001
From: denisdulici <denisdulici@gmail.com>
Date: Mon, 6 Jan 2020 15:01:38 +0300
Subject: [PATCH] use route names

---
 app/Http/Controllers/Settings/Settings.php             | 2 +-
 modules/OfflinePayments/Listeners/ShowSetting.php      | 2 +-
 modules/OfflinePayments/Resources/views/edit.blade.php | 2 +-
 resources/views/settings/modules/edit.blade.php        | 2 +-
 routes/admin.php                                       | 4 ++--
 5 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/app/Http/Controllers/Settings/Settings.php b/app/Http/Controllers/Settings/Settings.php
index 9c5d205c0..034f13c01 100644
--- a/app/Http/Controllers/Settings/Settings.php
+++ b/app/Http/Controllers/Settings/Settings.php
@@ -43,7 +43,7 @@ class Settings extends Controller
             $modules->settings[$m->getAlias()] = [
                 'name' => $m->getName(),
                 'description' => $m->getDescription(),
-                'url' => $m->getAlias() . '/settings',
+                'url' => route('settings.module.edit', ['alias' => $m->getAlias()]),
                 'icon' => $m->get('icon', 'fa fa-cog'),
             ];
         }
diff --git a/modules/OfflinePayments/Listeners/ShowSetting.php b/modules/OfflinePayments/Listeners/ShowSetting.php
index b989f09c8..2a2c28e59 100644
--- a/modules/OfflinePayments/Listeners/ShowSetting.php
+++ b/modules/OfflinePayments/Listeners/ShowSetting.php
@@ -17,7 +17,7 @@ class ShowSetting
         $event->modules->settings['offline-payments'] = [
             'name' => trans('offline-payments::general.name'),
             'description' => trans('offline-payments::general.description'),
-            'url' => 'offline-payments/settings',
+            'url' => route('settings.module.edit', ['alias' => 'offline-payments']),
             'icon' => 'fas fa-credit-card',
         ];
     }
diff --git a/modules/OfflinePayments/Resources/views/edit.blade.php b/modules/OfflinePayments/Resources/views/edit.blade.php
index d7bdf623e..10f860fc5 100644
--- a/modules/OfflinePayments/Resources/views/edit.blade.php
+++ b/modules/OfflinePayments/Resources/views/edit.blade.php
@@ -12,7 +12,7 @@
 
                 {!! Form::open([
                     'id' => 'offline-payments',
-                    'route' => 'offline-payments.update',
+                    'route' => ['settings.module.update', 'offline-payments'],
                     '@submit.prevent' => 'onSubmit',
                     '@keydown' => 'form.errors.clear($event.target.name)',
                     'files' => true,
diff --git a/resources/views/settings/modules/edit.blade.php b/resources/views/settings/modules/edit.blade.php
index 15f3cd922..b5ceb0916 100644
--- a/resources/views/settings/modules/edit.blade.php
+++ b/resources/views/settings/modules/edit.blade.php
@@ -7,7 +7,7 @@
         {!! Form::model($setting, [
             'id' => 'module',
             'method' => 'PATCH',
-            'url' => [$module->getAlias() . '/settings'],
+            'route' => ['settings.module.update', $module->getAlias()],
             '@submit.prevent' => 'onSubmit',
             '@keydown' => 'form.errors.clear($event.target.name)',
             'files' => true,
diff --git a/routes/admin.php b/routes/admin.php
index 43007c8ac..c4e246c97 100644
--- a/routes/admin.php
+++ b/routes/admin.php
@@ -152,8 +152,8 @@ Route::group(['prefix' => 'settings'], function () {
 });
 
 Route::group(['as' => 'settings.'], function () {
-    Route::get('{alias}/settings', 'Settings\Modules@edit');
-    Route::patch('{alias}/settings', 'Settings\Modules@update');
+    Route::get('{alias}/settings', 'Settings\Modules@edit')->name('module.edit');
+    Route::patch('{alias}/settings', 'Settings\Modules@update')->name('module.update');
 });
 
 Route::group(['as' => 'apps.', 'prefix' => 'apps'], function () {