Yajra\Auditable\AuditableTraitObserver

Complexity

Total Complexity

7

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	4
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	creating()	0	10	3
A	getAuthenticatedUserId()	0	4	2
A	updating()	0	6	2

<?php

namespace Yajra\Auditable;

/**
 * Class AuditableObserver
 *
 * @package Yajra\Auditable
 */
class AuditableTraitObserver
{
    /**
     * Model's creating event hook.
     *
     * @param \Yajra\Auditable\AuditableTrait $model

The type AuditableTrait for parameter $model is a trait, and thus cannot be used for type-hinting in PHP. Maybe consider adding an interface and use that for type-hinting?

     */
    public function creating($model)
    {
        if (! $model->created_by) {
            $model->created_by = $this->getAuthenticatedUserId($model);
        }

        if (! $model->updated_by) {
            $model->updated_by = $this->getAuthenticatedUserId($model);
        }
    }

    /**
     * Get authenticated user id depending on model's auth guard.
     *
     * @param mixed $model
     * @return int
     */
    protected function getAuthenticatedUserId($model)
    {
        return auth($model->getAuthGuard())->check() ? auth($model->getAuthGuard())->id() : 0;
    }

    /**
     * Model's updating event hook.
     *
     * @param \Yajra\Auditable\AuditableTrait $model

The type AuditableTrait for parameter $model is a trait, and thus cannot be used for type-hinting in PHP. Maybe consider adding an interface and use that for type-hinting?

     */
    public function updating($model)
    {
        if (! $model->updated_by) {
            $model->updated_by = $this->getAuthenticatedUserId($model);
        }
    }
}