Merge pull request #17 from skipperbent/development

Development
[BUGFIX] Fixed router paths.
2026-06-22 11:09:16 +00:00 · 2015-10-27 11:06:36 +01:00 · 2015-10-27 10:35:57 +01:00 · 2015-10-27 10:29:40 +01:00 · 2015-10-25 15:31:15 +01:00 · 2015-10-25 15:29:14 +01:00
5 changed files with 9 additions and 7 deletions
@@ -20,7 +20,7 @@
    },
    "autoload": {
        "psr-4": {
-            "Pecee\\": "src/"
+            "Pecee\\": "src/Pecee/"
        }
    }
 }
@@ -10,7 +10,7 @@ class Request {

    public function __construct() {
        $this->host = $_SERVER['HTTP_HOST'];
-        $this->uri = rtrim($_SERVER['REQUEST_URI'], '/') . '/';
+        $this->uri = $_SERVER['REQUEST_URI'];
        $this->method = (isset($_POST['_method'])) ? strtolower($_POST['_method']) : strtolower($_SERVER['REQUEST_METHOD']);
        $this->headers = getallheaders();
    }
@@ -54,12 +54,11 @@ class Response {
    /**
     * Json encode array
     * @param array $value
-     * @return self $this
     */
    public function json(array $value) {
        $this->header('Content-type: application/json');
        echo json_encode($value);
-        return $this;
+        die();
    }

    /**
@@ -102,6 +102,9 @@ class RouterBase {

        // Make sure the urls is in the right order when comparing
        usort($this->controllerUrlMap, function($a, $b) {
+            if(stripos($b->getUrl(), '{') !== false) {
+                return $a->getUrl();
+            }
            return strcmp($b->getUrl(), $a->getUrl());
        });

@@ -19,6 +19,7 @@ class RouterRoute extends RouterEntry {
    }

    protected function parseParameters($url, $multiple = false, $regex = self::PARAMETERS_REGEX_MATCH) {
+        $url = rtrim($url, '/');
        $parameters = array();

        if($multiple) {
@@ -90,8 +91,8 @@ class RouterRoute extends RouterEntry {
                            }
                        }

-                        // Add parameter value
-                        $parameters[$parameter] = $parameterValue;
+                        // Add parameter value, if it doesn't exist - replace it with null value
+                        $parameters[$parameter] = ($parameterValue === '') ? null : $parameterValue;
                    }
                }
            }
@@ -160,5 +161,4 @@ class RouterRoute extends RouterEntry {
        return parent::setSettings($settings);
    }

-
 }
Author	SHA1	Message	Date
Simon Sessingø	af730e6e15	Merge pull request #17 from skipperbent/development Development	2015-10-27 11:06:36 +01:00
Simon Sessingø	f5a32cf520	[BUGFIX] Fixed router paths.	2015-10-27 10:35:57 +01:00
Simon Sessingø	1843ea0594	[BUGFIX] Fixed routes always matching /show/{id} when other routes are added.	2015-10-27 10:29:40 +01:00
Simon Sessingø	fd28f4549f	Merge pull request #16 from skipperbent/development [BUGFIX] Bugfixes	2015-10-25 15:31:15 +01:00
Simon Sessingø	5e5a424ee8	[BUGFIX] Bugfixes - getUri() method in Request class no longer appends "/". - Optimised parseParameters method in RouterRoute class for regex matches.	2015-10-25 15:29:14 +01:00
Simon Sessingø	099f04fc10	Merge pull request #15 from skipperbent/development [BUGFIX] Bugfixes and optimisations	2015-10-25 02:35:49 +02:00
Simon Sessingø	ac2993f804	[BUGFIX] Bugfixes and optimisations - When no parameter is recieved, router now returns null instead of empty string. - json method in Response class now stops response after execution. - Updated composer.json with correct autoload.	2015-10-25 02:34:25 +02:00