LogOtherDevicesLogout::handle() - Code Metrics - Inspection of "Adds Support for logoutOtherDevices Method" - yadahan/laravel-authentication-log - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#31)

by Moinuddin

created 2021-02-25 11:56 UTC

LogOtherDevicesLogout::handle() A

↳ Parent: LogOtherDevicesLogout

Complexity

Conditions	5
Paths	7

Size

Total Lines	19
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	5
eloc	13
c	1
b	0
f	0
nc	7
nop	1
dl	0
loc	19
rs	9.5222

<?php

namespace Yadahan\AuthenticationLog\Listeners;

use Illuminate\Auth\Events\OtherDeviceLogout;
filter:
    dependency_paths: ["lib/*"]
use Illuminate\Http\Request;
use Illuminate\Support\Carbon;
use Yadahan\AuthenticationLog\AuthenticationLog;

class LogOtherDevicesLogout
{
    /**
     * The request.
     *
     * @var \Illuminate\Http\Request
     */
    public $request;

    /**
     * Create the event listener.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return void
     */
    public function __construct(Request $request)
    {
        $this->request = $request;
    }

    /**
     * Handle the event.
     *
     * @param  Logout  $event
filter:
    dependency_paths: ["lib/*"]
     * @return void
     */
    public function handle(OtherDeviceLogout $event)
    {
        if ($event->user) {
            $currentUser = $event->user;
            $currentUserIp = $this->request->ip();
            $currentUserAgent = $this->request->userAgent();
            $currentAuthLog = $currentUser->authentications()->whereIpAddress($currentUserIp)->whereUserAgent($currentUserAgent)->first();

            if (!$currentAuthLog) {
                $currentAuthLog = new AuthenticationLog([
                    'ip_address' => $currentUserIp,
                    'user_agent' => $currentUserAgent,
                ]);
            }

            foreach ($currentUser->authentications as $authLog) {
                if ($authLog->id != $currentAuthLog->id) {
                    $authLog->logout_at = Carbon::now();
                    $authLog->save();
                }
            }
        }
    }
}


1			<?php
2
3			namespace Yadahan\AuthenticationLog\Listeners;
4
5			use Illuminate\Auth\Events\OtherDeviceLogout;
			0 ignored issues – show Bug introduced 2021-02-25 12:02 UTC by Report Bug Copy Issue Report The type `Illuminate\Auth\Events\OtherDeviceLogout` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
6			use Illuminate\Http\Request;
7			use Illuminate\Support\Carbon;
8			use Yadahan\AuthenticationLog\AuthenticationLog;
9
10			class LogOtherDevicesLogout
11			{
12			/**
13			* The request.
14			*
15			* @var \Illuminate\Http\Request
16			*/
17			public $request;
18
19			/**
20			* Create the event listener.
21			*
22			* @param \Illuminate\Http\Request $request
23			* @return void
24			*/
25			public function __construct(Request $request)
26			{
27			$this->request = $request;
28			}
29
30			/**
31			* Handle the event.
32			*
33			* @param Logout $event
			0 ignored issues – show Bug introduced 2021-02-25 12:02 UTC by Report Bug Copy Issue Report The type `Yadahan\AuthenticationLog\Listeners\Logout` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
34			* @return void
35			*/
36			public function handle(OtherDeviceLogout $event)
37			{
38			if ($event->user) {
39			$currentUser = $event->user;
40			$currentUserIp = $this->request->ip();
41			$currentUserAgent = $this->request->userAgent();
42			$currentAuthLog = $currentUser->authentications()->whereIpAddress($currentUserIp)->whereUserAgent($currentUserAgent)->first();
43
44			if (!$currentAuthLog) {
45			$currentAuthLog = new AuthenticationLog([
46			'ip_address' => $currentUserIp,
47			'user_agent' => $currentUserAgent,
48			]);
49			}
50
51			foreach ($currentUser->authentications as $authLog) {
52			if ($authLog->id != $currentAuthLog->id) {
53			$authLog->logout_at = Carbon::now();
54			$authLog->save();
55			}
56			}
57			}
58			}
59			}
60

yadahan / laravel-authentication-log

Pull Request — master (#31)

LogOtherDevicesLogout::handle() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like