HomePhabricator

bug 44798: Don't abuse comma-separator message

Authored by saper on Mar 5 2013, 2:08 PM.

Description

bug 44798: Don't abuse comma-separator message

Rename all checkuser-log-* messages related
to Special:CheckUserLog entry types to
"checkuser-log-entry-*" to clearly
identify i18n messages used for various checkuser
events. Also the format of those is changed
as described below:

This change to CheckUser.i18n.php is done
automatically using the following sed(1) script:

s/checkuser-log-userips/checkuser-log-entry-userips/g
s/checkuser-log-ipedits/checkuser-log-entry-ipedits/g
s/checkuser-log-ipusers/checkuser-log-entry-ipusers/g
s/checkuser-log-ipedits-xff/checkuser-log-entry-ipedits-xff/g
s/checkuser-log-ipusers-xff/checkuser-log-entry-ipusers-xff/g
s/checkuser-log-useredits/checkuser-log-entry-useredits/g

Timestamp in checkuser-log-entry-* is now included
in the messages as "$3". Update all messages accordingly,
using previously used separator ("comma-separator"), with
  replaced with a literal space. Some messages are
manually tweaked as well.

Also remove redundant check for an empty comment.
The comment cannot be included as $4 as it already
contains pre-formatted HTML.

https://bugzilla.wikimedia.org/show_bug.cgi?id=44798

Bug: 44798
Change-Id: Ife21fd09aa6d0c74241dbd7b9130b10292f306ac

Details