Merge pull request #1350 from phalcon/fix/#1305-url-phalcon-serve

Jeckerson · web-flow · commit fbc578afcf04 · 2019-11-03T17:27:11.000Z
#1305 - Fix URL during phalcon serve
diff --git a/scripts/Phalcon/Web/Tools/Views/controllers/edit.volt b/scripts/Phalcon/Web/Tools/Views/controllers/edit.volt
@@ -3,7 +3,7 @@
         {{ content() }}
         {{ flashSession.output() }}
         <div class="box box-success">
-            <form role="form" name="edit-controller" method="post" action="{{ url.get("/webtools.php?_url=/controllers/update") }}">
+            <form role="form" name="edit-controller" method="post" action="{{ url.get(webtools_uri ~ "?_url=/controllers/update") }}">
                 <div class="box-header with-border">
                     <p class="pull-left">{{ controller_name }} - [{{ controller_path }}]</p>
                     {{ submit_button("Save", "class": "btn btn-success pull-right") }}
diff --git a/scripts/Phalcon/Web/Tools/Views/controllers/generate.volt b/scripts/Phalcon/Web/Tools/Views/controllers/generate.volt
@@ -6,7 +6,7 @@
         {{ flashSession.output() }}
 
         <div class="box box-success">
-            <form role="form" class="form-horizontal" name="generate-controller" method="post" action="{{ url.get("/webtools.php?_url=/controllers/generate") }}">
+            <form role="form" class="form-horizontal" name="generate-controller" method="post" action="{{ url.get(webtools_uri ~ "?_url=/controllers/generate") }}">
                 <div class="box-header with-border">
                     <p class="pull-left">{{ controller_name }} - [{{ controller_path }}]</p>
                     {{ submit_button("Generate", "class": "btn btn-success pull-right") }}
diff --git a/scripts/Phalcon/Web/Tools/Views/controllers/index.volt b/scripts/Phalcon/Web/Tools/Views/controllers/index.volt
@@ -5,7 +5,7 @@
         <div class="box">
             <div class="box-header">
                 <h3 class="box-title">Controllers List</h3>
-                {{ link_to("/webtools.php?_url=/controllers/generate", "Generate", 'class': 'btn btn-primary pull-right') }}
+                {{ link_to(webtools_uri ~ "?_url=/controllers/generate", "Generate", 'class': 'btn btn-primary pull-right') }}
             </div>
             <div class="box-body table-responsive no-padding">
                 <table class="table table-hover">
@@ -41,7 +41,7 @@
                                 <td>{{ controller.owner }}</td>
                                 <td>{{ controller.modified_time }}</td>
                                 <td>
-                                    {{ link_to("/webtools.php?_url=/controllers/edit/" ~ rawurlencode(controller.filename),
+                                    {{ link_to(webtools_uri ~ "?_url=/controllers/edit/" ~ rawurlencode(controller.filename),
                                     '<i class="fa fa-pencil"></i>', 'class': 'btn btn-default btn-xs') }}
                                 </td>
                             </tr>
diff --git a/scripts/Phalcon/Web/Tools/Views/migrations/generate.volt b/scripts/Phalcon/Web/Tools/Views/migrations/generate.volt
@@ -6,7 +6,7 @@
         {{ flashSession.output() }}
 
         <div class="box box-success">
-            <form role="form" class="form-horizontal" name="generate-migration" method="post" action="{{ url.get("/webtools.php?_url=/migrations/generate") }}">
+            <form role="form" class="form-horizontal" name="generate-migration" method="post" action="{{ url.get(webtools_uri ~ "?_url=/migrations/generate") }}">
                 <div class="box-header with-border">
                     <p class="pull-left">New model will be placed at: [{{ migration_path }}]</p>
                     {{ submit_button("Generate", "class": "btn btn-success pull-right") }}
diff --git a/scripts/Phalcon/Web/Tools/Views/migrations/index.volt b/scripts/Phalcon/Web/Tools/Views/migrations/index.volt
@@ -5,7 +5,7 @@
         <div class="box">
             <div class="box-header">
                 <h3 class="box-title">Migrations List</h3>
-                {{ link_to("/webtools.php?_url=/migrations/generate", "Generate", 'class': 'btn btn-primary pull-right') }}
+                {{ link_to(webtools_uri ~ "?_url=/migrations/generate", "Generate", 'class': 'btn btn-primary pull-right') }}
             </div>
             <div class="box-body table-responsive no-padding">
                 <table class="table table-hover table-bordered">
diff --git a/scripts/Phalcon/Web/Tools/Views/migrations/run.volt b/scripts/Phalcon/Web/Tools/Views/migrations/run.volt
@@ -6,7 +6,7 @@
         {{ flashSession.output() }}
 
         <div class="box box-success">
-            <form role="form" class="form-horizontal" name="generate-migration" method="post" action="{{ url.get("/webtools.php?_url=/migrations/run") }}">
+            <form role="form" class="form-horizontal" name="generate-migration" method="post" action="{{ url.get(webtools_uri ~ "?_url=/migrations/run") }}">
                 <div class="box-header with-border">
                     <p class="pull-left">Migrations from: [{{ migration_path }}]</p>
                     {{ submit_button("Run", "class": "btn btn-success pull-right") }}
diff --git a/scripts/Phalcon/Web/Tools/Views/models/edit.volt b/scripts/Phalcon/Web/Tools/Views/models/edit.volt
@@ -3,7 +3,7 @@
         {{ content() }}
         {{ flashSession.output() }}
         <div class="box box-success">
-            <form role="form" name="edit-model" method="post" action="{{ url.get("/webtools.php?_url=/models/update") }}">
+            <form role="form" name="edit-model" method="post" action="{{ url.get(webtools_uri ~ "?_url=/models/update") }}">
                 <div class="box-header with-border">
                     <p class="pull-left">{{ model_name }} - [{{ model_path }}]</p>
                     {{ submit_button("Save", "class": "btn btn-success pull-right") }}
diff --git a/scripts/Phalcon/Web/Tools/Views/models/generate.volt b/scripts/Phalcon/Web/Tools/Views/models/generate.volt
@@ -6,7 +6,7 @@
         {{ flashSession.output() }}
 
         <div class="box box-success">
-            <form role="form" class="form-horizontal" name="generate-model" method="post" action="{{ url.get("/webtools.php?_url=/models/generate") }}">
+            <form role="form" class="form-horizontal" name="generate-model" method="post" action="{{ url.get(webtools_uri ~ "?_url=/models/generate") }}">
                 <div class="box-header with-border">
                     <p class="pull-left">New model will be placed at: [{{ model_path }}]</p>
                     {{ submit_button("Generate", "class": "btn btn-success pull-right") }}
diff --git a/scripts/Phalcon/Web/Tools/Views/models/index.volt b/scripts/Phalcon/Web/Tools/Views/models/index.volt
@@ -5,7 +5,7 @@
         <div class="box">
             <div class="box-header">
                 <h3 class="box-title">Models List</h3>
-                {{ link_to("/webtools.php?_url=/models/generate", "Generate", 'class': 'btn btn-primary pull-right') }}
+                {{ link_to(webtools_uri ~ "?_url=/models/generate", "Generate", 'class': 'btn btn-primary pull-right') }}
             </div>
             <div class="box-body table-responsive no-padding">
                 <table class="table table-hover">
@@ -41,7 +41,7 @@
                                 <td>{{ model.owner }}</td>
                                 <td>{{ model.modified_time }}</td>
                                 <td>
-                                    {{ link_to("/webtools.php?_url=/models/edit/" ~ rawurlencode(model.filename),
+                                    {{ link_to(webtools_uri ~ "?_url=/models/edit/" ~ rawurlencode(model.filename),
                                     '<i class="fa fa-pencil"></i>', 'class': 'btn btn-default btn-xs') }}
                                 </td>
                             </tr>
diff --git a/scripts/Phalcon/Web/Tools/Views/scaffold/generate.volt b/scripts/Phalcon/Web/Tools/Views/scaffold/generate.volt
@@ -6,7 +6,7 @@
         {{ flashSession.output() }}
 
         <div class="box box-success">
-            <form role="form" class="form-horizontal" name="generate-scaffold" method="post" action="{{ url.get("/webtools.php?_url=/scaffold/generate") }}">
+            <form role="form" class="form-horizontal" name="generate-scaffold" method="post" action="{{ url.get(webtools_uri ~ "?_url=/scaffold/generate") }}">
                 <div class="box-header with-border">
                     <p class="pull-left">We will use templates from: [{{ template_path }}]</p>
                     {{ submit_button("Generate", "class": "btn btn-success pull-right") }}
diff --git a/templates/project/micro/config.ini b/templates/project/micro/config.ini
@@ -9,4 +9,4 @@ dbname   = test
 modelsDir     = ../app/models/
 migrationsDir = ../app/migrations/
 viewsDir      = ../app/views/
-baseUri       = /@@name@@/
+baseUri       = /
diff --git a/templates/project/micro/config.php b/templates/project/micro/config.php
@@ -20,6 +20,6 @@
         'modelsDir'      => APP_PATH . '/models/',
         'migrationsDir'  => APP_PATH . '/migrations/',
         'viewsDir'       => APP_PATH . '/views/',
-        'baseUri'        => '/@@name@@/',
+        'baseUri'        => '/',
     ]
 ]);
diff --git a/templates/project/micro/services.php b/templates/project/micro/services.php
@@ -56,3 +56,14 @@
     return $connection;
 });
 
+/**
+ * Register router
+ */
+$di->setShared('router', function () {
+    $router = new \Phalcon\Mvc\Router();
+    $router->setUriSource(
+        \Phalcon\Mvc\Router::URI_SOURCE_SERVER_REQUEST_URI
+    );
+
+    return $router;
+});
diff --git a/templates/project/modules/config.ini b/templates/project/modules/config.ini
@@ -12,4 +12,4 @@ modelsDir      = ../apps/frontend/models/
 migrationsDir  = ../apps/migrations/
 cacheDir       = ../../cache/
 viewsDir       = ../apps/frontend/views/
-baseUri        = /@@name@@/
+baseUri        = /
diff --git a/templates/project/modules/config.php b/templates/project/modules/config.php
@@ -23,11 +23,7 @@
         'modelsDir'      => APP_PATH . '/common/models/',
         'migrationsDir'  => APP_PATH . '/migrations/',
         'cacheDir'       => BASE_PATH . '/cache/',
-
-        // This allows the baseUri to be understand project paths that are not in the root directory
-        // of the webpspace.  This will break if the public/index.php entry point is moved or
-        // possibly if the web server rewrite rules are changed. This can also be set to a static path.
-        'baseUri'        => preg_replace('/public([\/\\\\])index.php$/', '', $_SERVER["PHP_SELF"]),
+        'baseUri'        => '/',
     ],
 
     /**
diff --git a/templates/project/modules/services_web.php b/templates/project/modules/services_web.php
@@ -15,6 +15,9 @@
     $router = new Router();
 
     $router->setDefaultModule('frontend');
+    $router->setUriSource(
+        Router::URI_SOURCE_SERVER_REQUEST_URI
+    );
 
     return $router;
 });
diff --git a/templates/project/simple/config.ini b/templates/project/simple/config.ini
@@ -14,7 +14,7 @@ viewsDir       = ../app/views/
 pluginsDir     = ../app/plugins/
 libraryDir     = ../app/library/
 cacheDir       = ../cache/
-baseUri        = /@@name@@/
+baseUri        = /
 
 [models]
 metadata.adapter = "Memory"
diff --git a/templates/project/simple/config.php b/templates/project/simple/config.php
@@ -24,10 +24,6 @@
         'pluginsDir'     => APP_PATH . '/plugins/',
         'libraryDir'     => APP_PATH . '/library/',
         'cacheDir'       => BASE_PATH . '/cache/',
-
-        // This allows the baseUri to be understand project paths that are not in the root directory
-        // of the webpspace.  This will break if the public/index.php entry point is moved or
-        // possibly if the web server rewrite rules are changed. This can also be set to a static path.
-        'baseUri'        => preg_replace('/public([\/\\\\])index.php$/', '', $_SERVER["PHP_SELF"]),
+        'baseUri'        => '/',
     ]
 ]);
diff --git a/templates/project/simple/services.php b/templates/project/simple/services.php
@@ -1,5 +1,6 @@
 <?php
 
+use Phalcon\Mvc\Router;
 use Phalcon\Mvc\View;
 use Phalcon\Mvc\View\Engine\Php as PhpEngine;
 use Phalcon\Mvc\Url as UrlResolver;
@@ -110,3 +111,15 @@
 
     return $session;
 });
+
+/**
+ * Register router
+ */
+$di->setShared('router', function () {
+    $router = new Router();
+    $router->setUriSource(
+        Router::URI_SOURCE_SERVER_REQUEST_URI
+    );
+
+    return $router;
+});
