fix coding style

Author: sgiehl

Parser/AbstractParser.php

@@ -240,6 +240,7 @@ protected function getRegexesDirectory(): string
      * @param string $regex
      *
      * @return ?array
+     *
      * @throws \Exception
      */
     protected function matchUserAgent(string $regex)
@@ -253,7 +254,7 @@ protected function matchUserAgent(string $regex)
             if (preg_match($regex, $this->userAgent, $matches)) {
                 return $matches;
             }
-        } catch (\Exception $exception){
+        } catch (\Exception $exception) {
             throw new \Exception(
                 sprintf("%s\nRegex: %s", $exception->getMessage(), $regex),
                 $exception->getCode(),

Parser/Client/Browser.php

@@ -325,13 +325,13 @@ class Browser extends AbstractClientParser
             'UBrowser', 'Lovense Browser', 'AVG Secure Browser', 'TV Bro', 'Atom', '115 Browser',
             'Steam In-Game Overlay', 'Elements Browser', 'AOL Desktop', 'VMware AirWatch', 'Origin In-Game Overlay',
             'ToGate', 'Glass Browser', 'Sizzy', 'OhHai Browser', 'Yahoo! Japan Browser', 'Lulumi', 'Tungsten',
-            'Xvast', 'Colibri', 'Quark', 'Yaani Browser'
+            'Xvast', 'Colibri', 'Quark', 'Yaani Browser',
         ],
         'Firefox'            => [
             'Firefox', 'Fennec', 'Firefox Mobile', 'Swiftfox', 'Firebird', 'Phoenix', 'MicroB', 'Epic', 'Waterfox',
             'Cunaguaro', 'TenFourFox', 'Qwant Mobile', 'Firefox Rocket', 'IceCat', 'Mobicip', 'Firefox Mobile iOS',
             'Basilisk', 'Minimo', 'BlackHawk', 't-online.de Browser', 'Ordissimo', 'Mypal', 'Firefox Reality',
-            'Arctic Fox', 'Centaury', 'Light', 'Safe Exam Browser', 'Zvu'
+            'Arctic Fox', 'Centaury', 'Light', 'Safe Exam Browser', 'Zvu',
         ],
         'Internet Explorer'  => ['Internet Explorer', 'IE Mobile', 'Microsoft Edge'],
         'Konqueror'          => ['Konqueror'],
@@ -444,7 +444,11 @@ public function parse(): ?array
         }
 
         // This Exception should never be thrown. If so a defined browser name is missing in $availableBrowsers
-        throw new \Exception(sprintf('Detected browser name "%s" was not found in $availableBrowsers. Tried to parse user agent: %s', $name, $this->userAgent)); // @codeCoverageIgnore
+        throw new \Exception(sprintf(
+            'Detected browser name "%s" was not found in $availableBrowsers. Tried to parse user agent: %s',
+            $name,
+            $this->userAgent
+        )); // @codeCoverageIgnore
     }
 
     /**

Parser/Client/Browser/Engine.php

@@ -49,7 +49,7 @@ class Engine extends AbstractClientParser
         'Edge',
         'NetSurf',
         'Servo',
-        'Goanna'
+        'Goanna',
     ];
 
     /**
@@ -67,6 +67,7 @@ public static function getAvailableEngines(): array
     public function parse(): ?array
     {
         $matches = false;
+
         foreach ($this->getRegexes() as $regex) {
             $matches = $this->matchUserAgent($regex['regex']);
 
@@ -88,6 +89,9 @@ public function parse(): ?array
         }
 
         // This Exception should never be thrown. If so a defined browser name is missing in $availableEngines
-        throw new \Exception(sprintf('Detected browser engine was not found in $availableEngines. Tried to parse user agent: %s', $this->userAgent)); // @codeCoverageIgnore
+        throw new \Exception(sprintf(
+            'Detected browser engine was not found in $availableEngines. Tried to parse user agent: %s',
+            $this->userAgent
+        )); // @codeCoverageIgnore
     }
 }

Parser/Device/AbstractDeviceParser.php

@@ -866,7 +866,13 @@ public function parse(): ?array
 
             if (false === $brandId) {
                 // This Exception should never be thrown. If so a defined brand name is missing in $deviceBrands
-                throw new \Exception(sprintf("The brand with name '%s' should be listed in the deviceBrands array. Tried to parse user agent: %s", $brand, $this->userAgent)); // @codeCoverageIgnore
+                throw new \Exception(
+                    sprintf(
+                        "The brand name '%s' needs to be in deviceBrands array. Tried parsing user agent: %s",
+                        $brand,
+                        $this->userAgent
+                    )
+                ); // @codeCoverageIgnore
             }
 
             $this->brand = (string) $brandId;

Parser/OperatingSystem.php

@@ -145,7 +145,7 @@ class OperatingSystem extends AbstractParser
         'GNU/Linux'             => [
             'GNU/Linux', 'Arch Linux', 'Debian', 'Knoppix', 'Mint', 'Ubuntu', 'Kubuntu', 'Xubuntu', 'Lubuntu', 'Fedora',
             'Red Hat', 'VectorLinux', 'Mandriva', 'Gentoo', 'Sabayon', 'Slackware', 'SUSE', 'CentOS', 'BackTrack',
-            'Sailfish OS', 'Ordissimo', 'TmaxOS'
+            'Sailfish OS', 'Ordissimo', 'TmaxOS',
         ],
         'Mac'                   => ['Mac'],
         'Mobile Gaming Console' => ['PlayStation Portable', 'Nintendo Mobile', 'Xbox'],