|
From: | Doursenaud , Raphaël |
Subject: | Re: [Dolibarr-dev] setEventMessages instead of setEventMessage |
Date: | Mon, 22 Sep 2014 11:50:01 +0200 |
But what is the problem with having an string or and array in the same argument? An is_array check should be made but I think that is more comfortable than using 2 params... Another way could be using setEventMessage for string and setEventMessages for a set of strings...Regards,2014-09-22 1:08 GMT+02:00 Destailleur Laurent <address@hidden>:If you do so, you must provide a string XOR an array.
In past only string was used. More and more we try to store errors into array.
But for a transition need, we need both parameters (a lot of part of
code currently store error into string AND other errors into array).
So we need to provide both type at same time to function.
> _______________________________________________
2014-09-22 1:00 GMT+02:00 Marcos García <address@hidden>:
> Hi:
>
> I've noticed that setEventMessage is now a deprecated function and
> setEventMessages should be used instead.
>
> Its signature is
> function setEventMessages($mesg, $mesgs, $style='mesgs')
> where $mesg is for a string and $mesgs is filled when we want to attach an
> array of messages.
>
> My question is... why doing this and not doing a setEventMessage where the
> first param can be either an array or a string? I must say I don't really
> like functions that act differently depending on 1st or 2nd param filled.
>
> Regards,
>
>
> Marcos García
>
> address@hidden
>
>
> Dolibarr-dev mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
>
--
Laurent Destailleur (alias Eldy)
------------------------------------------------------------------------------------
Social networks of my OpenSource projects:
Dolibarr Google+: https://plus.google.com/+DolibarrOrg/
Dolibarr Facebook: https://www.facebook.com/dolibarr
Dolibarr Twitter: http://www.twitter.com/dolibarr
AWStats Google+: https://plus.google.com/+AWStatsOrgPoject/
AWStats Facebook: https://www.facebook.com/awstats.org
AWStats Twitter: http://www.twitter.com/awstats_project
_______________________________________________
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
[Prev in Thread] | Current Thread | [Next in Thread] |