[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