Pwirth has uploaded a new change for review.

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

Change subject: Fixed prepare message with html footer
......................................................................

Fixed prepare message with html footer

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


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpiceFoundation 
refs/changes/31/166731/1

diff --git a/includes/Mailer.class.php b/includes/Mailer.class.php
index f6dcc59..a27b883 100644
--- a/includes/Mailer.class.php
+++ b/includes/Mailer.class.php
@@ -107,23 +107,23 @@
                $sCombinedSubject = '['.$wgSitename.'] '.$sSubject;
 
                //Prepare message
+               if ( $this->bSendHTML ) {
+                       //http(s)://link -> <a 
href="http(s)://link>http(s)://link</a>"
+                       //! already followed by </a>
+                       //last char ! "."
+                       $sMsg = preg_replace(
+                               "#(\s|/>)(https?://[^\s]+?)\.?([\s|<])#",
+                               '<a href="$2">$2</a>',
+                               $sMsg
+                       );
+               }
+
                $sFooter = ( $this->bSendHTML ) ? "<br /><br 
/>---------------------<br /><br />" : "\n\n---------------------\n\n";
                $sFooter .= wfMessage( 'bs-email-footer', $wgSitename 
)->plain() . ( $this->bSendHTML )
                        ? "<br /><br />---------------------"
                        : "\n\n---------------------";
 
                $sCombinedMsg = $sMsg.$sFooter;
-
-               if ( $this->bSendHTML ) {
-                       //http(s)://link -> <a 
href="http(s)://link>http(s)://link</a>"
-                       //! already followed by </a>
-                       //last char ! "."
-                       $sCombinedMsg = preg_replace(
-                               "#(\s|/>)(https?://[^\s]+?)\.?([\s|<])#",
-                               '<a href="$2">$2</a>',
-                               $sCombinedMsg
-                       );
-               }
 
                foreach ( $aEmailTo as $aReceiver ) {
                        //Prepare message

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3f3a7ed5c2d6f5279d2ef231acf9cf9279c043e6
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceFoundation
Gerrit-Branch: master
Gerrit-Owner: Pwirth <[email protected]>

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

Reply via email to