claunia/qemudb
Archived
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Test result count should not include queued tests. Don't display queued test results in test results table.

Browse Source
This commit is contained in:
Tony Lambregts
2006-01-17 02:34:58 +00:00
committed by WineHQ
parent b587dbfd17
commit 9a72e97f53
1 changed files with 29 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
include/testResults.php
View File

@@ -566,6 +566,9 @@ class testData{
{ {
$oTest = new testData($oRow->testingId); $oTest = new testData($oRow->testingId);
$oVersion = new version($oTest->iVersionId); $oVersion = new version($oTest->iVersionId);
// dont show testing results of versions that are still queued.
if ($oVersion->sQueued == 'false')
{
$oApp = new application($oVersion->iAppId); $oApp = new application($oVersion->iAppId);
$oSubmitter = new User($oTest->iSubmitterId); $oSubmitter = new User($oTest->iSubmitterId);
if ($c % 2 == 1) { $bgcolor = 'color0'; } else { $bgcolor = 'color1'; } if ($c % 2 == 1) { $bgcolor = 'color0'; } else { $bgcolor = 'color1'; }
@@ -583,6 +586,9 @@ class testData{
echo "</tr>\n\n"; echo "</tr>\n\n";
$c++; $c++;
} }
}
echo "</table>","\n";
echo html_frame_end(); echo html_frame_end();
} }
@@ -591,7 +597,13 @@ class testData{
/* Get the number of TestResults in the database */ /* Get the number of TestResults in the database */
function getNumberOfQueuedTests() function getNumberOfQueuedTests()
{ {
$hResult = query_appdb("SELECT count(*) as num_tests FROM testResults WHERE queued='true';"); $sQuery = "SELECT count(*) as num_tests
FROM testResults, appVersion
WHERE appVersion.versionId=testResults.versionId
and appVersion.queued='false'
and testResults.queued='true';";
$hResult = query_appdb($sQuery);
if($hResult) if($hResult)
{ {
$row = mysql_fetch_object($hResult); $row = mysql_fetch_object($hResult);