Siebrand has uploaded a new change for review. https://gerrit.wikimedia.org/r/87490
Change subject: Support GENDER for "contribsub2" ...................................................................... Support GENDER for "contribsub2" Also remove BC code that was added in MediaWiki 1.9. Spotted by Protnet at https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Blocklogentry/el/reply Change-Id: I37eefd1de95d6ef3429403cd351db315bc3ab2bf --- M includes/specials/SpecialContributions.php M includes/specials/SpecialDeletedContributions.php M languages/messages/MessagesQqq.php 3 files changed, 5 insertions(+), 27 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core refs/changes/90/87490/1 diff --git a/includes/specials/SpecialContributions.php b/includes/specials/SpecialContributions.php index 614bd3e..65aa07e 100644 --- a/includes/specials/SpecialContributions.php +++ b/includes/specials/SpecialContributions.php @@ -267,19 +267,7 @@ } } - // Old message 'contribsub' had one parameter, but that doesn't work for - // languages that want to put the "for" bit right after $user but before - // $links. If 'contribsub' is around, use it for reverse compatibility, - // otherwise use 'contribsub2'. - // @todo Should this be removed at some point? - $oldMsg = $this->msg( 'contribsub' ); - if ( $oldMsg->exists() ) { - $linksWithParentheses = $this->msg( 'parentheses' )->rawParams( $links )->escaped(); - - return $oldMsg->rawParams( "$user $linksWithParentheses" ); - } - - return $this->msg( 'contribsub2' )->rawParams( $user, $links ); + return $this->msg( 'contribsub2' )->rawParams( $user, $links )->params( $userObj->getName() ); } /** diff --git a/includes/specials/SpecialDeletedContributions.php b/includes/specials/SpecialDeletedContributions.php index 28ca24b..2cf1730 100644 --- a/includes/specials/SpecialDeletedContributions.php +++ b/includes/specials/SpecialDeletedContributions.php @@ -482,16 +482,7 @@ } } - // Old message 'contribsub' had one parameter, but that doesn't work for - // languages that want to put the "for" bit right after $user but before - // $links. If 'contribsub' is around, use it for reverse compatibility, - // otherwise use 'contribsub2'. - $oldMsg = $this->msg( 'contribsub' ); - if ( $oldMsg->exists() ) { - return $oldMsg->rawParams( "$user ($links)" ); - } - - return $this->msg( 'contribsub2' )->rawParams( $user, $links ); + return $this->msg( 'contribsub2' )->rawParams( $user, $links )->params( $userObj->getName() ); } /** diff --git a/languages/messages/MessagesQqq.php b/languages/messages/MessagesQqq.php index e6b5ae3..23cf05f 100644 --- a/languages/messages/MessagesQqq.php +++ b/languages/messages/MessagesQqq.php @@ -6030,10 +6030,9 @@ * {{msg-mw|Tooltip-pt-mycontris}} {{Identical|Contribution}}', 'contribsub2' => 'Contributions for "user" (links). Parameters: -* $1 - any one of the following: -** IP address (if anonymous user) -** username, with a link which points to the user page (if registered user) -* $2 - list of tool links. The list contains a link which has text {{msg-mw|Sp-contributions-talk}} +* $1 is an IP address or a username, with a link which points to the user page (if registered user). +* $2 is list of tool links. The list contains a link which has text {{msg-mw|Sp-contributions-talk}}. +* $3 is a plain text username used for GENDER. {{Identical|For $1}}', 'nocontribs' => 'Used in [[Special:Contributions]] and [[Special:DeletedContributions]]. -- To view, visit https://gerrit.wikimedia.org/r/87490 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I37eefd1de95d6ef3429403cd351db315bc3ab2bf Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: Siebrand <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
