Replaced empty with strlen

cakephp · May 9, 2016 · a52a8d1 · a52a8d1
1 parent 0de5b1d
commit a52a8d1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/Network/Request.php b/src/Network/Request.php
@@ -547,7 +547,7 @@ public function referer($local = false)
         if (!empty($ref) && !empty($base)) {
             if ($local && strpos($ref, $base) === 0) {
                 $ref = substr($ref, strlen($base));
-                if (empty($ref)) {
+                if (!strlen($ref)) {
                     $ref = '/';
                 }
                 if ($ref[0] !== '/') {

diff --git a/tests/TestCase/Network/RequestTest.php b/tests/TestCase/Network/RequestTest.php
@@ -574,6 +574,10 @@ public function testReferer()
         $result = $request->referer(true);
         $this->assertSame('/some/path', $result);
 
+        $request->env('HTTP_REFERER', Configure::read('App.fullBaseUrl') . '/0');
+        $result = $request->referer(true);
+        $this->assertSame('/0', $result);
+
         $request->env('HTTP_REFERER', Configure::read('App.fullBaseUrl') . '/');
         $result = $request->referer(true);
         $this->assertSame('/', $result);