appdb/include testData.php
WineHQ
wineowner at wine.codeweavers.com
Sun Apr 22 14:38:48 CDT 2007
ChangeSet ID: 31089
CVSROOT: /opt/cvs-commit
Module name: appdb
Changes by: wineowner at winehq.org 2007/04/22 14:38:48
Modified files:
include : testData.php
Log message:
Alexander Nicolaysen Sørnes <alex at thehandofagony.com>
State what was changed in the e-mail when testData is updated
Patch: http://cvs.winehq.org/patch.py?id=31089
Old revision New revision Changes Path
1.46 1.47 +60 -1 appdb/include/testData.php
Index: appdb/include/testData.php
diff -u -p appdb/include/testData.php:1.46 appdb/include/testData.php:1.47
--- appdb/include/testData.php:1.46 22 Apr 2007 19:38:48 -0000
+++ appdb/include/testData.php 22 Apr 2007 19:38:48 -0000
@@ -109,6 +109,65 @@ class testData{
if($this == $oOldTest)
return TRUE;
+ /* Provide some feedback as to what was changed. Not all fields are
+ interesting */
+ $sWhatChanged = "";
+ if($this->shWhatWorks != $oOldTest->shWhatWorks)
+ {
+ $sWhatChanged .= "What works was changed from\n'$oOldTest->shWhatWorks'\n".
+ "to\n'$this->shWhatWorks'.\n";
+ }
+
+ if($this->shWhatDoesnt != $oOldTest->shWhatDoesnt)
+ {
+ $sWhatChanged .= "What does not work was changed from\n'"
+ .$oOldTest->shWhatDoesnt."'\n to\n'$this->shWhatDoesnt'.\n";
+ }
+
+ if($this->shWhatNotTested != $oOldTest->shWhatNotTested)
+ {
+ $sWhatChanged .= "What was not tested was changed from\n'".
+ $oOldTest->shWhatNotTested."'\nto\n'$this->shWhatNotTested'.\n";
+ }
+
+ if($this->sComments != $oOldTest->sComments)
+ {
+ $sWhatChanged .= "Extra comments was changed from\n'".
+ $oOldTest->sComments."'\nto\n'$this->sComments'.\n";
+ }
+
+ if($this->iDistributionId != $oOldTest->iDistributionId)
+ {
+ $oNewDist = new distribution($this->iDistributionId);
+ $oOldDist = new distribution($oOldTest->iDistributionId);
+ $sWhatChanged .= "Distribution was changed from $oOldDist->sName ".
+ "to $oNewDist->sName.\n";
+ }
+
+ if($this->sInstalls != $oOldTest->sInstalls)
+ {
+ $sWhatChanged .= "Installs? was changed from $oOldTest->sInstalls to ".
+ "$this->sInstalls.\n";
+ }
+
+ if($this->sRuns != $oOldTest->sRuns)
+ {
+ $sWhatChanged .= "Runs? was changed from $oOldTest->sRuns to ".
+ "$this->sRuns.\n";
+ }
+
+ if($this->sTestedRating != $oOldTest->sTestedRating)
+ {
+ $sWhatChanged .= "Rating was changed from $oOldTest->sTestedRating ".
+ "to $this->sTestedRating.\n";
+ }
+
+ if($this->sTestedRelease != $oOldTest->sTestedRelease)
+ {
+ $sWhatChanged .= "Tested release was changed from ".
+ $oOldTest->sTestedRelease." to $this->stestedRelease.\n";
+ }
+
if(query_parameters("UPDATE testResults SET
versionId = '?',
whatWorks = '?',
@@ -136,7 +195,7 @@ class testData{
$this->iTestingId))
{
if(!$bSilent)
- $this->SendNotificationMail("edit");
+ $this->SendNotificationMail("edit", $sWhatChanged);
return true;
}
else
More information about the wine-cvs
mailing list