[PEPr] Comment on Web Services::Services_TinyURL
| From: | Chuck Burgess | Date: | Tue, 11 Sep 2007 02:25:09 +0000 |
| Subject: | [PEPr] Comment on Web Services::Services_TinyURL | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-47960@lists.php.net to get a copy of this message | ||
Chuck Burgess (http://pear.php.net/user/ashnazg) has commented on the proposal for Web
Services::Services_TinyURL.
Comment:
Looks like a nice concise package to do the straightforward job of setting
& getting TinyURLs.
Your package.xml says BSD license, though all the code files show the PHP
license, both in @license tag and in license text block.
Granted, this is only the second proposal I've ever looked at, but I would
have expected to see the PEAR/Exception be required by the Exception class
file, and the regular class file in turn requiring the TinyURL_Exception
file. That's just an observation... I don't know what the coding standards
say on that.
I can't seem to get the PHPTs to pass. I have Validate 0.7.0 and the curl
extension installed, using PHP 5.2.4.
Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=505
--
Sent by PEPr, the automatic proposal system at http://pear.php.net