X-Git-Url: https://git.openstreetmap.org./nominatim.git/blobdiff_plain/6be2077d922098d65f069557dea55542c15a5c98..4ad88188095a3ce19c8fd6f5449d78b5e4aa82d4:/lib-php/ParameterParser.php?ds=sidebyside diff --git a/lib-php/ParameterParser.php b/lib-php/ParameterParser.php index 90e0cee3..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)); } @@ -114,11 +123,10 @@ class ParameterParser } foreach ($aLanguages as $sLanguage => $fLanguagePref) { - $this->addNameTag($aLangPrefOrder, $sLanguage); + $this->addNameTag($aLangPrefOrder, 'name:'.$sLanguage); } - $this->addNameTag($aLangPrefOrder, 'default'); + $this->addNameTag($aLangPrefOrder, 'name'); $this->addNameTag($aLangPrefOrder, 'brand'); - foreach ($aLanguages as $sLanguage => $fLanguagePref) { $this->addNameTag($aLangPrefOrder, 'official_name:'.$sLanguage); $this->addNameTag($aLangPrefOrder, 'short_name:'.$sLanguage);