Longman\Platfourm\Database\Eloquent\EntityLock\EntityLockTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	107
Duplicated Lines	15.89 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
B	lockIn()	0	29	2
A	getEntityLockModelInstance()	8	8	2
A	withoutEntityLock()	0	5	1
A	isLocked()	0	5	1
A	entityLock()	9	9	2

<?php
/*
 * This file is part of the Laravel Platfourm package.
 *
 * (c) Avtandil Kikabidze aka LONGMAN <akalongman@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Longman\Platfourm\Database\Eloquent\EntityLock;

use Auth;
use Carbon\Carbon;
use Longman\Platfourm\Database\Eloquent\EntityLock\EntityLock;

trait EntityLockTrait
{

    protected $entityLock = true;

    /**
     * Boot the trait for the model.
     *
     * @return void
     */
/*    public static function bootActionLogTrait()

67% of this comment could be valid code. Did you maybe forget this after debugging?

    {

        static::created(function ($model) {
            if (!empty($model->actionLogging)) {
                static::storeActionLog($model, 'create');
            }
        });

        static::updated(function ($model) {
            if (!empty($model->actionLogging)) {
                static::storeActionLog($model, 'update');
            }
        });

        static::deleted(function ($model) {
            if (!empty($model->actionLogging)) {
                static::storeActionLog($model, 'delete');
            }
        });

        if (method_exists(static::class, 'restored')) {
            static::restored(function ($model) {
                if (!empty($model->actionLogging)) {
                    static::storeActionLog($model, 'restore');
                }
            });
        }
    }*/

    public function lockIn()
    {
        $entityLock = static::getEntityLockModelInstance();

        $model = $this;

        $user_id = Auth::user() ? Auth::user()->id : '';

        $lang = app()->getLocale();

        $className = get_class($model);

$className is not used, you could remove the assignment.

        $data = [
            'user_id' => $user_id,
            'lang' => $lang,
            'entity' => static::getModelClassName($model),
            'scope' => app()->getScope(),
            'action' => $action,

The variable $action does not exist. Did you forget to declare it?

            'before_data' => $before_data,

The variable $before_data does not exist. Did you forget to declare it?

            'after_data' => $after_data,

The variable $after_data does not exist. Did you forget to declare it?

            'request_data' => app('request')->all(),
            'url' => app('request')->fullUrl(),
            'referer' => app('request')->server->get('HTTP_REFERER'),
            'ip_address' => app('request')->ip(),
            'user_agent' => app('request')->server->get('HTTP_USER_AGENT'),
        ];

        $entityLock->create($data);
    }

    protected static function getEntityLockModelInstance()

This method seems to be duplicated in your project.

    {
        $model = EntityLock::class;
        if (class_exists(\App\Models\EntityLock::class)) {
            $model = \App\Models\EntityLock::class;
        }
        return new $model;
    }


    public function withoutEntityLock()
    {
        $this->entityLock = false;
        return $this;
    }

    public function isLocked()
    {
        $date = Carbon::now()->subMinutes(5);
        return $this->entityLock()->where('locked_at', '>=', $date->toDateTimeString());
    }

    /**
     * Polymorphic relationship. Second parameter to morphOne/morphMany
     * should be the same as the prefix for the *_id/*_type fields.
     */
    public function entityLock()

This method seems to be duplicated in your project.

    {
        $model = EntityLock::class;
        if (class_exists(\App\Models\EntityLock::class)) {
            $model = \App\Models\EntityLock::class;
        }

        return $this->morphOne($model, 'entity');

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

    }

}