[PEPr] +1 for Web Services::Services_Yahoo_JP

Philippe Jausions (http://pear.php.net/user/jausions) has voted +1 on the proposal for Web Services::Services_Yahoo_JP. Proposal information: http://pear.php.net/pepr/pepr-proposal-show.php?id=545 Vote information: http://pear.php.net/pepr/pepr-vote-show.php?id=545&handle=jausions Comment: Some comments: - What does MA stand for? Is it defined in the Yahoo! API as simply "MA"? I couldn't figure it out from the code. Put some explicative comments in the class header. - Some private properties are missing docblocks - Some "if" blocks are missing {} (make sure to run PHP_CodeSniffer) - Some files don't match the class name, for instance Services/Yahoo/JP/MA/parse.php for Services_Yahoo_JP_MA_Parse (parse vs Parse) - The factory() methods are not extensible as the possible types are hardcoded. This closes the door to other users wanting to add their own implementations and/or features. - It may be convenient to have a way to register the "appid" once and to set it as default (not very important :-) - When you extend classes from another package, make sure to call parent::__construct(), if applicable -- Sent by PEPr, the automatic proposal system at http://pear.php.net