Not bothered - if it helps the devs do their job the it is ok by me.John Wheater wrote: Do we think this comment is OK?:
w:\forum\admin\admin_users.php(352): // Awww, the user wants to change their password, isn't that cute..
What is wrong with this - it is a perfectly valid php statementJohn Wheater wrote: w:\forum\admin\admin_users.php(354): if($password != $password_confirm)
CyberAlien wrote: Comments like that make editing phpBB code much more fun, which helps a lot when doing some commercial work for customers. Please add more such comments!
just.xTc wrote: All of you just made me hate myself for never commenting my code I want to join the fun, too!
// Strip all tags from data ... may p**s some people off, bah, strip_tags is
// doing the job but can still break HTML output ... have no choice, have
// to use htmlspecialchars ... be prepared to be moaned at.
just.xTc wrote: MOOOOOOOAAAAAN......
xD
But why is htmlspecialchars a problem anyway?