TUrlManager: urldecode pathInfo before assigning to variables. Fixes usage of "array" parameters like ?ids[]=xxx

ctrlaltca · ctrlaltca · commit 44b1c15b106b · 2025-01-17T17:38:18.000+01:00
diff --git a/framework/Web/TUrlManager.php b/framework/Web/TUrlManager.php
@@ -124,7 +124,7 @@ public function constructUrl($serviceID, $serviceParam, $getItems, $encodeAmpers
 	public function parseUrl()
 	{
 		$request = $this->getRequest();
-		$pathInfo = trim($request->getPathInfo(), '/');
+		$pathInfo = urldecode(trim($request->getPathInfo(), '/'));
 		if (($request->getUrlFormat() === THttpRequestUrlFormat::Path ||
 			$request->getUrlFormat() === THttpRequestUrlFormat::HiddenPath) &&
 			$pathInfo !== '') {

Original file line number	Diff line number	Diff line change
`@@ -124,7 +124,7 @@ public function constructUrl($serviceID, $serviceParam, $getItems, $encodeAmpers`
`124`	`124`	`public function parseUrl()`
`125`	`125`	`{`
`126`	`126`	`$request = $this->getRequest();`
`127`		`- $pathInfo = trim($request->getPathInfo(), '/');`
	`127`	`+ $pathInfo = urldecode(trim($request->getPathInfo(), '/'));`
`128`	`128`	`if (($request->getUrlFormat() === THttpRequestUrlFormat::Path \|\|`
`129`	`129`	`$request->getUrlFormat() === THttpRequestUrlFormat::HiddenPath) &&`
`130`	`130`	`$pathInfo !== '') {`