Changeset 9047 for trunk/src/bp-groups/bp-groups-template.php

Timestamp:

09/27/2014 12:49:45 AM (11 years ago)

Author:

boonebgorges

Message:

Use _n() for translations in bp_get_group_member_count()

The previous technique (manually switching between 1 and not-1) does not
provide adequate internationalizability for all languages.

Adds unit tests.

File:

• trunk/src/bp-groups/bp-groups-template.php (modified) (1 diff)

trunk/src/bp-groups/bp-groups-template.php

@@ -1634 +1634 @@
         global $groups_template;
 
-        if ( 1 == (int) $groups_template->group->total_member_count )
-            return apply_filters( 'bp_get_group_member_count', sprintf( __( '%s member', 'buddypress' ), bp_core_number_format( $groups_template->group->total_member_count ) ) );
-        else
-            return apply_filters( 'bp_get_group_member_count', sprintf( __( '%s members', 'buddypress' ), bp_core_number_format( $groups_template->group->total_member_count ) ) );
+        if ( isset( $groups_template->group->total_member_count ) ) {
+            $count = (int) $groups_template->group->total_member_count;
+        } else {
+            $count = 0;
+        }
+
+        $count_string = sprintf( _n( '%s member', '%s members', $count, 'buddypress' ), bp_core_number_format( $count ) );
+
+        return apply_filters( 'bp_get_group_member_count', $count_string );
     }