Runtime enable/disable auditing suggestion

[lauri911]

Q	A
New Feature?	yes
Package version	6.0.x

Expected Behaviour

Sometimes is need to disable auditing for one model temporarily, for example when doing some big imports and auditing for every row is not necessary.

For example:

Customer::disableAuditing();

foreach($thousandsImportedCustomers as $importedCustomer){
    Customer::create($data);   // don't want to audit this here
}

Customer::enableAuditing();

Possible Solutions

I am doing it via additional trait at moment but you could just add something like that to have it right out of the box :)

public static $auditingDisabled = false;

public static function isAuditingEnabled(): bool
 {
        if (self::$auditingDisabled) {
            return false;
        }

        if (App::runningInConsole()) {
            return Config::get('audit.console', false);
        }

        return true;
}

public static function disableAuditing()
{
        self::$auditingDisabled = true;
}

public static function enableAuditing()
{
        self::$auditingDisabled = false;
}

[quetzyg]

Thanks for your suggestion @lauri911.
I'll take it into consideration.

[lauri911]

Thanks, because adding it with additional trait is little bit tricky and "dirty":

    use Auditable, AuditingDisablingFunctionality {

        AuditingDisablingFunctionality::isAuditingEnabled insteadOf Auditable;

        Auditable::isAuditingEnabled as originalIsAuditingEnabled;

    }

And then in trait:

    public static function isAuditingEnabled(): bool
    {
        if (self::$auditingDisabled) {
            return false;
        }

        return static::originalIsAuditingEnabled();
    }

[quetzyg]

I'll update the documentation ASAP.