ChangeSet ID: 31049 CVSROOT: /opt/cvs-commit Module name: appdb Changes by: wineowner@winehq.org 2007/04/17 18:28:10
Modified files: include : testData.php
Log message: Alexander Nicolaysen Sørnes alex@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