ChangeSet ID: 26360 CVSROOT: /opt/cvs-commit Module name: appdb Changes by: wineowner@winehq.org 2006/07/09 01:08:00
Modified files: include : distributions.php tables : testResults.sql
Log message: Tony Lambregts tony.lambregts@gmail.com Sort test results by rating
Patch: http://cvs.winehq.org/patch.py?id=26360
Old revision New revision Changes Path 1.11 1.12 +4 -2 appdb/include/distributions.php 1.2 1.3 +1 -1 appdb/tables/testResults.sql
Index: appdb/include/distributions.php diff -u -p appdb/include/distributions.php:1.11 appdb/include/distributions.php:1.12 --- appdb/include/distributions.php:1.11 9 Jul 2006 6: 8: 0 -0000 +++ appdb/include/distributions.php 9 Jul 2006 6: 8: 0 -0000 @@ -50,7 +50,8 @@ class distribution{ { $sQuery = "SELECT testingId FROM testResults - WHERE distributionId = '?'"; + WHERE distributionId = '?' + ORDER BY testedRating;" ; } else /* only let users view test results that aren't queued and for apps that */ /* aren't queued or versions that aren't queued */ { @@ -61,7 +62,8 @@ class distribution{ appFamily.appId = appVersion.appId AND appFamily.queued = 'false' AND appVersion.queued = 'false' AND - distributionId = '?'"; + distributionId = '?' + ORDER BY testedRating;"; }
if($hResult = query_parameters($sQuery, $iDistributionId)) Index: appdb/tables/testResults.sql diff -u -p appdb/tables/testResults.sql:1.2 appdb/tables/testResults.sql:1.3 --- appdb/tables/testResults.sql:1.2 9 Jul 2006 6: 8: 0 -0000 +++ appdb/tables/testResults.sql 9 Jul 2006 6: 8: 0 -0000 @@ -16,7 +16,7 @@ create table testResults ( testedRelease tinytext, installs enum('Yes','No','N/A') NOT NULL default 'Yes', runs enum('Yes','No','Not Installable') NOT NULL default 'Yes', - testedRating tinytext, + testedRating enum('Platinum','Gold','Silver','Bronze','Garbage') NOT NULL, comments text, submitTime timestamp(14) NOT NULL, submitterId int(11) NOT NULL default '0',