CSteipp has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/52092


Change subject: Add username for GENDER in LogFormatter only when visible
......................................................................

Add username for GENDER in LogFormatter only when visible

For sanity.

bug:42621
Change-Id: If133597f06edcfdda4ad7a47506635743c80cd53
---
M includes/logging/LogFormatter.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/92/52092/1

diff --git a/includes/logging/LogFormatter.php 
b/includes/logging/LogFormatter.php
index eb8779f..4d3565d 100644
--- a/includes/logging/LogFormatter.php
+++ b/includes/logging/LogFormatter.php
@@ -439,7 +439,7 @@
                $entry = $this->entry;
                $params = $this->extractParameters();
                $params[0] = Message::rawParam( $this->getPerformerElement() );
-               $params[1] = $entry->getPerformer()->getName();
+               $params[1] = $this->canView( LogPage::DELETED_USER ) ? 
$entry->getPerformer()->getName() : '';
                $params[2] = Message::rawParam( $this->makePageLink( 
$entry->getTarget() ) );
 
                // Bad things happens if the numbers are not in correct order

-- 
To view, visit https://gerrit.wikimedia.org/r/52092
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If133597f06edcfdda4ad7a47506635743c80cd53
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: wmf/1.21wmf11
Gerrit-Owner: CSteipp <[email protected]>
Gerrit-Reviewer: Umherirrender <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to