[buddypress-trac] [BuddyPress] #5231: Number format not being applied

buddypress-trac noreply at wordpress.org
Thu Nov 7 15:06:43 UTC 2013


#5231: Number format not being applied
--------------------------+------------------------------
 Reporter:  sgr33n        |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Core          |     Version:
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+------------------------------

Comment (by boonebgorges):

 Yes, you're right.

 Our treatment of number format is inconsistent in the activity/index.php
 template. It's mostly my fault. See #4997 [7177], #5228 [7516].

 I've done an audit of the _count() functions used in this template, and
 applied the following logic:
 - All number formatting should be done in the -template.php files, as
 filters applied to the output (`add_filter( hook, 'bp_core_number_format'
 )`)
 - No number formatting should be done in the template
 - Add format filters to functions where it was missing
 - When creating Groups tabs, use `%s` rather than `%d` to account for
 number formats

 I believe this should be consistent across templates.

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


More information about the buddypress-trac mailing list