Skip to content
Snippets Groups Projects
Commit 006ee433 authored by Steven Ayers's avatar Steven Ayers
Browse files

Issue #3496400: Optional: Device Detect

parent ba1b1a51
Branches
No related tags found
1 merge request!201Issue #3496400 by bluegeek9: Optional: Device Detect
Pipeline #380850 failed
......@@ -159,7 +159,7 @@ final class Visitors extends ControllerBase {
$server = $request->server;
$query = $request->query->all();
$fields = [];
$fields = $this->getDefaultFields();
$ip = $request->getClientIp();
$fields['visitors_ip'] = $ip;
......@@ -200,6 +200,18 @@ final class Visitors extends ControllerBase {
return $response;
}
/**
* Get the default fields.
*
* @return array
* The default fields.
*/
protected function getDefaultFields(): array {
return [
'bot' => 0,
];
}
/**
* Detects the visitor url.
*
......@@ -248,6 +260,10 @@ final class Visitors extends ControllerBase {
* The server array.
*/
protected function doDeviceDetect(array &$fields, ServerBag $server) {
if (!class_exists('DeviceDetector\ClientHints')) {
return NULL;
}
$user_agent = $server->get('HTTP_USER_AGENT', '');
$dd = $this->device->getDeviceDetector($user_agent, $server->all());
......
......@@ -177,6 +177,7 @@ class VisitorsControllerTest extends UnitTestCase {
* @covers ::doTime
* @covers ::doLanguage
* @covers ::doLocation
* @covers ::getDefaultFields
*/
public function testTrack() {
$request = $this->createMock('\Symfony\Component\HttpFoundation\Request');
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment