jenkins-bot has submitted this change and it was merged.

Change subject: Add Username to the Unsubscribe log message
......................................................................


Add Username to the Unsubscribe log message

Bug: T118651
Change-Id: I257ddd0028d7efdcdaa1d82afb352da940650d49
---
M includes/BounceHandlerActions.php
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Reedy: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/BounceHandlerActions.php 
b/includes/BounceHandlerActions.php
index c90498b..ad0bf8f 100644
--- a/includes/BounceHandlerActions.php
+++ b/includes/BounceHandlerActions.php
@@ -149,7 +149,7 @@
                                $caUser->saveSettings();
                                $this->notifyGlobalUser( $bounceUserId, 
$originalEmail );
                                wfDebugLog( 'BounceHandler',
-                                       "Un-subscribed global user 
$originalEmail for exceeding Bounce Limit $this->bounceRecordLimit.\nProcessed 
Headers:\n" .
+                                       "Un-subscribed global user 
{$caUser->getName()} <$originalEmail> for exceeding Bounce Limit 
$this->bounceRecordLimit.\nProcessed Headers:\n" .
                                                $this->formatHeaders( 
$emailHeaders ) . "\nBounced Email: \n$this->emailRaw"
                                );
                                
RequestContext::getMain()->getStats()->increment( 'bouncehandler.unsub.global' 
);
@@ -160,7 +160,7 @@
                        $user->saveSettings();
                        $this->createEchoNotification( $bounceUserId, 
$originalEmail );
                        wfDebugLog( 'BounceHandler',
-                               "Un-subscribed $originalEmail for exceeding 
Bounce limit $this->bounceRecordLimit.\nProcessed Headers:\n" .
+                               "Un-subscribed {$user->getName()} 
<$originalEmail> for exceeding Bounce limit 
$this->bounceRecordLimit.\nProcessed Headers:\n" .
                                        $this->formatHeaders( $emailHeaders ). 
"\nBounced Email: \n$this->emailRaw"
                        );
                        RequestContext::getMain()->getStats()->increment( 
'bouncehandler.unsub.local' );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I257ddd0028d7efdcdaa1d82afb352da940650d49
Gerrit-PatchSet: 5
Gerrit-Project: mediawiki/extensions/BounceHandler
Gerrit-Branch: master
Gerrit-Owner: 01tonythomas <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Reedy <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to