From 2216a83e02bb120313a78f315c640750152cba46 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alexander=20Nicolaysen=20S=C3=B8rnes?=
 <alex@thehandofagony.com>
Date: Sun, 21 Jan 2007 18:22:41 +0000
Subject: [PATCH] Small fixes for license handling - Rename MakeLicenseList to
 makeLIcenseList - Print the selected argument once - Add optional parameter
 for matching selected license

---
 include/version.php | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/include/version.php b/include/version.php
index 7b4f7fb..7409c14 100644
--- a/include/version.php
+++ b/include/version.php
@@ -609,7 +609,7 @@ class Version {
         // version license
         echo html_tr(array(
             array("<b>License</b>", "class=\"color0\""),
-            $this->MakeLicenseList()));
+            $this->makeLicenseList()));
 
         // version description
         echo '<tr valign=top><td class=color0><b>Version description</b></td>',"\n";
@@ -1083,8 +1083,11 @@ class Version {
         return $sResult;
     }
 
-    function MakeLicenseList()
+    function makeLicenseList($sLicense = NULL)
     {
+        if(!$sLicense)
+            $sLicense = $this->sLicense;
+
         $sReturn = "<select name=\"sLicense\">\n";
         $sReturn .= "<option value=\"\">Choose . . .</option>\n";
         $aLicense = array(LICENSE_RETAIL, LICENSE_OPENSOURCE, LICENSE_FREEWARE,
@@ -1093,8 +1096,10 @@ class Version {
 
         for($i = 0; $i < $iMax; $i++)
         {
-            if($aLicense[$i] == $this->sLicense)
+            if($aLicense[$i] == $sLicense)
                 $sSelected = " selected=\"selected\"";
+            else
+                $sSelected = "";
 
                 $sReturn .= "<option value=\"$aLicense[$i]\"$sSelected>".
                             "$aLicense[$i]</option>\n";