Try to avoid using site url which ends with index.php #101

prinsss · Nov 7, 2017 · 0f065ef · 0f065ef
1 parent 788a30a
commit 0f065ef
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 3 deletions.
diff --git a/app/Http/Controllers/AdminController.php b/app/Http/Controllers/AdminController.php
@@ -108,8 +108,14 @@ public function options()
             $form->text('site_url')
                 ->hint()
                 ->format(function ($url) {
-                    if (ends_with($url, '/'))
+                    if (ends_with($url, '/')) {
                         $url = substr($url, 0, -1);
+                    }
+
+                    if (ends_with($url, '/index.php')) {
+                        $url = substr($url, 0, -10);
+                    }
+
                     return $url;
                 });
 

diff --git a/app/Http/Controllers/SetupController.php b/app/Http/Controllers/SetupController.php
@@ -57,7 +57,14 @@ public function finish(Request $request)
         Log::info("[SetupWizard] Tables migrated.");
 
         Option::set('site_name', $request->input('site_name'));
-        Option::set('site_url',  url('/'));
+
+        $siteUrl = url('/');
+
+        if (ends_with($siteUrl, '/index.php')) {
+            $siteUrl = substr($siteUrl, 0, -10);
+        }
+
+        Option::set('site_url',  $siteUrl);
 
         // register super admin
         $user = User::register(
@@ -137,7 +144,7 @@ public function doUpdate()
             Artisan::call('view:clear');
         } catch (\Exception $e) {
             Log::error('Error occured when processing view:clear', $e);
-            
+
             File::cleanDirectory(storage_path('framework/views'));
         }