appdb/include version.php

[WineHQ]

ChangeSet ID:	31264
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner(a)winehq.org	2007/07/23 15:39:49

Modified files:
	include        : version.php 

Log message:
	Chris Morgan <cmorgan(a)alum.wpi.edu>
	Add check for empty tested ratings to the conditions upon which we will skip trying to color
	using the tested rating. Remove two unused variables.

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

Old revision  New revision  Changes     Path
 1.137         1.138         +3 -4       appdb/include/version.php

Index: appdb/include/version.php
diff -u -p appdb/include/version.php:1.137 appdb/include/version.php:1.138
--- appdb/include/version.php:1.137	23 Jul 2007 20:39:49 -0000
+++ appdb/include/version.php	23 Jul 2007 20:39:49 -0000
@@ -1112,20 +1112,19 @@ class version {
                     $oTableRowHighlight = null;
 
                     // if we have a valid tested rating
-                    if($oVersion->sTestedRating && $oVersion->sTestedRating != "/")
+                    if($oVersion->sTestedRating && ($oVersion->sTestedRating != "/") &&
+                       ($oVersion->sTestedRating != " "))
                     {
-                        $sRatingColor = "class=\"$oVersion->sTestedRating\"";
                         $sClass = $oVersion->sTestedRating;
 
                         $oInactiveColor = new Color();
-                        $oInactiveColor->setColorByName($oVersion->sTestedRating);
+                        $oInactiveColor->SetColorByName($oVersion->sTestedRating);
 
                         $oHighlightColor = GetHighlightColorFromInactiveColor($oInactiveColor);
 
                         $oTableRowHighlight = new TableRowHighlight($oHighlightColor, $oInactiveColor);
                     } else
                     {
-                        $sRatingColor = "class=\"$bgcolor\"";
                         $sClass = $bgcolor;
 
                         $oTableRowHighlight = GetStandardRowHighlight($c);