Re: [Dolibarr-dev] Facture class

[Destailleur Laurent]

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
address@hidden
+33 6 03 76 48 07
http://www.open-concept.pro
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 ?

Regards,

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

_______________________________________________
Dolibarr-dev mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev

_______________________________________________
Dolibarr-dev mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev