OwenIt\Auditing\Drivers\Database

Complexity

Total Complexity

7

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Test Coverage

Coverage

80%

Importance

Changes	7
Bugs	0	Features	2

2 Methods

Rating	Name	Duplication	Size	Complexity
A	audit()	0	14	4
A	prune()	0	17	3

<?php

namespace OwenIt\Auditing\Drivers;

use Illuminate\Support\Facades\Config;
use OwenIt\Auditing\Contracts\Audit;
use OwenIt\Auditing\Contracts\Auditable;
use OwenIt\Auditing\Contracts\AuditDriver;

class Database implements AuditDriver
{
    /**
     * {@inheritdoc}
     */
    public function audit(Auditable $model): Audit
    {
        $implementation = Config::get('audit.implementation', \OwenIt\Auditing\Models\Audit::class);

        // If we want to avoid storing Audits with empty old_values & new_values, return null here.
        if (! Config::get('audit.empty_values')) {
            $attributeGetter = $model->resolveAttributeGetter($model->getAuditEvent());

The method resolveAttributeGetter() does not exist on OwenIt\Auditing\Contracts\Auditable. Since it exists in all sub-types, consider adding an abstract or default implementation to OwenIt\Auditing\Contracts\Auditable.

            list($old, $new) = $model->$attributeGetter();
            if (empty($old) && empty($new)) {
                return null;

The expression return null returns the type null which is incompatible with the type-hinted return OwenIt\Auditing\Contracts\Audit.

            }
        }

        return call_user_func([$implementation, 'create'], $model->toAudit());
    }

    /**
     * {@inheritdoc}
     */
    public function prune(Auditable $model): bool
    {
        if (($threshold = $model->getAuditThreshold()) > 0) {
            $forRemoval = $model->audits()
                ->latest()
                ->get()
                ->slice($threshold)
                ->pluck('id');

            if (!$forRemoval->isEmpty()) {
                return $model->audits()
                    ->whereIn('id', $forRemoval)
                    ->delete() > 0;
            }
        }

        return false;
    }
}