[buddypress-trac] [BuddyPress] #4140: URI page router
    buddypress-trac at lists.automattic.com 
    buddypress-trac at lists.automattic.com
       
    Thu Apr 12 13:30:04 UTC 2012
    
    
  
#4140: URI page router
-------------------------+-----------------------------
 Reporter:  foxly        |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Future Release
Component:  Core         |     Version:
 Severity:  major        |  Resolution:
 Keywords:  1.7-early    |
-------------------------+-----------------------------
Comment (by boonebgorges):
 > You'll also notice we use "headings" to break up sections of large
 functions. This makes them much easier to skim when you're looking for
 something.
 +1. I like this.
 > That's just not easy to read.
 > Overall, just notice how much easier it is to go through our code than
 many other files in BP.
 It's understandable that you'd find your own code easier to go through :)
 In any case, it's all pretty subjective. If we want to start a
 conversation about departing from (or modifying) WP coding standards,
 let's not do it in this ticket. It's a minor issue and I don't want to
 detract from substantive discussion of your patch.
-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4140#comment:9>
BuddyPress <http://buddypress.org/>
BuddyPress
    
    
More information about the buddypress-trac
mailing list