[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