Changeset 9765 for trunk/src/bp-members/bp-members-admin.php

Timestamp:

04/17/2015 01:27:06 PM (10 years ago)

Author:

boonebgorges

Message:

Improved plural forms for localizable strings.

Strings that are intended for use only in the case of a single item should not
be bundled with strings intended for use with plurals, as this pattern does not
work with languages that use the singular nominal form for numbers other than
one. See ​https://core.trac.wordpress.org/ticket/28502#comment:23.

This changeset also fixes a few places where a formatted string, rather than a
raw integer, was being passed as the $count value to _n().

Props SergeyBiryukov.
Fixes #6368.

File:

• trunk/src/bp-members/bp-members-admin.php (modified) (1 diff)

trunk/src/bp-members/bp-members-admin.php

@@ -1895 +1895 @@
             case 'delete' :
                 $header_text = __( 'Delete Pending Accounts', 'buddypress' );
-                $helper_text = _n( 'You are about to delete the following account:', 'You are about to delete the following accounts:', count( $signup_ids ), 'buddypress' );
+                if ( 1 == count( $signup_ids ) ) {
+                    $helper_text = __( 'You are about to delete the following account:', 'buddypress' );
+                } else {
+                    $helper_text = __( 'You are about to delete the following accounts:', 'buddypress' );
+                }
                 break;
 
             case 'activate' :
                 $header_text = __( 'Activate Pending Accounts', 'buddypress' );
-                $helper_text = _n( 'You are about to activate the following account:', 'You are about to activate the following accounts:', count( $signup_ids ), 'buddypress' );
+                if ( 1 == count( $signup_ids ) ) {
+                    $helper_text = __( 'You are about to activate the following account:', 'buddypress' );
+                } else {
+                    $helper_text = __( 'You are about to activate the following accounts:', 'buddypress' );
+                }
                 break;
 
             case 'resend' :
                 $header_text = __( 'Resend Activation Emails', 'buddypress' );
-                $helper_text = _n( 'You are about to resend an activation email to the following account:', 'You are about to resend activation emails to the following accounts:', count( $signup_ids ), 'buddypress' );
+                if ( 1 == count( $signup_ids ) ) {
+                    $helper_text = __( 'You are about to resend an activation email to the following account:', 'buddypress' );
+                } else {
+                    $helper_text = __( 'You are about to resend an activation email to the following accounts:', 'buddypress' );
+                }
                 break;
         }