Merge pull request #2824 from Instrye/patch-setPath

fix. URI path is empty

Author: lonnieezell

system/HTTP/URI.php

@@ -710,7 +710,9 @@ public function setPath(string $path)
 	{
 		$this->path = $this->filterPath($path);
 
-		$this->segments = explode('/', $this->path);
+		$tempPath = trim($this->path, '/');
+
+		$this->segments = ($tempPath === '') ? [] : explode('/', $tempPath);
 
 		return $this;
 	}
@@ -726,7 +728,9 @@ public function refreshPath()
 	{
 		$this->path = $this->filterPath(implode('/', $this->segments));
 
-		$this->segments = explode('/', $this->path);
+		$tempPath = trim($this->path, '/');
+
+		$this->segments = ($tempPath === '') ? [] : explode('/', $tempPath);
 
 		return $this;
 	}
@@ -960,7 +964,9 @@ protected function applyParts(array $parts)
 		// Populate our segments array
 		if (isset($parts['path']) && $parts['path'] !== '')
 		{
-			$this->segments = explode('/', trim($parts['path'], '/'));
+			$tempPath = trim($parts['path'], '/');
+
+			$this->segments = ($tempPath === '') ? [] : explode('/', $tempPath);
 		}
 	}
 

tests/system/HTTP/URITest.php

@@ -848,4 +848,12 @@ public function testZeroAsURIPath()
 		$this->assertEquals('/0', $uri->getPath());
 	}
 
+	public function testEmptyURIPath()
+	{
+		$url = 'http://example.com/';
+		$uri = new URI($url);
+		$this->assertEquals([], $uri->getSegments());
+		$this->assertEquals(0, $uri->getTotalSegments());
+	}
+
 }