Make sure to use a cached result when we have it so we wont detect it again (matomo-org#54)

* Make sure to use a cached result when we have it so we wont detect it again

* fix tests

Co-authored-by: sgiehl <stefan@matomo.org>

Author: tsteur

Diff for: CachedEntry.php

@@ -39,11 +39,13 @@ public function __construct(string $userAgent, $clientHints, array $values)
         // We use some special parsers, which use the cached user agent result and parses it again using client hints
         if (!empty($clientHints) && $values['client']['type'] === 'browser') {
             $browserParser = new CachedBrowserParser($userAgent, $clientHints);
+            $browserParser->setCachedResult($this->client);
             $this->client = $browserParser->parse();
         }
 
         if (!empty($clientHints)) {
             $osParser = new CachedOperatingSystemParser($userAgent, $clientHints);
+            $osParser->setCachedResult($this->os);
             $this->os = $osParser->parse();
         }
     }

Diff for: tests/Unit/DeviceDetectorCacheFactoryTest.php

@@ -122,8 +122,9 @@ public function testGetInstanceFromCacheWithClientHints()
             'name' => 'Microsoft Edge',
             'version' => '98.0',
             'short_name' => 'PS',
-            'engine' => 'Blink',
-            'engine_version' => '97.0.4692.71',
+            // empty because cached result doesn't contain it and engine won't be detected through client hints only
+            'engine' => '',
+            'engine_version' => '',
             'family' => 'Internet Explorer',
         ];