Robert Vogel has submitted this change and it was merged.

Change subject: BsMailer: Reverted Use of real name commit
......................................................................


BsMailer: Reverted Use of real name commit

* I996a6f4be4e37bfef2bd8c38ca3200209ad8a65b
* Must be username, cause it is needed for GENDER destinction
* However realname is $2

Change-Id: Iad357f51d1a616b53a17a186158b232df6730c68
---
M includes/Mailer.class.php
1 file changed, 4 insertions(+), 7 deletions(-)

Approvals:
  Robert Vogel: Verified; Looks good to me, approved



diff --git a/includes/Mailer.class.php b/includes/Mailer.class.php
index 1c69e62..e419aff 100644
--- a/includes/Mailer.class.php
+++ b/includes/Mailer.class.php
@@ -77,8 +77,8 @@
                        if ( $vReceiver instanceof User ) {
                                if ( $vReceiver->getEmail() ) {
                                        $aEmailTo[] = array(
-                                                       'mail' => new 
MailAddress($vReceiver),
-                                                       'greeting' => 
BsCore::getUserDisplayName( $vReceiver ),
+                                               'mail' => new 
MailAddress($vReceiver),
+                                               'greeting' => 
$vReceiver->getName(),
                                        );
                                }
                        } elseif ( strpos( $vReceiver, '@' ) !== false ) {
@@ -97,7 +97,7 @@
                                if ( $oUser->getEmail() ) {
                                        $aEmailTo[] = array(
                                                'mail' => new MailAddress( 
$oUser ),
-                                               'greeting' => 
BsCore::getUserDisplayName( $vReceiver ),
+                                               'greeting' => 
$vReceiver->getName(),
                                        );
                                }
                        }
@@ -129,10 +129,7 @@
                        //Prepare message
                        if ( $aReceiver['greeting'] ) {
                                $oUser = User::newFromName( 
$aReceiver['greeting'] );
-                               $sRealname = $oUser->getRealName();
-                               if ( empty( $sRealname ) ) {
-                                       $sRealname = $aReceiver['greeting'];
-                               }
+                               $sRealname = BsCore::getUserDisplayName( $oUser 
);
                                $sGreeting = wfMessage( 
'bs-email-greeting-receiver', $aReceiver['greeting'], $sRealname )
                                        ->inLanguage( $oUser->getOption( 
'language' ) )
                                        ->text();

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iad357f51d1a616b53a17a186158b232df6730c68
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceFoundation
Gerrit-Branch: master
Gerrit-Owner: Pwirth <wi...@hallowelt.biz>
Gerrit-Reviewer: Mglaser <gla...@hallowelt.biz>
Gerrit-Reviewer: Pigpen <reym...@hallowelt.biz>
Gerrit-Reviewer: Robert Vogel <vo...@hallowelt.biz>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to