Req #3836 [Opn]: Coding standards overlook topics

Edit report at http://pear.php.net/bugs/bug.php?id=3836&edit=1 ID: 3836 Updated by: wiesemann@php.net Reported By: drewish at katherinehouse dot com Status: Open Id: 3836 Type: Feature/Change Request Package: Documentation PHP Version: Irrelevant New Comment: I'd like to add here, that the CS section in the manual does not state that using die() is not allowed. Previous Comments: ------------------------------------------------------------------------ [2005-03-16 03:37:15] drewish at katherinehouse dot com I can understand the reluctance to start making changes. It's practically inviting religious warfare. That said, I think even acknowledging that there is no agreed upon standard is better than omitting it. ------------------------------------------------------------------------ [2005-03-16 03:03:37] smith at backendmedia dot com I think Toby started an RFC a while back, but he in the end didnt persue it to the end. I also agree this should be in the CS. One of the tricky topics was if non public classes need to reside in their own file as well or not. There are also other topics like multiline conditions that are not in the CS. All in all most people see the CS as a pandoras box in glass. Do not open it to add anything, cause the result could be very unpleasent quickly. ------------------------------------------------------------------------ [2005-03-15 21:13:04] drewish at katherinehouse dot com Description: ------------ I feel that there are two, relatively minor, omissions in the existing PEAR coding standards. Many outside PHP projects adopted the coding standards so it seems to me that you ought to try to be thorough. * Class to file mapping. Should each class be in a file of it's own? Most developers seem to follow this convention but I can't find a definitive answer. * Naming of abstract classes and interfaces. Should incomplete classes have a name to identify them as such? ------------------------------------------------------------------------ -- Edit this bug report at http://pear.php.net/bugs/bug.php?id=3836&edit=1