getActivitylogOptions() has to be implemented on every models that uses LogsActivity trait

[acgtwentyone]

"The trait contains an abstract method getActivitylogOptions() that you can use to customize options. It needs to return a LogOptions instance built from LogOptions::defaults()" as mentioned on the docs

Let's say i have thousand of models and just upgrade to from version 3 to version 4.

And then i run composer dump-autoload

The results is something like that:

See ?

In this case i solved this by adding those lines of code:

use Spatie\Activitylog\LogOptions;

And then

public function getActivitylogOptions(): LogOptions
{
    return LogOptions::defaults()
}

But with this approach i had to add those line of code on all of my models that were using this use LogsActivity which may seems to be a better approach.

List of my models (a few of them actually):

So i think the getActivitylogOptions abstract method should be revised.

[sherifalaa55]

I know this is an old thread but I just bumped into this and the decision to make us implement the abstract method in each model is really weird... I'd love to know the reasoning for it

[acgtwentyone]

@sherifalaa55 It does not makes sense for sure.

[AshauranceHeavens]

The documentation on this link cannot be found https://spatie.be/docs/laravel-activitylog/v4/advanced-usage/logging-model-events

[mlars84]

Yeah. the docs are also only for v1 or v2. When I installed, it installed v4.7. Seems like they're maintaining the code, but not the documentation. I haven't tried reverting yet, but as long as there are no dependency conflicts, that should follow the docs.

[AshauranceHeavens]

Thanks, I was able to get it to work