* Tweak userrights messages a bit and add PLURAL for better localization

* Do not show "Member of" if user is not a member.

diff --git a/includes/SpecialUserrights.php b/includes/SpecialUserrights.php
index 0394b7d..2dccb40 100644 (file)
--- a/includes/SpecialUserrights.php
+++ b/includes/SpecialUserrights.php
@@ -267,14 +267,17 @@ class UserrightsForm extends HTMLForm {
         */
        protected function showEditUserGroupsForm( $user, $groups ) {
                global $wgOut, $wgUser;
-               
+
                list( $addable, $removable ) = $this->splitGroups( $groups );
-               
+
                $list = array();
                foreach( $user->getGroups() as $group )
                        $list[] = self::buildGroupLink( $group );
-               $grouplist = implode( ', ', $list );
-               
+
+               $grouplist = '';
+               if( count( $list ) > 0 ) {
+                       $grouplist = '<p>' . wfMsgHtml( 'userrights-groupsmember' ) . ' ' . implode( ', ', $list ) . '</p>';
+               }
                $wgOut->addHTML(
                        Xml::openElement( 'form', array( 'method' => 'post', 'action' => $this->action, 'name' => 'editGroup' ) ) .
                        Xml::hidden( 'user-editname', $user->getName() ) .
@@ -283,7 +286,7 @@ class UserrightsForm extends HTMLForm {
                        Xml::element( 'legend', array(), wfMsg( 'userrights-editusergroup' ) ) .
                        wfMsgExt( 'editinguser', array( 'parse' ),
                                wfEscapeWikiText( $user->getName() ) ) .
-                       '<p>' . wfMsgHtml('userrights-groupsmember') . ' ' . $grouplist . '</p>' .
+                       $grouplist .
                        $this->explainRights() .
                        "<table border='0'>
                        <tr>
@@ -341,17 +344,17 @@ class UserrightsForm extends HTMLForm {
         */
        private function explainRights() {
                global $wgUser, $wgLang;
-               
+
                $out = array();
                list( $add, $remove ) = array_values( $this->changeableGroups() );
-               
+
                if( count( $add ) > 0 )
-                       $out[] = wfMsgExt( 'userrights-available-add', 'parseinline', $wgLang->listToText( $add ) );
+                       $out[] = wfMsgExt( 'userrights-available-add', 'parseinline', $wgLang->listToText( $add ), count( $add ) );
                if( count( $remove ) > 0 )
-                       $out[] = wfMsgExt( 'userrights-available-remove', 'parseinline', $wgLang->listToText( $remove ) );
-                       
+                       $out[] = wfMsgExt( 'userrights-available-remove', 'parseinline', $wgLang->listToText( $remove ), count( $add ) );
+
                return count( $out ) > 0
-                       ? implode( ' ', $out )
+                       ? implode( '<br />', $out )
                        : wfMsgExt( 'userrights-available-none', 'parseinline' );
        }
 

diff --git a/languages/messages/MessagesDe.php b/languages/messages/MessagesDe.php
index b02d848..ca7d1cd 100644 (file)
--- a/languages/messages/MessagesDe.php
+++ b/languages/messages/MessagesDe.php
@@ -721,7 +721,7 @@ Sollte das Problem bestehen bleiben, melde dich ab und danach wieder an.</strong
 'token_suffix_mismatch'     => '<strong>Deine Bearbeitung wurde zurückgewiesen, da dein Browser Zeichen im Bearbeiten-Token verstümmelt hat.
 Eine Speicherung kann den Seiteninhalt zerstören. Dies geschieht bisweilen durch die Benutzung eines anonymen Proxy-Dienstes, der fehlerhaft arbeitet.</strong>',
 'editing'                   => 'Bearbeiten von $1',
-'editinguser'               => 'Bearbeiten von Benutzer „<b>$1</b>“',
+'editinguser'               => 'Bearbeiten von Benutzer „<b>$1</b>“.',
 'editingsection'            => 'Bearbeiten von $1 (Absatz)',
 'editingcomment'            => 'Bearbeiten von $1 (Kommentar)',
 'editconflict'              => 'Bearbeitungskonflikt: $1',
@@ -978,7 +978,7 @@ Wenn du dich mit dem Thema auskennen, kannst du selbst die Seite „[[$1]]“ ve
 'userrights-lookup-user'      => 'Verwalte Gruppenzugehörigkeit',
 'userrights-user-editname'    => 'Benutzername:',
 'editusergroup'               => 'Benutzerrechte bearbeiten',
-'userrights-editusergroup'    => 'Bearbeite Gruppenzugehörigkeit des Benutzers',
+'userrights-editusergroup'    => 'Benutzer-Gruppenzugehörigkeit bearbeiten',
 'saveusergroups'              => 'Gruppenzugehörigkeit speichern',
 'userrights-groupsmember'     => 'Mitglied von:',
 'userrights-groupsremovable'  => 'Entfernbare Gruppen:',
@@ -987,8 +987,8 @@ Wenn du dich mit dem Thema auskennen, kannst du selbst die Seite „[[$1]]“ ve
 Nicht selektierte Gruppen werden nicht geändert. Eine Selektion kann mit '''Strg + Linksklick''' (bzw. Ctrl + Linksklick) entfernt werden.",
 'userrights-reason'           => 'Grund:',
 'userrights-available-none'   => 'Du darst keine Benutzerrechte verändern.',
-'userrights-available-add'    => 'Du darst Benutzer den Grupppen $1 hinzufügen.',
-'userrights-available-remove' => 'Du darst Benutzer aus den Grupppen $1 entfernen.',
+'userrights-available-add'    => 'Du darst Benutzer {{PLURAL:$2|der folgenden Gruppe|den folgenden $2 Grupppen}} hinzufügen: $1.',
+'userrights-available-remove' => 'Du darst Benutzer aus {{PLURAL:$2|der folgenden Gruppe|den folgenden $2 Grupppen}} entfernen: $1.',
 'userrights-no-interwiki'     => 'Du hast keine Berechtigung, die Benutzerrechte in anderen Wikis zu ändern.',
 'userrights-nodatabase'       => 'Die Datenbank $1 ist nicht vorhanden oder nicht lokal.',
 'userrights-nologin'          => 'Du musst dich mit einem Administrator-Benutzerkonto [[Special:Userlogin|anmelden]], um Benutzerrechte zu ändern.',

diff --git a/languages/messages/MessagesEn.php b/languages/messages/MessagesEn.php
index e2188b0..d5a4c1a 100644 (file)
--- a/languages/messages/MessagesEn.php
+++ b/languages/messages/MessagesEn.php
@@ -1338,8 +1338,8 @@ containing all of the search terms will appear in the result).",
 Unselected groups will not be changed. You can deselect a group with CTRL + Left Click',
 'userrights-reason'           => 'Reason for change:',
 'userrights-available-none'   => 'You may not alter group membership.',
-'userrights-available-add'    => 'You can add users to $1.',
-'userrights-available-remove' => 'You can remove users from $1.',
+'userrights-available-add'    => 'You can add users to {{PLURAL:$2|this group|these groups}}: $1.',
+'userrights-available-remove' => 'You can remove users from {{PLURAL:$2|this group|these groups}}: $1.',
 'userrights-no-interwiki'     => 'You do not have permission to edit user rights on other wikis.',
 'userrights-nodatabase'       => 'Database $1 does not exist or is not local.',
 'userrights-nologin'          => 'You must [[Special:Userlogin|log in]] with an administrator account to assign user rights.',