Re: [PEPr] Comment on Web Services::Services_Memotoo

Pequet Thomas wrote: > Pequet Thomas (http://pear.php.net/user/memotoo) has commented on the proposal for Web > Services::Services_Memotoo. > > Comment: > > Thanks for your comments :-) > > Ok I have modify as you say: > - add better documentation for the method > - correct error with phpcs (I have just a warning with php version ?) Things like missing {} blocks for if's, missing spaces around =, <, etc... need to be corrected. Odd, phpcs might be crashing, I'm not getting the error list either. > - write the error in a file if you want, or display these in the browser Let's keep it simple: Please use exception only, no exit, die, echo. Only return boolean false/true when something is actually tested (e.g. isActive()) not as a flag to determine if the action performed ok. I'm guessing Till was referring to using the PEAR Log package, no need to reinvent the wheel here. Repackage the SOAP exception into your own class exception, don't only keep code and message. Exception have much more information that may be helpful in debugging. -Philippe