endorphin-studio / browser-detector

src/Device.php

Braces +1 added lines, -2 removed lines

@@ -25,8 +25,7 @@
         {
             $this->Name = 'Desktop';
             $this->Type = 'desktop';
-        }
-        else
+        } else
         {
             parent::__construct($xmlData);
 

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
      */
     public function __construct(\SimpleXMLElement $xmlData)
     {
-        if($xmlData === null || $xmlData->getName() == 'null')
+        if ($xmlData === null || $xmlData->getName() == 'null')
         {
             parent::__construct($xmlData);
 

src/OS.php

Spacing +4 added lines, -4 removed lines

@@ -22,7 +22,7 @@ 
     public function __construct(\SimpleXMLElement $xmlData)
     {
         parent::__construct($xmlData);
-        if($xmlData !== null)
+        if ($xmlData !== null)
         {
             foreach ($xmlData->children() as $child) {
                 switch ($child->getName()) {
@@ -46,6 +46,6 @@ 
 
 }
 
-define('UNX','unix');
-define('WIN','windows');
-define('MC','mac');
+define('UNX', 'unix');
+define('WIN', 'windows');
+define('MC', 'mac');

src/Data.php

Spacing +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
     {
         parent::__construct($xmlData);
 
-        if($xmlData !== null)
+        if ($xmlData !== null)
         {
             foreach ($xmlData->children() as $child)
             {

src/Robot.php

Spacing +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
     {
         parent::__construct($xmlData);
 
-        if($xmlData !== null)
+        if ($xmlData !== null)
         {
             foreach ($xmlData->children() as $child)
             {
@@ -86,7 +86,7 @@ 
                         break;
                     case 'isse':
                         $val = $child->__toString();
-                        if($val == 'true')
+                        if ($val == 'true')
                         {
                             $this->setSearchEngine(true);
                         }

Braces +2 added lines, -2 removed lines

@@ -89,9 +89,9 @@
                         if($val == 'true')
                         {
                             $this->setSearchEngine(true);
+                        } else {
+                                                    $this->setSearchEngine(false);
                         }
-                        else
-                            $this->setSearchEngine(false);
                         break;
                 }
             }

src/DetectorRule.php

Spacing +6 added lines, -6 removed lines

@@ -104,7 +104,7 @@ 
 
     public function __construct(\SimpleXMLElement $xmlData)
     {
-        if($xmlData !== null)
+        if ($xmlData !== null)
         {
             foreach ($xmlData->children() as $child) {
                 switch ($child->getName())
@@ -123,7 +123,7 @@ 
                         break;
                     case 'targetProperty':
                         $boolVal = $child->__toString();
-                        if($boolVal == 'true')
+                        if ($boolVal == 'true')
                         {
                             $this->TargetValue = true;
                         }
@@ -142,15 +142,15 @@ 
      */
     public static function loadRulesFromFile()
     {
-        $path = str_replace('src','data',__DIR__).'/rules/';
+        $path = str_replace('src', 'data', __DIR__).'/rules/';
         $files = scandir($path);
         $rules = array();
-        foreach($files as $file)
+        foreach ($files as $file)
         {
-            if($file != '.' && $file != '..')
+            if ($file != '.' && $file != '..')
             {
                 $xmlObj = simplexml_load_file($path.$file);
-                foreach($xmlObj->children() as $rule)
+                foreach ($xmlObj->children() as $rule)
                 {
                     $rules[] = new self($rule);
                 }

Braces +1 added lines, -2 removed lines

@@ -126,8 +126,7 @@
                         if($boolVal == 'true')
                         {
                             $this->TargetValue = true;
-                        }
-                        else
+                        } else
                         {
                             $this->TargetValue = false;
                         }

src/Detector.php

Braces +3 added lines, -6 removed lines

@@ -117,8 +117,7 @@ 
                             $os = new OS($result);
                             $os->setVersion(self::getVersion($result, $ua));
                             $detectorResult->OS = $os;
-                        }
-                        else
+                        } else
                         {
                             $os = OS::initEmpty();
                             $detectorResult->OS = $os;
@@ -129,8 +128,7 @@ 
                         {
                             $device = new Device($result);
                             $detectorResult->Device = $device;
-                        }
-                        else
+                        } else
                         {
                             $device = Device::initEmpty();
                             $detectorResult->Device = $device;
@@ -142,8 +140,7 @@ 
                             $browser = new Browser($result);
                             $browser->setVersion(self::getVersion($result, $ua));
                             $detectorResult->Browser = $browser;
-                        }
-                        else
+                        } else
                         {
                             $browser = Browser::initEmpty();
                             $detectorResult->Browser = $browser;

Spacing +27 added lines, -27 removed lines

@@ -52,26 +52,26 @@ 
      * Detector constructor.
      * @param string $pathToData Path to directory with xml data files
      */
-    private function __construct($pathToData='auto')
+    private function __construct($pathToData = 'auto')
     {
-        if($pathToData == 'auto')
+        if ($pathToData == 'auto')
         {
-            $this->setPathToData(str_replace('src','data',__DIR__).'/');
+            $this->setPathToData(str_replace('src', 'data', __DIR__).'/');
         }
 
-        $xml = array('robot','browser','device','os');
+        $xml = array('robot', 'browser', 'device', 'os');
         $xmlData = array();
-        foreach($xml as $name)
+        foreach ($xml as $name)
         {
             $xmlData[$name] = simplexml_load_file($this->getPathToData().$name.'.xml');
         }
         $this->setXmlData($xmlData);
     }
 
-    public static function analyse($uaString='UA', $pathToData='auto')
+    public static function analyse($uaString = 'UA', $pathToData = 'auto')
     {
         $ua = $uaString;
-        if($uaString == 'UA')
+        if ($uaString == 'UA')
         {
             $ua = $_SERVER['HTTP_USER_AGENT'];
         }
@@ -79,23 +79,23 @@ 
         $detector = new Detector($pathToData);
         $xml = $detector->getXmlData();
         $data = array();
-        foreach($xml as $key => $item)
+        foreach ($xml as $key => $item)
         {
-            $data[$key] = self::analysePart($xml,$key,$ua);
+            $data[$key] = self::analysePart($xml, $key, $ua);
         }
 
         $detectorResult = new DetectorResult();
         $detectorResult->uaString = $ua;
         $isRobot = false;
 
-        foreach($data as $key => $result)
+        foreach ($data as $key => $result)
         {
-            if(!$isRobot)
+            if (!$isRobot)
             {
                 switch ($key)
                 {
                     case 'robot':
-                        if($result !== null)
+                        if ($result !== null)
                         {
                             $robot = new Robot($result);
                             $detectorResult->Robot = $robot;
@@ -105,7 +105,7 @@ 
                         }
                         break;
                     case 'os':
-                        if($result !== null)
+                        if ($result !== null)
                         {
                             $os = new OS($result);
                             $os->setVersion(self::getVersion($result, $ua));
@@ -118,7 +118,7 @@ 
                         }
                         break;
                     case 'device':
-                        if($result !== null)
+                        if ($result !== null)
                         {
                             $device = new Device($result);
                             $detectorResult->Device = $device;
@@ -130,7 +130,7 @@ 
                         }
                         break;
                     case 'browser':
-                        if($result !== null)
+                        if ($result !== null)
                         {
                             $browser = new Browser($result);
                             $browser->setVersion(self::getVersion($result, $ua));
@@ -157,13 +157,13 @@ 
      * @param string $uaString User agent
      * @return \SimpleXMLElement xml element
      */
-    private static function analysePart($xmlData,$key,$uaString)
+    private static function analysePart($xmlData, $key, $uaString)
     {
         $data = $xmlData[$key]->data;
-        foreach($data as $xmlItem)
+        foreach ($data as $xmlItem)
         {
             $pattern = '/'.$xmlItem->pattern.'/';
-            if(preg_match($pattern,$uaString))
+            if (preg_match($pattern, $uaString))
             {
                 return $xmlItem;
             }
@@ -176,21 +176,21 @@ 
      * @param string $uaString User agent
      * @return string Version
      */
-    private static function getVersion(\SimpleXMLElement $xmlItem,$uaString)
+    private static function getVersion(\SimpleXMLElement $xmlItem, $uaString)
     {
-        if($xmlItem !== null)
+        if ($xmlItem !== null)
         {
-            foreach($xmlItem->children() as $node)
+            foreach ($xmlItem->children() as $node)
             {
-                if($node->getName() == 'versionPattern')
+                if ($node->getName() == 'versionPattern')
                 {
                     $vPattern = $node->__toString();
-                    $version = '/' . $vPattern . '(\/| )[\w-._]{1,15}/';
+                    $version = '/'.$vPattern.'(\/| )[\w-._]{1,15}/';
                     $uaString = str_replace(' NT', '', $uaString);
                     if (preg_match($version, $uaString)) {
                         preg_match($version, $uaString, $v);
                         $version = $v[0];
-                        $version = preg_replace('/' . $vPattern . '/', '', $version);
+                        $version = preg_replace('/'.$vPattern.'/', '', $version);
                         $version = str_replace(';', '', $version);
                         $version = str_replace(' ', '', $version);
                         $version = str_replace('/', '', $version);
@@ -234,14 +234,14 @@ 
     private static function checkRules(DetectorResult $result)
     {
         $Rules = DetectorRule::loadRulesFromFile();
-        foreach($Rules as $rule)
+        foreach ($Rules as $rule)
         {
             $objectType = $rule->getObjectType();
             $objectProperty = $rule->getObjectProperty();
             $targetType = $rule->getTargetType();
             $targetValue = $rule->isTargetValue();
             $func = 'get'.$objectProperty;
-            if($result->$objectType !== null)
+            if ($result->$objectType !== null)
             {
                 if ($result->$objectType->$func() == $rule->getObjectPropertyValue()) {
                     $result->$targetType = $targetValue;
@@ -252,4 +252,4 @@ 
         return $result;
     }
 }
-define('D_NA','N\A');
+define('D_NA', 'N\A');