Moecasts\Laravel\UserLoginLog\Traits\LoginLoggable

Complexity

Total Complexity

5

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	isNewLogin()	0	3	1
A	loginLogs()	0	3	1
A	createLoginLog()	0	17	1
A	logLogin()	0	10	2

<?php

namespace Moecasts\Laravel\UserLoginLog\Traits;

use Carbon\Carbon;
use Illuminate\Database\Eloquent\Relations\MorphMany;
use Jenssegers\Agent\Agent;
use Moecasts\Laravel\UserLoginLog\Cacher;
use Moecasts\Laravel\UserLoginLog\DeviceType;
use Moecasts\Laravel\UserLoginLog\Models\UserLoginLog;

trait LoginLoggable
{
    public function loginLogs(): MorphMany
    {
        return $this->morphMany(UserLoginLog::class, 'loggable');

It seems like morphMany() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    public function logLogin($seconds = null): ?UserLoginLog
    {
        $log = null;
        if ($this->isNewLogin()) {
            $log = $this->createLoginLog();
        }

        Cacher::setCache($this, $seconds);

        return $log;
    }

    public function createLoginLog(): UserLoginLog
    {
        $agent = new Agent();
        $log = new UserLoginLog([
            'ip' => request()->ip(),
            'device' => $agent->device(),
            'platform' => $agent->platform(),
            'browser' => $agent->browser(),
            'device_type' => DeviceType::getValue($agent),
            'login_at' => Carbon::now(),
        ]);

        $log->platform_version = $agent->version($log->platform);
        $log->browser_version = $agent->version($log->browser);

        $this->loginLogs()->save($log);
        return $log;
    }

    public function isNewLogin(): bool
    {
        return !Cacher::hasCache($this);
    }
}