[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