How can I audit delete options

[cesarcruzc]

When I delete a record dont log in the table audits

[anteriovieira]

Hi @cesarcruzc , I would like to better understand your case. Can you provide an example of what you are doing.

[cesarcruzc]

Hi @anteriovieira.

I'm using Laravel 5.2 and working with repository pattern.

This is an example of my Models:

`
namespace App\Entities;

use Illuminate\Database\Eloquent\Model;
use OwenIt\Auditing\Auditable;
use Spatie\Activitylog\Traits\LogsActivity;

class AccidentCase extends Model
{

Use Auditable, LogsActivity;

protected $table = 'bdc_caso_accidente';

protected $fillable = ['nombres', 'telefono', 'celular', 'ciudad_id', 'bdc_datos_trabajador_id', 'entry_by'];

protected $guarded = ['id'];

// LogsActivity
protected static $logAttributes = ['nombres', 'telefono', 'celular', 'ciudad_id', 'bdc_datos_trabajador_id', 'entry_by'];


public function getLogNameToUse(string $eventName = ''): string
{
    return $this->table;
}
//

// Auditable
protected $auditableTypes = ['created', 'saved', 'deleted'];
//


// Mutators

public function getNombresAttribute($value)
{
    return title_case(strtolower($value));
}

//--

// Relations

public function workerData()
{
    return $this->belongsTo(WorkerData::class, 'bdc_datos_trabajador_id');
}

// --

}
`

[cesarcruzc]

I am testing this package https://github.com/spatie/laravel-activitylog and I have the same issue.

am I doing something wrong?

[anteriovieira]

As you are deleting the model?

Because have some queries that laravel does not run events.

[anteriovieira]

This should work.

$AccidentCase = AccidentCase::find(1);

$AccidentCase ->delete();

[cesarcruzc]

My Repository:

class AccidentCaseRepository extends BaseRepository
{

public function getModel() { return new AccidentCase(); }

public function deleteDataByWorker($workerId) { $model = $this->getModel(); return $model->where('bdc_datos_trabajador_id', $workerId)->delete(); }

[anteriovieira]

This is the same case reported in issues 139.

[anteriovieira]

The solution is the following:

class AccidentCaseRepository extends BaseRepository 
{ 
    protected $accidentCase;

    public function __construct(AccidentCase $accidentCase)
    {
        $this->accidentCase = $accidentCase;
    }

    public function deleteDataByWorker($workerId) 
    { 
        $accidentCase = $this->accidentCase->find($workerId);

        return $accidentCase->delete();
    } 
}

[anteriovieira]

ops!

[anteriovieira]

class AccidentCaseRepository extends BaseRepository 
{ 
    protected $accidentCase;

    public function __construct(AccidentCase $accidentCase)
    {
        $this->accidentCase = $accidentCase;
    }

    public function deleteDataByWorker($workerId) 
    { 
        $accidentCase = $this->accidentCase
                             ->where('bdc_datos_trabajador_id', $workerId)
                             ->firstOrFail();

        return $accidentCase->delete();
    } 
}

[cesarcruzc]

@anteriovieira Thank you so much!
This solution worked for me!!