refactor: use superglobals service in the UserAgent class

Author: michalsn

system/HTTP/UserAgent.php

@@ -109,8 +109,10 @@ public function __construct(?UserAgents $config = null)
     {
         $this->config = $config ?? config(UserAgents::class);
 
-        if (isset($_SERVER['HTTP_USER_AGENT'])) {
-            $this->agent = trim($_SERVER['HTTP_USER_AGENT']);
+        $userAgent = service('superglobals')->server('HTTP_USER_AGENT');
+
+        if ($userAgent !== null) {
+            $this->agent = trim($userAgent);
             $this->compileData();
         }
     }
@@ -175,10 +177,11 @@ public function isMobile(?string $key = null): bool
     public function isReferral(): bool
     {
         if (! isset($this->referrer)) {
-            if (empty($_SERVER['HTTP_REFERER'])) {
+            $referer = service('superglobals')->server('HTTP_REFERER');
+            if ($referer === null || $referer === '') {
                 $this->referrer = false;
             } else {
-                $refererHost = @parse_url($_SERVER['HTTP_REFERER'], PHP_URL_HOST);
+                $refererHost = @parse_url($referer, PHP_URL_HOST);
                 $ownHost     = parse_url(\base_url(), PHP_URL_HOST);
 
                 $this->referrer = ($refererHost && $refererHost !== $ownHost);
@@ -241,7 +244,8 @@ public function getMobile(): string
      */
     public function getReferrer(): string
     {
-        return empty($_SERVER['HTTP_REFERER']) ? '' : trim($_SERVER['HTTP_REFERER']);
+        $referrer = service('superglobals')->server('HTTP_REFERER');
+        return $referrer === null ? '' : trim($referrer);
     }
 
     /**

tests/system/HTTP/UserAgentTest.php

@@ -31,17 +31,16 @@ protected function setUp(): void
         parent::setUp();
 
         // set a baseline user agent
-        $_SERVER['HTTP_USER_AGENT'] = $this->_user_agent;
+        service('superglobals')->setServer('HTTP_USER_AGENT', $this->_user_agent);
 
         $this->agent = new UserAgent();
     }
 
     public function testMobile(): void
     {
         // Mobile Not Set
-        $_SERVER['HTTP_USER_AGENT'] = $this->_mobile_ua;
+        service('superglobals')->setServer('HTTP_USER_AGENT', $this->_mobile_ua);
         $this->assertFalse($this->agent->isMobile());
-        unset($_SERVER['HTTP_USER_AGENT']);
     }
 
     public function testIsFunctions(): void
@@ -55,12 +54,13 @@ public function testIsFunctions(): void
 
     public function testReferrer(): void
     {
-        $_SERVER['HTTP_REFERER'] = 'http://codeigniter.com/user_guide/';
+        service('superglobals')->setServer('HTTP_REFERER', 'http://codeigniter.com/user_guide/');
+
         $this->assertTrue($this->agent->isReferral());
         $this->assertSame('http://codeigniter.com/user_guide/', $this->agent->getReferrer());
 
         $this->setPrivateProperty($this->agent, 'referrer', null);
-        unset($_SERVER['HTTP_REFERER']);
+        service('superglobals')->setServer('HTTP_REFERER', '');
         $this->assertFalse($this->agent->isReferral());
         $this->assertSame('', $this->agent->getReferrer());
     }
@@ -108,7 +108,7 @@ public function testParseBot(): void
 
     public function testEmptyUserAgentVariable(): void
     {
-        unset($_SERVER['HTTP_USER_AGENT']);
+        service('superglobals')->setServer('HTTP_USER_AGENT', '');
         $agent = new UserAgent();
         $this->assertEmpty((string) $agent);
     }