useless call
[lhc/web/wiklou.git] / includes / UserMailer.php
index 0eb72c6..0097620 100644 (file)
  * with this program; if not, write to the Free Software Foundation, Inc.,
  * 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
  * http://www.gnu.org/copyleft/gpl.html
- * 
+ *
  * @author <brion@pobox.com>
  * @author <mail@tgries.de>
- * 
+ *
  * @package MediaWiki
  */
 
@@ -48,13 +48,13 @@ function wfRFC822Phrase( $phrase ) {
  * @param string $replyto optional reply-to email (default : false)
  */
 function userMailer( $to, $from, $subject, $body, $replyto=false ) {
-       global $wgUser, $wgSMTP, $wgOutputEncoding, $wgErrorString, $wgEmergencyContact;
-       
+       global $wgUser, $wgSMTP, $wgOutputEncoding, $wgErrorString;
+
        if (is_array( $wgSMTP )) {
                require_once( 'Mail.php' );
-               
+
                $timestamp = time();
-       
+
                $headers['From'] = $from;
                if ( $replyto ) {
                        $headers['Reply-To'] = $replyto;
@@ -65,13 +65,13 @@ function userMailer( $to, $from, $subject, $body, $replyto=false ) {
                $headers['Content-transfer-encoding'] = '8bit';
                $headers['Message-ID'] = "<{$timestamp}" . $wgUser->getName() . '@' . $wgSMTP['IDHost'] . '>';
                $headers['X-Mailer'] = 'MediaWiki mailer';
-       
+
                // Create the mail object using the Mail::factory method
                $mail_object =& Mail::factory('smtp', $wgSMTP);
-       
+               wfDebug( "Sending mail via PEAR::Mail\n" );
                $mailResult =& $mail_object->send($to, $headers, $body);
-               
-               # Based on the result return an error string, 
+
+               # Based on the result return an error string,
                if ($mailResult === true)
                        return '';
                else if (is_object($mailResult))
@@ -93,8 +93,8 @@ function userMailer( $to, $from, $subject, $body, $replyto=false ) {
 
                $wgErrorString = '';
                set_error_handler( 'mailErrorHandler' );
-               # added -f parameter, see PHP manual for the fifth parameter when using the mail function
-               mail( $to, $subject, $body, $headers, " -f {$wgEmergencyContact}\n");
+               wfDebug( "Sending mail via internal mail() function\n" );
+               mail( $to, $subject, $body, $headers );
                restore_error_handler();
 
                if ( $wgErrorString ) {
@@ -130,9 +130,9 @@ function mailErrorHandler( $code, $string ) {
  * same timeoffset in their preferences.
  *
  * Visit the documentation pages under http://meta.wikipedia.com/Enotif
- * 
+ *
  * @package MediaWiki
- * 
+ *
  */
 class EmailNotification {
        /**#@+
@@ -140,9 +140,9 @@ class EmailNotification {
         */
        var $to, $subject, $body, $replyto, $from;
        var $user, $title, $timestamp, $summary, $minorEdit, $oldid;
-       
+
        /**#@-*/
-       
+
        /**
         * @todo document
         * @param $currentPage
@@ -153,40 +153,37 @@ class EmailNotification {
         * @param $oldid (default: false)
         */
        function notifyOnPageChange(&$title, $timestamp, $summary, $minorEdit, $oldid=false) {
-       
+
                # we use $wgEmergencyContact as sender's address
-               global $wgUser, $wgLang, $wgEmergencyContact;
-               global $wgEnotifWatchlist, $wgEnotifMinorEdits;
-               global $wgEnotifUserTalk;
-               global $wgEnotifRevealEditorAddress;
-               global $wgEnotifFromEditor;
-               global $wgEmailAuthentication;
+               global $wgUser, $wgEnotifWatchlist;
+               global $wgEnotifMinorEdits, $wgEnotifUserTalk, $wgShowUpdatedMarker;
 
                $fname = 'UserMailer::notifyOnPageChange';
                wfProfileIn( $fname );
-               
+
                # The following code is only run, if several conditions are met:
                # 1. EmailNotification for pages (other than user_talk pages) must be enabled
                # 2. minor edits (changes) are only regarded if the global flag indicates so
-       
+
                $isUserTalkPage = ($title->getNamespace() == NS_USER_TALK);
                $enotifusertalkpage = ($isUserTalkPage && $wgEnotifUserTalk);
-               $enotifwatchlistpage = (!$isUserTalkPage && $wgEnotifWatchlist);        
+               $enotifwatchlistpage = (!$isUserTalkPage && $wgEnotifWatchlist);
+
 
                if ( ($enotifusertalkpage || $enotifwatchlistpage) && (!$minorEdit || $wgEnotifMinorEdits) ) {
                        $dbr =& wfGetDB( DB_MASTER );
                        extract( $dbr->tableNames( 'watchlist' ) );
-                       $res = $dbr->select( 'watchlist', array( 'wl_user' ), 
+                       $res = $dbr->select( 'watchlist', array( 'wl_user' ),
                                array(
                                        'wl_title' => $title->getDBkey(),
                                        'wl_namespace' => $title->getNamespace(),
                                        'wl_user <> ' . $wgUser->getID(),
-                                       'wl_notificationtimestamp <= 1',
+                                       'wl_notificationtimestamp ' . $dbr->isNullTimestamp(),
                                ), $fname );
 
                        # if anyone is watching ... set up the email message text which is
                        # common for all receipients ...
-                       if ( $dbr->numRows( $res ) > 0 ) { 
+                       if ( $dbr->numRows( $res ) > 0 ) {
                                $this->user &= $wgUser;
                                $this->title =& $title;
                                $this->timestamp = $timestamp;
@@ -196,9 +193,9 @@ class EmailNotification {
 
                                $this->composeCommonMailtext();
                                $watchingUser = new User();
-       
+
                                # ... now do for all watching users ... if the options fit
-                               for ($i = 1; $i <= $dbr->numRows( $res ); $i++) { 
+                               for ($i = 1; $i <= $dbr->numRows( $res ); $i++) {
 
                                        $wuser = $dbr->fetchObject( $res );
                                        $watchingUser->setID($wuser->wl_user);
@@ -212,50 +209,51 @@ class EmailNotification {
 
                                        } # if the watching user has an email address in the preferences
                                }
-                               
-                               # mark the changed watch-listed page with a timestamp, so that the page is 
-                               # listed with an "updated since your last visit" icon in the watch list, ...
-                               $dbw =& wfGetDB( DB_MASTER );
-                               $success = $dbw->update( 'watchlist',
-                                       array( /* SET */
-                                               'wl_notificationtimestamp' => $timestamp
-                                       ), array( /* WHERE */
-                                               'wl_title' => $title->getDBkey(),
-                                               'wl_namespace' => $title->getNamespace(),
-                                       ), 'UserMailer::NotifyOnChange'
-                               );
                        } # if anyone is watching
                } # if $wgEnotifWatchlist = true
+
+               if ( $wgShowUpdatedMarker || $wgEnotifWatchlist ) {
+                       # mark the changed watch-listed page with a timestamp, so that the page is
+                       # listed with an "updated since your last visit" icon in the watch list, ...
+                       $dbw =& wfGetDB( DB_MASTER );
+                       $success = $dbw->update( 'watchlist',
+                               array( /* SET */
+                                       'wl_notificationtimestamp' => $dbw->timestamp($timestamp)
+                               ), array( /* WHERE */
+                                       'wl_title' => $title->getDBkey(),
+                                       'wl_namespace' => $title->getNamespace(),
+                               ), 'UserMailer::NotifyOnChange'
+                       );
+               }
+
        } # function NotifyOnChange
-       
+
        /**
         * @access private
         */
        function composeCommonMailtext() {
-               global $wgLang, $wgUser, $wgEmergencyContact;
-               global $wgEnotifRevealEditorAddress;
-               global $wgEnotifFromEditor;
-               global $wgNoReplyAddress;
-               
+               global $wgUser, $wgEmergencyContact, $wgNoReplyAddress;
+               global $wgEnotifFromEditor, $wgEnotifRevealEditorAddress;
+
                $summary = ($this->summary == '') ? ' - ' : $this->summary;
                $medit   = ($this->minorEdit) ? wfMsg( 'minoredit' ) : '';
-               
+
                # You as the WikiAdmin and Sysops can make use of plenty of
                # named variables when composing your notification emails while
                # simply editing the Meta pages
-               
-               $to      = wfMsgForContent( 'enotif_to' );
+
                $subject = wfMsgForContent( 'enotif_subject' );
                $body    = wfMsgForContent( 'enotif_body' );
                $from    = ''; /* fail safe */
                $replyto = ''; /* fail safe */
                $keys    = array();
-               
+
                # regarding the use of oldid as an indicator for the last visited version, see also
                # http://bugzilla.wikipeda.org/show_bug.cgi?id=603 "Delete + undelete cycle doesn't preserve old_id"
                # However, in the case of a new page which is already watched, we have no previous version to compare
                if( $this->oldid ) {
-                       $keys['$NEWPAGE'] = wfMsgForContent( 'enotif_lastvisited' );
+                       $difflink = $this->title->getFullUrl( 'diff=0&oldid=' . $this->oldid );
+                       $keys['$NEWPAGE'] = wfMsgForContent( 'enotif_lastvisited', $difflink );
                        $keys['$OLDID']   = $this->oldid;
                        $keys['$CHANGEDORCREATED'] = wfMsgForContent( 'changed' );
                } else {
@@ -264,19 +262,15 @@ class EmailNotification {
                        $keys['$OLDID']   = '';
                        $keys['$CHANGEDORCREATED'] = wfMsgForContent( 'created' );
                }
-               
-               $body = strtr( $body, $keys ); 
+
+               $body = strtr( $body, $keys );
                $pagetitle = $this->title->getPrefixedText();
-               
-               $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 );
+               $keys['$PAGETITLE_URL']      = $this->title->getFullUrl();
+               
                $keys['$PAGEMINOREDIT']      = $medit;
                $keys['$PAGESUMMARY']        = $summary;
-       
+
                $subject = strtr( $subject, $keys );
 
                # Reveal the page editor's address as REPLY-TO address only if
@@ -294,36 +288,31 @@ class EmailNotification {
                                $from    = $adminAddress;
                                $replyto = $editorAddress;
                        }
-                       $keys['$PAGEEDITORNAMEANDEMAILADDR'] = $editorAddress;
                } else {
                        $from    = $adminAddress;
                        $replyto = $wgNoReplyAddress;
-                       $keys['$PAGEEDITORNAMEANDEMAILADDR'] = $replyto;
                }
-       
+
                if( $wgUser->isIP( $name ) ) {
                        #real anon (user:xxx.xxx.xxx.xxx)
                        $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;
-                       $keys['%24PAGEEDITORE']       = $anon;
-                       $keys['$PAGEEDITORE']         = $anon;
-                       $keys['$PAGEEDITOR']          = 'anonymous user ' . $name;
+
+                       $keys['$PAGEEDITOR']       = 'anonymous user ' . $name;
+                       $keys['$PAGEEDITOR_EMAIL'] = wfMsgForContent( 'noemailtitle' );
                } else {
                        $subject = str_replace('$PAGEEDITOR', $name, $subject);
-                       $keys['$PAGEEDITOR_RAWURL']   = wfUrlencode( $name );
-                       $keys['%24PAGEEDITOR_RAWURL'] = wfUrlencode( $name );
-                       $keys['%24PAGEEDITORE']       = $wgUser->getTitleKey();
-                       $keys['$PAGEEDITORE']         = $wgUser->getTitleKey();
                        $keys['$PAGEEDITOR']          = $name;
+                       $emailPage = Title::makeTitle( NS_SPECIAL, 'Emailuser/' . $name );
+                       $keys['$PAGEEDITOR_EMAIL'] = $emailPage->getFullUrl();
                }
+               $userPage = $wgUser->getUserPage();
+               $keys['$PAGEEDITOR_WIKI'] = $userPage->getFullUrl();
                $body = strtr( $body, $keys );
-       
+               $body = wordwrap( $body, 72 );
+
                # now save this as the constant user-independent part of the message
-               $this->to      = $to;
                $this->from    = $from;
                $this->replyto = $replyto;
                $this->subject = $subject;
@@ -331,7 +320,7 @@ class EmailNotification {
        }
 
 
-       
+
        /**
         * Does the per-user customizations to a notification e-mail (name,
         * timestamp in proper timezone, etc) and sends it out.
@@ -345,24 +334,20 @@ class EmailNotification {
        function composeAndSendPersonalisedMail( $watchingUser ) {
                global $wgLang;
                // From the PHP manual:
-               //     Note:  The to parameter cannot be an address in the form of "Something <someone@example.com>". 
+               //     Note:  The to parameter cannot be an address in the form of "Something <someone@example.com>".
                //     The mail command will not parse this properly while talking with the MTA.
                $to = $watchingUser->getEmail();
                $body = str_replace( '$WATCHINGUSERNAME', $watchingUser->getName() , $this->body );
-               $body = str_replace( '$WATCHINGUSEREMAILADDR', $watchingUser->getEmail(), $body );
-       
+
                $timecorrection = $watchingUser->getOption( 'timecorrection' );
-               if( !$timecorrection ) {
-                       # fail safe - I prefer it. TomGries
-                       $timecorrection = '00:00';
-               }
+               
                # $PAGEEDITDATE is the time and date of the page change
                # expressed in terms of individual local time of the notification
                # recipient, i.e. watching user
                $body = str_replace('$PAGEEDITDATE',
                        $wgLang->timeanddate( $this->timestamp, true, false, $timecorrection ),
                        $body);
-               
+
                $error = userMailer( $to, $this->from, $this->subject, $body, $this->replyto );
                return ($error == '');
        }