appdb/include version.php

[WineHQ]

ChangeSet ID:	31264
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	[e-mail]	2007/07/23 15:39:49

Modified files:
    include        : version.php

Log message:
    Chris Morgan 
    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);