Trim unknown from tracking

Author: mikebronner

CHANGELOG.md

@@ -11,11 +11,17 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 (This is already detected in subscription update.)
 - Filter any incoming webhook events that are in test mode.
 
-## [0.4.9] - 28 OCt 2015
+## [0.4.10 - 0.4.11] - 28 OCt 2015
+### Added
+- tracking of robots, if a browser isn't detected, and it is confirmed as a robot.
+
 ### Changed
 - reverted from using getAttribute(), as it is redundant.
 - refactored `track()` method to track: Url, Operating System, Hardware, Browser, Referring Domain, IP (For GeoLocation)
 
+### Removed
+- any "unknown" values from being passed.
+
 ## [0.4.7 - 0.4.8] - 25 Oct 2015
 ### Changed
 - referenced to user model properties to use `getAttribute()` instead of referencing them directly.

src/LaravelMixpanel.php

@@ -40,13 +40,14 @@ public function track($event, $properties = [])
         $browserInfo = new Browser();
         $osInfo = new Os();
         $deviceInfo = new Device();
-        $osVersion = $osInfo->getName() . ' ' . $osInfo->getVersion();
-        $hardware = $deviceInfo->getName() . ' ' . $deviceInfo->getVersion();
+        $browserVersion = trim(str_replace('unknown', '', $browserInfo->getName() . ' ' . $browserInfo->getVersion()));
+        $osVersion = trim(str_replace('unknown', '', $osInfo->getName() . ' ' . $osInfo->getVersion()));
+        $hardwareVersion = trim(str_replace('unknown', '', $deviceInfo->getName() . ' ' . $deviceInfo->getVersion()));
         $data = [
             'Url' => $this->request->getUri(),
             'Operating System' => $osVersion,
-            'Hardware' => $hardware,
-            '$browser' => $browserInfo->getName() . ' ' . $browserInfo->getVersion(),
+            'Hardware' => $hardwareVersion,
+            '$browser' => $browserVersion,
             'Referrer' => $this->request->header('referer'),
             '$referring_domain' => ($this->request->header('referer')
                 ? parse_url($this->request->header('referer'))['host']
@@ -55,8 +56,11 @@ public function track($event, $properties = [])
         ];
         array_filter($data);
         array_filter($properties);
-        $properties = $data + $properties;
 
-        parent::track($event, $properties);
+        if ((! array_key_exists('$browser', $data)) && $browserInfo->isRobot()) {
+            $data['$browser'] = 'Robot';
+        }
+
+        parent::track($event, $data + $properties);
     }
 }