appdb/include maintainer.php

[WineHQ]

ChangeSet ID:	31300
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner at winehq.org	2007/07/31 21:19:33

Modified files:
	include        : maintainer.php 

Log message:
	Alexander Nicolaysen Sørnes <alex at thehandofagony.com>, Chris Morgan <cmorgan at alum.wpi.edu>
	Clean up some html in maintainer notification emails, add information about who the email is
	addressed to so there is some way to see who the notification system is notifying.

Patch: http://cvs.winehq.org/patch.py?id=31300

Old revision  New revision  Changes     Path
 1.51          1.52          +9 -8       appdb/include/maintainer.php

Index: appdb/include/maintainer.php
diff -u -p appdb/include/maintainer.php:1.51 appdb/include/maintainer.php:1.52
--- appdb/include/maintainer.php:1.51	1 Aug 2007  2:19:33 -0000
+++ appdb/include/maintainer.php	1 Aug 2007  2:19:33 -0000
@@ -930,20 +930,21 @@ class maintainer
 
       // construct the subject of the notification email that we may send
       $sSubject = "Notification of queued data for ";
-      $sMsg = "You are receiving this email to notify you that there is queued data";
+      $sMsg = "";
+      $sMsg.= "Hello ".$oUser->sRealname."<".$oUser->sEmail.">".".\n\n";
+      $sMsg.= "You are receiving this email to notify you that there is queued data";
       $sMsg.=" for the ";
       if($this->bSuperMaintainer)
       {
         $oApp = new Application($this->iAppId);
         $sSubject.= $oApp->sName;
-        $sMsg.='application, '.$oApp->objectMakeLink().', that you maintain.'."\n";
+        $sMsg.='application, '.$oApp->sName.'('.$oApp->objectMakeUrl().'), that you maintain.'."\n";
       } else
       {
         $sFullname = version::fullName($this->iVersionId);
         $oVersion = new Version($this->iVersionId);
         $sSubject.= $sFullname;
-        $sMsg.='version, <a href="'.$oVersion->objectMakeUrl().'">'.$sFullname.
-          '</a>, that you maintain.'."\n";
+        $sMsg.='version, '.$sFullname.'('.$oVersion->objectMakeUrl().'), that you maintain.'."\n";
       }
       $sSubject.=" ready for your processing";
 
@@ -964,8 +965,8 @@ class maintainer
       {
         $oVersion = new Version($iVersionId);
           
-        $sMsg .= 'Version <a href="'.$oVersion->objectMakeUrl().'">'.$sFullname.
-          '</a> is queued and ready for processing.';
+        $sMsg .= 'Version '.$sFullname.' ('.$oVersion->objectMakeUrl().'">'.$sFullname.
+          ') is queued and ready for processing.';
 
         $iSubmitTime = strtotime($oVersion->sSubmitTime);
 
@@ -981,8 +982,8 @@ class maintainer
       {
         // FIXME: should use a function to generate these urls and use it here and
         // in sidebar_maintainer.php and sidebar_admin.php
-        $sMsg = 'Please visit <a href="'.BASE."objectManager.php?sClass=version_queue&bIsQueue=true&sTitle=".
-          "Version%20Queue".'">AppDB Version queue</a> to process queued versions for applications you maintain.\n';
+        $sMsg = 'Please visit the test data queue ('.APPDB_ROOT."objectManager.php?sClass=version_queue&bIsQueue=true&sTitle=".
+          "Version%20Queue".') to process queued versions for applications you maintain.\n';
       }
 
       //////////////////