fix some warnings from modern PHP versions
authorRobert Kaiser <kairo@kairo.at>
Fri, 18 Feb 2022 22:43:09 +0000 (23:43 +0100)
committerRobert Kaiser <kairo@kairo.at>
Fri, 18 Feb 2022 22:43:09 +0000 (23:43 +0100)
classes/document.php-class
classes/useragent.php-class

index 198296f3e1a7eaa5ac8906ebec66f1cb3f2b5273..ed688b6e38b3c7dd50b38f41e68bc9ebdfeae3cf 100755 (executable)
@@ -439,7 +439,7 @@ class ExtendedDocument extends DOMDocument {
   public function createElement($name, $value = '') {
     // Adding the $value in DOMDocument's createElement does NOT escape it, so override it and use appendText to support that.
     $aelem = parent::createElement($name);
-    if (strlen($value)) { $aelem->appendText($value); }
+    if (strlen($value ?? '')) { $aelem->appendText($value); }
     return $aelem;
   }
 
index efda352738d2250cdf20e2cd1b324391bc95bc70..bebf5535a55a78e42b2a492f8bfc23133e991898 100755 (executable)
@@ -1003,12 +1003,17 @@ class userAgent {
   public function getAcceptLanguages() {
     if (!isset($this->uadata['accept-languages'])) {
       $headers = getAllHeaders();
-      $accLcomp = explode(',', @$headers['Accept-Language']);
+      $accLcomp = explode(',', ($headers['Accept-Language'] ?? ''));
       $accLang = array();
       foreach ($accLcomp as $lcomp) {
         if (strlen($lcomp)) {
           $ldef = explode(';', $lcomp);
-          $accLang[$ldef[0]] = (float)((strpos(@$ldef[1],'q=')===0)?substr($ldef[1],2):1);
+          if (count($ldef) > 1 && strpos($ldef[1], 'q=') === 0) {
+            $accLang[$ldef[0]] = substr($ldef[1], 2);
+          }
+          else {
+            $accLang[$ldef[0]] = 1;
+          }
         }
       }
       $this->uadata['accept-languages'] = $accLang;
@@ -1549,7 +1554,7 @@ class userAgent {
         if (strpos($this->uadata['os'], 'Windows Phone OS') !== false) { $this->mobile = true; }
         elseif (strpos($this->uadata['os'], 'Gameboy') !== false) { $this->mobile = true; }
 
-        $this->uadata['lang'] = str_replace('_', '-', @$this->uadata['lang']);
+        $this->uadata['lang'] = str_replace('_', '-', ($this->uadata['lang'] ?? ''));
       }
     }
   return $this->uadata['os'];