[wp-trac] [WordPress Trac] #11172: Login form template function
WordPress Trac
wp-trac at lists.automattic.com
Mon Jan 18 17:59:51 UTC 2010
#11172: Login form template function
----------------------------+-----------------------------------------------
Reporter: ryan | Owner: beaulebens
Type: task (blessed) | Status: assigned
Priority: normal | Milestone: 3.0
Component: Template | Version: 2.8.5
Severity: normal | Keywords: has-patch
----------------------------+-----------------------------------------------
Comment(by filosofo):
Replying to [comment:21 markmcwilliams]:
> I'm not 100% sure how it removes confusion, I'd be more confused if it
wasn't included, and like I said not everyone knows what to do to add this
check in the actual theme template! - So I;d still say this needs to be
included! :)
It's good programming practice as much as is possible not to have
functions do multiple things, especially when the logic of what they're
doing is based on global variables. When you fail to do that, you end up
with functions that are difficult to debug and test and that are rife with
backwards-compatibility issues.
Besides, don't you think it's somewhat ridiculous to write ''code'' for a
"developer" who doesn't know what he's doing? What he needs is
''education'' not functionality. So educate him with good documentation
and obvious function names, etc. But don't reduce the code itself to the
lowest common denominator.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/11172#comment:22>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list