How to retrieve all activities for the current user?

[nickywan123]

I followed the docs and implemented this package to my app. However, let say I have a Transaction model where it also logs into the activity_log table. There's no user ID on the table but just the subject_id that belongs to transactions table. My goal is to display all the activities that belong or done by the user so I can show it in my view.

I don't see any way to retrieve it on the docs, do I need to customize the table and add a user_id?

[Gummibeer]

There's no user_id but a causer which is for the user or any other entity doing activities.

https://spatie.be/docs/laravel-activitylog/v4/basic-usage/logging-activity#setting-a-causer
https://spatie.be/docs/laravel-activitylog/v4/advanced-usage/logging-model-events#using-the-causesactivity-trait
https://spatie.be/docs/laravel-activitylog/v4/advanced-usage/define-causer-for-runtime
https://spatie.be/docs/laravel-activitylog/v4/api/causer-resolver

[Gummibeer]

This wouldn't be enough. You should use the polymorphic methods to define the relationship.
If I remember right it should be $activity->causer()->associate($this->user)

laravel-activitylog/src/ActivityLogger.php

Line 67 in 1d8acda

$this->getActivity()->causer()->associate($model);

[nickywan123]

I wrote it like this:

 public function tapActivity(Activity $activity, string $eventName)
    {   
        $activity->causer_type = "App\User";
        $activity->causer_id = $this->id;
    }

and it does works as I tested. Alternatively, if user is logged, you can also do it like this:

 public function tapActivity(Activity $activity, string $eventName)
    {   
     
        $activity->causer_id = auth()->id();
    }

and it will populate causer_type as well as causer_id automatically.

[Gummibeer]

If you are logged in you don't have to do anything as we associate the logged in user by default to all activities.
But assigning a polymorphic relationship manually has two big problems.

1. refactoring the class doesn't change the type automatically - moving or renaming
2. it doesn't use the morph map to customize the type - using a morph map is highly recommended to decouple your database from code/class structure.

[nickywan123]

So how should I refactor this to :

 public function tapActivity(Activity $activity, string $eventName)
    {   
        $activity->causer_type = "App\User";
        $activity->causer_id = $this->user_id;
    }

?

My current way of doing works, is it okay?

The reason I do it this way, for example if a user successfully register an account, it does not logged the causer details, that's why I use tapActivity. Another reason is because when I am in the payment processor page(external site), it does not logged the user in my system so after successful transaction, it brings user back to my system where it couldn't get the logged user.

[Gummibeer]

Like I already wrote. Nothing wrong about the tap method. But you shouldn't assign the polymorphic attributes manually but let eloquent do it.

This wouldn't be enough. You should use the polymorphic methods to define the relationship.
If I remember right it should be $activity->causer()->associate($this->user)

laravel-activitylog/src/ActivityLogger.php

Line 67 in 1d8acda

$this->getActivity()->causer()->associate($model);