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, *Marcos García* marcos...@gmail.com 2014-09-22 1:08 GMT+02:00 Destailleur Laurent <e...@destailleur.fr>: > 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 <marcos...@gmail.com>: > > 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 > > > > marcos...@gmail.com > > > > > > _______________________________________________ > > Dolibarr-dev mailing list > > Dolibarr-dev@nongnu.org > > 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 > Dolibarr-dev@nongnu.org > https://lists.nongnu.org/mailman/listinfo/dolibarr-dev >
_______________________________________________ Dolibarr-dev mailing list Dolibarr-dev@nongnu.org https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
