What is best practice to save history in laravel?

We have big laravel application . In many controller method we are calling golbal helper function which is saving history. Here is our method.

            $user_id =  $userId;
$user_model = $userModel;
$model = Consignment::class;
$object_id = $consignment->id;
$status = getConsignmentStatus($consignment->status);
$comment = "Consignment has been created";
$visibility = "Internal Only";
//Hitory Helper function
history($user_id, $object_id, $model, $status, $comment, $visibility, $user_model);

History function

function history($user_id, $object_id, $model, $status, $comment, $visibility = '', $user_model = null)
$history = new History;
$history->user_id = $user_id;
$history->object_id = $object_id;
$history->model = $model;
$history->status = $status;
$history->comment = $comment;
$history->visibility = $visibility;
if (!empty($user_model))
$history->user_model = $user_model;

return "history inserted";

But I personally think that there could be a better approach, I heared that this type of things should be done by events but why and how I dont know.

from Newest questions tagged laravel-5 – Stack Overflow https://ift.tt/3mr5wui

Related Posts

Codeigniter : Parse error: syntax error, unexpected ‘const’ (T_CONST), expecting variable (T_VARIABLE) in Laravel project

I’m getting following error: **Parse error: syntax error, unexpected ‘const’ (T_CONST), expecting variable (T_VARIABLE)** Note : It’s working in local but facing issue in production server. private…

Firebase receive notification while tab is active or on focus

What i want is to be able to perform an action when a user receives a notification while the browser is open and tab is active or…

Laravel’s alias loader does not find class

We have a legacy project that we cannot update and we need to make some changes in symfony’s Response.php in vendor. We have solved this by copying…

Laravel 5 – generic document management

I have a system where you can create different types of unique documents. For instance, one document is called Project Identified and this expects certain inputs. Originally,…

Laravel Nova limit the results in indexQuery

I ran intro a situation where I need to limit the results of a resource to only 3 results. To be more specific, based on the logged…

Auditoria en laravel 5.8 [closed]

Cómo puedo automatizar el registro de actividades de un usuario en laravel? Si un usuario ingresa a un app de laravel, debo guardar toda su actividas, a…

Leave a Reply

Your email address will not be published.