X-Git-Url: https://git.openstreetmap.org./nominatim.git/blobdiff_plain/943e5fe6997f71709d344a73363021ffcefe3539..4ad88188095a3ce19c8fd6f5449d78b5e4aa82d4:/lib-php/ParameterParser.php?ds=sidebyside diff --git a/lib-php/ParameterParser.php b/lib-php/ParameterParser.php index 98b95388..a4936d37 100644 --- a/lib-php/ParameterParser.php +++ b/lib-php/ParameterParser.php @@ -22,7 +22,10 @@ class ParameterParser public function getBool($sName, $bDefault = false) { - if (!isset($this->aParams[$sName]) || strlen($this->aParams[$sName]) == 0) { + if (!isset($this->aParams[$sName]) + || !is_string($this->aParams[$sName]) + || strlen($this->aParams[$sName]) == 0 + ) { return $bDefault; } @@ -31,7 +34,7 @@ class ParameterParser public function getInt($sName, $bDefault = false) { - if (!isset($this->aParams[$sName])) { + if (!isset($this->aParams[$sName]) || is_array($this->aParams[$sName])) { return $bDefault; } @@ -44,7 +47,7 @@ class ParameterParser public function getFloat($sName, $bDefault = false) { - if (!isset($this->aParams[$sName])) { + if (!isset($this->aParams[$sName]) || is_array($this->aParams[$sName])) { return $bDefault; } @@ -57,7 +60,10 @@ class ParameterParser public function getString($sName, $bDefault = false) { - if (!isset($this->aParams[$sName]) || strlen($this->aParams[$sName]) == 0) { + if (!isset($this->aParams[$sName]) + || !is_string($this->aParams[$sName]) + || strlen($this->aParams[$sName]) == 0 + ) { return $bDefault; } @@ -66,11 +72,14 @@ class ParameterParser public function getSet($sName, $aValues, $sDefault = false) { - if (!isset($this->aParams[$sName]) || strlen($this->aParams[$sName]) == 0) { + if (!isset($this->aParams[$sName]) + || !is_string($this->aParams[$sName]) + || strlen($this->aParams[$sName]) == 0 + ) { return $sDefault; } - if (!in_array($this->aParams[$sName], $aValues)) { + if (!in_array($this->aParams[$sName], $aValues, true)) { userError("Parameter '$sName' must be one of: ".join(', ', $aValues)); }