endorphin-studio / browser-detector

src/Detector.php

Spacing +29 added lines, -29 removed lines

@@ -54,16 +54,16 @@ 
      * 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');
         }
@@ -72,10 +72,10 @@ 
         $this->Rules = DetectorRule::loadRulesFromFile();
     }
 
-    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'];
         }
@@ -83,23 +83,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;
@@ -109,7 +109,7 @@ 
                         }
                         break;
                     case 'os':
-                        if($result !== null)
+                        if ($result !== null)
                         {
                             $os = new OS($result);
                             $os->setVersion(self::getVersion($result, $ua));
@@ -122,7 +122,7 @@ 
                         }
                         break;
                     case 'device':
-                        if($result !== null)
+                        if ($result !== null)
                         {
                             $device = new Device($result);
                             $detectorResult->Device = $device;
@@ -134,7 +134,7 @@ 
                         }
                         break;
                     case 'browser':
-                        if($result !== null)
+                        if ($result !== null)
                         {
                             $browser = new Browser($result);
                             $browser->setVersion(self::getVersion($result, $ua));
@@ -150,7 +150,7 @@ 
             }
         }
 
-        $detectorResult = self::checkRules($detector,$detectorResult);
+        $detectorResult = self::checkRules($detector, $detectorResult);
 
         return $detectorResult;
     }
@@ -161,13 +161,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;
             }
@@ -180,21 +180,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);
@@ -235,16 +235,16 @@ 
      * @param DetectorResult $result Detector result
      * @return DetectorResult Final result
      */
-    private static function checkRules(Detector $object,DetectorResult $result)
+    private static function checkRules(Detector $object, DetectorResult $result)
     {
-        foreach($object->Rules as $rule)
+        foreach ($object->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;
@@ -255,4 +255,4 @@ 
         return $result;
     }
 }
-define('D_NA','N\A');
+define('D_NA', 'N\A');

tests/RobotTest.php

Spacing +23 added lines, -23 removed lines

@@ -22,8 +22,8 @@ 
             'Tiny Tiny RSS/1.10 (http://tt-rss.org/)'
         );
 
-        testUaList($this,'Robot','Name',$ualist,'Tiny RSS');
-        testUaList($this,'Robot','Type',$ualist,'RSS Reader');
+        testUaList($this, 'Robot', 'Name', $ualist, 'Tiny RSS');
+        testUaList($this, 'Robot', 'Type', $ualist, 'RSS Reader');
     }
     /**
      * Test Google Bots
@@ -43,9 +43,9 @@ 
             'Mediapartners-Google'
         );
 
-        testUaList($this,'Robot','Owner',$ualist,'Google Inc.');
-        testUaList($this,'Robot','Type',$ualist,'Search Engine');
-        testUaListBooleanTrue($this,'Robot','SearchEngine',$ualist);
+        testUaList($this, 'Robot', 'Owner', $ualist, 'Google Inc.');
+        testUaList($this, 'Robot', 'Type', $ualist, 'Search Engine');
+        testUaListBooleanTrue($this, 'Robot', 'SearchEngine', $ualist);
     }
     /**
      * Test Yandex Bots
@@ -84,9 +84,9 @@ 
             'Mozilla/5.0 (compatible; YandexSpravBot/1.0; +http://yandex.com/bots)'
         );
 
-        testUaList($this,'Robot','Owner',$ualist,'Yandex LLC.');
-        testUaList($this,'Robot','Type',$ualist,'Search Engine');
-        testUaListBooleanTrue($this,'Robot','SearchEngine',$ualist);
+        testUaList($this, 'Robot', 'Owner', $ualist, 'Yandex LLC.');
+        testUaList($this, 'Robot', 'Type', $ualist, 'Search Engine');
+        testUaListBooleanTrue($this, 'Robot', 'SearchEngine', $ualist);
     }
     /**
      * Test Bing Bots
@@ -106,9 +106,9 @@ 
             'Mozilla/5.0 (Windows Phone 8.1; ARM; Trident/7.0; Touch; rv:11.0; IEMobile/11.0; NOKIA; Lumia 530) like Gecko BingPreview/1.0b'
         );
 
-        testUaList($this,'Robot','Name',$ualist,'Bing');
-        testUaList($this,'Robot','Type',$ualist,'Search Engine');
-        testUaListBooleanTrue($this,'Robot','SearchEngine',$ualist);
+        testUaList($this, 'Robot', 'Name', $ualist, 'Bing');
+        testUaList($this, 'Robot', 'Type', $ualist, 'Search Engine');
+        testUaListBooleanTrue($this, 'Robot', 'SearchEngine', $ualist);
     }
     /**
      * Test WASALive
@@ -119,8 +119,8 @@ 
             'Mozilla/5.0 (compatible; WASALive-Bot ; http://blog.wasalive.com/wasalive-bots/) '
         );
 
-        testUaList($this,'Robot','Name',$ualist,'WASALive Bot');
-        testUaList($this,'Robot','Type',$ualist,'Bot/Crawler');
+        testUaList($this, 'Robot', 'Name', $ualist, 'WASALive Bot');
+        testUaList($this, 'Robot', 'Type', $ualist, 'Bot/Crawler');
     }
     /**
      * Test .Net robot
@@ -131,8 +131,8 @@ 
             'MS Web Services Client Protocol 1.0.3705.0'
         );
 
-        testUaList($this,'Robot','Name',$ualist,'.NET Framework CLR');
-        testUaList($this,'Robot','Type',$ualist,'Bot/Crawler');
+        testUaList($this, 'Robot', 'Name', $ualist, '.NET Framework CLR');
+        testUaList($this, 'Robot', 'Type', $ualist, 'Bot/Crawler');
     }
     /**
      * Test 007ac9 robot
@@ -143,8 +143,8 @@ 
             'Mozilla/5.0 (compatible; 007ac9 Crawler; http://crawler.007ac9.net/) '
         );
 
-        testUaList($this,'Robot','Name',$ualist,'007ac9 Crawler');
-        testUaList($this,'Robot','Type',$ualist,'Bot/Crawler');
+        testUaList($this, 'Robot', 'Name', $ualist, '007ac9 Crawler');
+        testUaList($this, 'Robot', 'Type', $ualist, 'Bot/Crawler');
     }
     /**
      * Test 80legs robot
@@ -155,8 +155,8 @@ 
             'Mozilla/5.0 (compatible; 008/0.83; http://www.80legs.com/webcrawler.html) Gecko/2008032620 '
         );
 
-        testUaList($this,'Robot','Name',$ualist,'80legs Crawler');
-        testUaList($this,'Robot','Type',$ualist,'Bot/Crawler');
+        testUaList($this, 'Robot', 'Name', $ualist, '80legs Crawler');
+        testUaList($this, 'Robot', 'Type', $ualist, 'Bot/Crawler');
     }
     /**
      * Test 80123metaspider robot
@@ -167,8 +167,8 @@ 
             '123metaspider-Bot (Version: 1.04, powered by www.123metaspider.com) '
         );
 
-        testUaList($this,'Robot','Name',$ualist,'123metaspider Crawler');
-        testUaList($this,'Robot','Type',$ualist,'Bot/Crawler');
+        testUaList($this, 'Robot', 'Name', $ualist, '123metaspider Crawler');
+        testUaList($this, 'Robot', 'Type', $ualist, 'Bot/Crawler');
     }
     /**
      * Test 1470 robot
@@ -179,7 +179,7 @@ 
             '1470.net crawler '
         );
 
-        testUaList($this,'Robot','Name',$ualist,'1470 Crawler');
-        testUaList($this,'Robot','Type',$ualist,'Bot/Crawler');
+        testUaList($this, 'Robot', 'Name', $ualist, '1470 Crawler');
+        testUaList($this, 'Robot', 'Type', $ualist, 'Bot/Crawler');
     }
 }