[buddypress-trac] [BuddyPress] #2086: Some slugs need to be changeable, add language strings.

[buddypress-trac at lists.automattic.com]

#2086: Some slugs need to be changeable, add language strings.
-------------------------------------------------+-------------------------
 Reporter:  sulley                               |       Owner:
     Type:  enhancement                          |  boonebgorges
 Priority:  major                                |      Status:  assigned
Component:  Core                                 |   Milestone:  1.6
 Severity:  normal                               |     Version:
 Keywords:  dev-feedback has-patch needs-        |  Resolution:
  testing                                        |
-------------------------------------------------+-------------------------

Comment (by boonebgorges):

 Thanks for the helpful feedback, Paul!

 > The slug array could be e.g. array( $slug_key => array( $slug,
 $description= ) ).

 Great idea. I've made the change. It required making the parsing logic a
 bit more complex (needed to wp_parse_args at a lower level) but it seems
 to work nicely. It's set up currently so that plugins would be able to
 filter these values if desired, and they could also be saved in the
 database (if that was needed for some reason), but they'll always fall
 back on those provided in the component constructor.

 > I'd like to vote for using something like: bp_activity_get_slug( 'key' )

 I'm kind of indifferent here. If we really want to make it simple, why not
 just use bp_get_slug( 'activity', $key ) throughout? I personally prefer
 fewer template tags, but I know that we have a convention of setting
 things up so that you never have to pass an argument to a function inside
 of a template file. I didn't change anything in 03.patch, but I can do
 whatever if there's a general consensus. It's trivial to swap out.

 > Probably best to do the bp_parse_component_slugs() stuff in the
 BP_Component class itself.

 Good thinking. I was trying to keep BP_Component clean (so I've left the
 _parse_ function in the global scope instead of making it into a method of
 BP_Component), but you're right that this should be handled there.

 See 03.patch.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/2086#comment:14>
BuddyPress <http://buddypress.org/>
BuddyPress