[PEPr] Comment on RFC::Constant_Contact

Christian Weiske (http://pear.php.net/user/cweiske) has commented on the proposal for RFC::Constant_Contact. Comment: - Call the package Services_ConstantContact. Currently it looks as if it is a package called "Contact" of the "Constant" category. - The extension check has to be done when instantiating the class, not without any reason. Also using "exit" does not allow the programmer to handle this error in code. Throwing a PEAR_Exception would be best, or simply depend on it in package.xml - Don't use tabs but spaces (see PEAR Coding Standards in the manual, and the PHP_CodeSniffer package that checks your code automatically) - You don't urlencode username and password. - Please add parameter descriptions for methods like addMember - I don't know what "postCode" is. Proposal information: http://pear.php.net/pepr/pepr-proposal-show.php?id=471 -- Sent by PEPr, the automatic proposal system at http://pear.php.net