[PEPr] -1 for Web Services::Livejournal
| From: | Christian Weiske | Date: | Thu, 16 Mar 2006 21:32:45 +0000 |
| Subject: | [PEPr] -1 for Web Services::Livejournal | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-41814@lists.php.net to get a copy of this message | ||
Christian Weiske (http://pear.php.net/user/cweiske) has voted -1 on the proposal for Web
Services::Livejournal.
Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=360
Vote information:
http://pear.php.net/pepr/pepr-vote-show.php?id=360&handle=cweiske
Comment:
Beside the issues Justin found, there are more things:
- Why are you polluting the global namespace for error constants that can
be defined as class constants?
- Use the pear directory structure in your examples. You do things like
>require_once('../../Services_Livejournal/xxx')< that don't work in
pear-installed environment.
- ... probably some more
The package idea itself is good, but the code not yet.
--
Sent by PEPr, the automatic proposal system at http://pear.php.net