Yadahan\AuthenticationLog\Listeners\LogSuccessfulOtherDeviceLogout

Complexity

Total Complexity

3

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	handle()	0	9	2

<?php

namespace Yadahan\AuthenticationLog\Listeners;

use Illuminate\Auth\Events\OtherDeviceLogout;
use Illuminate\Http\Request;
use Illuminate\Support\Carbon;
use Yadahan\AuthenticationLog\AuthenticationLog;

class LogSuccessfulOtherDeviceLogout
{
    /**
     * 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

The type Yadahan\AuthenticationLog\Listeners\Logout was not found. Maybe you did not declare it correctly or list all dependencies?

     * @return void
     */
    public function handle(OtherDeviceLogout $event)
    {
        if ($event->user) {
            $user = $event->user;

            $authenticationLog = $user->authentications()->whereNull('logout_at')->get()->skip(1);

The method authentications() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of Illuminate\Contracts\Auth\Authenticatable such as Illuminate\Foundation\Auth\User.

            $user->authentications()->whereIn('id', $authenticationLog->pluck('id'))->update([
                'logout_at' => Carbon::now(),
            ]);
        }
    }
}