appdb/include testData.php

WineHQ wineowner at wine.codeweavers.com
Tue Apr 17 18:28:10 CDT 2007


ChangeSet ID:	31049
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner at winehq.org	2007/04/17 18:28:10

Modified files:
	include        : testData.php 

Log message:
	Alexander Nicolaysen Sørnes <alex at thehandofagony.com>
	Fix mails for updated test results. Pass the correct 'edit' parameter to
	sendNotificationMail() and only send mail if the test data has been changed.

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

Old revision  New revision  Changes     Path
 1.39          1.40          +6 -1       appdb/include/testData.php

Index: appdb/include/testData.php
diff -u -p appdb/include/testData.php:1.39 appdb/include/testData.php:1.40
--- appdb/include/testData.php:1.39	17 Apr 2007 23:28:10 -0000
+++ appdb/include/testData.php	17 Apr 2007 23:28:10 -0000
@@ -100,6 +100,11 @@ class testData{
             return;
         }
 
+        $oOldTest = new testData($this->iTestingId);
+        /* Nothing changed */
+        if($this == $oOldTest)
+            return TRUE;
+
         if(query_parameters("UPDATE testResults SET 
                                         versionId       = '?',
                                         whatWorks       = '?',
@@ -127,7 +132,7 @@ class testData{
                             $this->iTestingId))
         {
             if(!$bSilent)
-                $this->SendNotificationMail();
+                $this->SendNotificationMail("edit");
             return true;
         }
         else



More information about the wine-cvs mailing list