Take care of null values for in_array and stripos

AltamashShaikh · AltamashShaikh · commit 19b180b3a8d6 · 2026-02-13T07:54:12.000+05:30
diff --git a/API.php b/API.php
@@ -57,7 +57,7 @@ public function getTreemapData(
         list($module, $method) = explode('.', $apiMethod);
         $disAllowedApiActions = ['getBulkRequest'];
         // Block if API action does not start with get
-        if (in_array($method, $disAllowedApiActions) || stripos($method, 'get') !== 0) {
+        if (!$method || in_array(strtolower($method), $disAllowedApiActions) || stripos($method, 'get') !== 0) {
             throw new BadRequestException(Piwik::translate('TreemapVisualization_InvalidApiMethodException'));
         }
 

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ public function getTreemapData(`
`57`	`57`	`list($module, $method) = explode('.', $apiMethod);`
`58`	`58`	`$disAllowedApiActions = ['getBulkRequest'];`
`59`	`59`	`// Block if API action does not start with get`
`60`		`- if (in_array($method, $disAllowedApiActions) \|\| stripos($method, 'get') !== 0) {`
	`60`	`+ if (!$method \|\| in_array(strtolower($method), $disAllowedApiActions) \|\| stripos($method, 'get') !== 0) {`
`61`	`61`	`throw new BadRequestException(Piwik::translate('TreemapVisualization_InvalidApiMethodException'));`
`62`	`62`	`}`
`63`	`63`