Re: AW: [PEAR-DEV] huh ? UDDI not in CVS
| From: | Alan Knowles | Date: | Wed, 20 Aug 2003 00:33:26 +0000 |
| Subject: | Re: AW: [PEAR-DEV] huh ? UDDI not in CVS | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-20113@lists.php.net to get a copy of this message | ||
There are a number of issues with this..
a) it installs in UDDI/UDDI.php
- it should be UDDI.php
b) coding standards - this should be if ... { }
fputs($fp, $header)
or PEAR::raiseError("Couldn't send HTTP headers.");
c) I'm officially -1 on this package. it is just a SOAP call hard coded.., and could be done using existing packages with less than 5 lines of code..
I serriously suggest removing it..
Regards
Alan
Stephan Schmidt wrote:
Hi,Btw its shitty code, did it get really approved?Has been proposed one week ago, got about four +1's. I aksed Christian to implement PEAR Error Management before releasing it :-( Stephan