* Moving img_exif to img_metadata, makes more sense if we're going to drop

[lhc/web/wiklou.git] / includes / UserMailer.php

diff --git a/includes/UserMailer.php b/includes/UserMailer.php
index 0870839..f5f27f6 100644 (file)
--- a/includes/UserMailer.php
+++ b/includes/UserMailer.php
@@ -48,8 +48,6 @@
  * case 3:     (should not happen)
  * input:      "name"
  * output:     "quoted-printable(name)"
- * 
- * @package MediaWiki
  */
 function wfQuotedPrintable_name_and_emailaddr( $string ) {
 
@@ -132,7 +130,7 @@ function userMailer( $to, $from, $subject, $body, $replyto=false ) {
 }
 
 /**
- * @package MediaWiki
+ * @todo document
  */
 function mailErrorHandler( $code, $string ) {
        global $wgErrorString;
@@ -304,13 +302,13 @@ class EmailNotification {
                if( $article->mNamespace != NS_MAIN ) {
                        $pagetitle = $wgLang->getNsText( $article->mNamespace ) . ':' . $pagetitle;
                }
-               $subject = str_replace( '$PAGETITLE_QP', wfQuotedPrintable( str_replace( '_', ' ', $pagetitle ) ), $subject);
+               $subject = str_replace( '$PAGETITLE', str_replace( '_', ' ', $pagetitle ) , $subject);
                $keys['%24PAGETITLE_RAWURL'] = wfUrlencode( $pagetitle );
                $keys['$PAGETITLE_RAWURL']   = wfUrlencode( $pagetitle );
                $keys['%24PAGETITLE']        = $pagetitle; # needed for the {{localurl:$PAGETITLE}} in the messagetext, "$" appears here as "%24"
                $keys['$PAGETITLE']          = $pagetitle;
                $keys['$PAGETIMESTAMP']      = $article->mTimestamp;    # this is the raw internal timestamp - can be useful, too
-               $keys['$PAGEEDITDATEUTC']    = $wgLang->timeanddate( $article->mTimestamp, false, false, false, true );
+               $keys['$PAGEEDITDATEUTC']    = $wgLang->timeanddate( $article->mTimestamp, false, false, false );
                $keys['$PAGEMINOREDIT']      = $medit;
                $keys['$PAGESUMMARY']        = $summary;
        
@@ -318,9 +316,9 @@ class EmailNotification {
                # Reveal the page editor's address as REPLY-TO address only if
                # the user has not opted-out and the option is enabled at the
                # global configuration level.
-               $pageeditor_qp = wfQuotedPrintable( $pageeditorUser->getName() );
+               $name    = $pageeditorUser->getName();
                $adminAddress = 'WikiAdmin <' . $wgEmergencyContact . '>';
-               $editorAddress = $pageeditorUser->getName() . ' <' . $pageeditorUser->getEmail() . '>';
+               $editorAddress = $name . ' <' . $pageeditorUser->getEmail() . '>';
                if( $wgEmailNotificationRevealPageEditorAddress
                    && ( $pageeditorUser->getEmail() != '' )
                    && $pageeditorUser->getOption( 'enotifrevealaddr' ) ) {
@@ -337,12 +335,11 @@ class EmailNotification {
                        $keys['$PAGEEDITORNAMEANDEMAILADDR'] = $replyto;
                }
        
-               if( $pageeditorUser->isIP( $pageeditorUser->getName() ) ) {
+               if( $pageeditorUser->isIP( $name ) ) {
                        #real anon (user:xxx.xxx.xxx.xxx)
-                       $subject = str_replace('$PAGEEDITOR_QP', 'anonymous user ' . $pageeditorUser->getName(), $subject);
-                       $name    = $pageeditorUser->getName();
                        $anon    = $name . ' (anonymous user)';
                        $anonUrl = wfUrlencode( $name ) . ' (anonymous user)';
+                       $subject = str_replace('$PAGEEDITOR', 'anonymous user '. $name, $subject);
                        
                        $keys['$PAGEEDITOR_RAWURL']   = $anonUrl;
                        $keys['%24PAGEEDITOR_RAWURL'] = $anonUrl;
@@ -350,13 +347,12 @@ class EmailNotification {
                        $keys['$PAGEEDITORE']         = $anon;
                        $keys['$PAGEEDITOR']          = 'anonymous user ' . $name;
                } else {
-                       $name = $pageeditorUser->getName();
-                       $subject = str_replace('$PAGEEDITOR_QP', $pageeditor_qp, $subject);
+                       $subject = str_replace('$PAGEEDITOR', $name, $subject);
                        $keys['$PAGEEDITOR_RAWURL']   = wfUrlencode( $name );
                        $keys['%24PAGEEDITOR_RAWURL'] = wfUrlencode( $name );
                        $keys['%24PAGEEDITORE']       = $pageeditorUser->getTitleKey();
                        $keys['$PAGEEDITORE']         = $pageeditorUser->getTitleKey();
-                       $keys['$PAGEEDITOR']          = $pageeditorUser->getName();
+                       $keys['$PAGEEDITOR']          = $name;
                }
                $body = strtr( $body, $keys );
        
@@ -364,7 +360,7 @@ class EmailNotification {
                $this->to      = $to;
                $this->from    = $from;
                $this->replyto = $replyto;
-               $this->subject = $subject;
+               $this->subject = wfQuotedprintable($subject);
                $this->body    = $body;
                return $this;
        }
@@ -398,7 +394,7 @@ class EmailNotification {
                # expressed in terms of individual local time of the notification
                # recipient, i.e. watching user
                $body = str_replace('$PAGEEDITDATE',
-                       $wgLang->timeanddate( $article->mTimestamp, true, false, $timecorrection, true),
+                       $wgLang->timeanddate( $article->mTimestamp, true, false, $timecorrection ),
                        $body);
                
                $error = userMailer( $to, $mail->from, $mail->subject, $body, $mail->replyto );