[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Dolibarr-dev] Facture class

From: Destailleur Laurent
Subject: Re: [Dolibarr-dev] Facture class
Date: Tue, 25 Jun 2013 09:38:12 +0200

Into crud method (create/update/delete), because user is used for audit
Because, sometimes, the event create or update or delete is done by a batch or for a different user than the one used to login, it is better to have $user as argument.
Some classes has it, some not. But we should change this to add it (into create/update/delete) when we can.

2013/6/10 Florian Henry <address@hidden>
I think the better things to do is to pass the user as parameters (and check each call to be sure the parameters is correctly passed)
It's done like that in all class CRUD method. Global $user should be avoid into class method.

Florian HENRY
+33 6 03 76 48 07
Twitter : @_Open_Concept_
Secretaire de l'association Dolibarr
Le 10/06/2013 09:55, Maxime Kohlhaas a écrit :
Hi all,

Just wondering why the FactureLigne update method has $user as first parameter... Propal and Commande don't have it, and the $user var is defined by global just after.

Should I remove this parameter and check where this function is called ?


Maxime Kohlhaas
Consultant associé
ATM Consulting
+33 6 33 42 92 43

Dolibarr-dev mailing list

Dolibarr-dev mailing list

reply via email to

[Prev in Thread] Current Thread [Next in Thread]