Ticket #3442: translate_roles.diff

wp-includes/l10n.php

@@ -74 +74 @@
-}
-
-/**
+ * translate_with_context() - Retrieve the translated text and strip context
+ *
+ * If the domain is set in the $l10n global, then the text is run
+ * through the domain's translate method. After it is passed to
+ * the 'gettext' filter hook, along with the untranslated text as
+ * the second parameter.
+ *
+ * If the domain is not set, the $text is just returned.
+ *
+ * @since 2.5
+ * @uses translate()
+ *
+ * @param string $text Text to translate
+ * @param string $domain Domain to retrieve the translated text
+ * @return string Translated text
+ */
+function translate_with_context($text, $domain) {
+        $whole = translate($text, $domain);
+        $last_bar = strrpos($whole, '|');
+        if ( false == $last_bar ) {
+                return $whole;
+        } else {
+                return substr($whole, 0, $last_bar);
+        }
+}
+
+/**
- * __() - Retrieve a translated string
- *
- * __() is a convenience function which retrieves the translated
@@ -130 +157 @@
- * @return string Translated context string without pipe
- */
-function _c($text, $domain = 'default') {
-
-
-
-
-
-
-
+
-}
-
-/**

wp-admin/users.php

@@ -263 +263 @@
-        if ( $role == $_GET['role'] )
-                $class = ' class="current"';
-
+        $name = translate_with_context($name);
-        $name = sprintf(_c('%1$s (%2$s)|user role with count'), $name, $avail_roles[$role]);
-        $role_links[] = "<li><a href=\"users.php?role=$role\"$class>" . $name . '</a>';
-}

wp-admin/includes/schema.php

@@ -267 +267 @@
-
-function populate_roles_160() {
-        // Add roles
-        add_role('administrator', _c('Administrator|User role'));
-        add_role('editor', _c('Editor|User role'));
-        add_role('author', _c('Author|User role'));
-        add_role('contributor', _c('Contributor|User role'));
-        add_role('subscriber', _c('Subscriber|User role'));
-
+        // Dummy gettext calls to get strings in the catalog.
+        _c('Administrator|User role');
+        _c('Editor|User role');
+        _c('Author|User role');
+        _c('Contributor|User role');
+        _c('Subscriber|User role');
+
+        add_role('administrator', 'Administrator|User role');
+        add_role('editor', 'Editor|User role');
+        add_role('author', 'Author|User role');
+        add_role('contributor', 'Contributor|User role');
+        add_role('subscriber', 'Subscriber|User role');
+
-        // Add caps for Administrator role
-        $role = get_role('administrator');
-        $role->add_cap('switch_themes');

wp-admin/includes/template.php

@@ -545 +545 @@
-        } else {
-                $edit = $user_object->user_login;
-        }
+        $role_name = translate_with_context($wp_roles->role_names[$role]);
-        $r = "<tr id='user-$user_object->ID'$style>
-                <td><input type='checkbox' name='users[]' id='user_{$user_object->ID}' class='$role' value='{$user_object->ID}' /></td>
-                <td><strong>$edit</strong></td>
-                <td>$user_object->first_name $user_object->last_name</td>
-                <td><a href='mailto:$email' title='" . sprintf( __('e-mail: %s' ), $email ) . "'>$email</a></td>
-{$wp_roles->role_names[$role]}
+$role_name
-        $r .= "\n\t\t<td>";
-        if ( $numposts > 0 ) {
-                $r .= "<a href='edit.php?author=$user_object->ID' title='" . __( 'View posts by this author' ) . "' class='edit'>";
@@ -891 +892 @@
-function wp_dropdown_roles( $default = false ) {
-        global $wp_roles;
-        $r = '';
-
+
+
-                if ( $default == $role ) // Make default first in list
-                        $p = "\n\t<option selected='selected' value='$role'>$name</option>";
-                else
-                        $r .= "\n\t<option value='$role'>$name</option>";
+        }
-        echo $p . $r;
-}
-