small fixes, typo, etc

2 files changed, 19 insertions, 7 deletions

diff --git a/index.php b/index.php
index 0bb04d4..8b91b69 100644
--- a/index.php
+++ b/index.php
@@ -62,6 +62,7 @@ if(myisset("new"))
       {
 	echo "couldn't find one of the names, please start a new game";
 	output_footer();
+	DB_close();
 	exit();
       }
     
@@ -80,6 +81,7 @@ if(myisset("new"))
       {
 	echo "Error defining ruleset: $ruleset";
 	output_footer();
+	DB_close();
 	exit();
       };
     
@@ -172,6 +174,7 @@ else if(myisset("cancle","me"))
 	echo "Can't find you in the database, please check the url.<br />\n";
 	echo "perhaps the game has been cancled, check by login in <a href=\"$host\">here</a>.";
 	output_footer();
+	DB_close();
 	exit();
       }
     
@@ -217,6 +220,7 @@ else if(myisset("me"))
 	echo "Can't find you in the database, please check the url.<br />\n";
 	echo "perhaps the game has been cancled, check by login in <a href=\"$host\">here</a>.";
 	output_footer();
+	DB_close();
 	exit();
       }
 
@@ -531,6 +535,7 @@ else if(myisset("me"))
 	      echo "The game has been canceled because ".DB_get_name_by_userid($nines).
 		" has five or more nines and nobody is playing solo.\n";
 	      output_footer();
+	      DB_close();
 	      exit();
 	    }
 	  else if($poverty==1)
@@ -900,6 +905,7 @@ else if(myisset("me"))
 	  
 	  echo "<p style=\"background-color:red\";>Game $gameid has been cancled.<br /><br /></p>";
 	  output_footer();
+	  DB_close();
 	  exit();
 	}
 
@@ -1539,6 +1545,7 @@ else if(myisset("me"))
       echo "error in testing the status";
     }
     output_footer();
+    DB_close();
     exit();
  } 
 /* user status page */ 
@@ -1667,15 +1674,15 @@ else if(myisset("me"))
 	       
 	       
 	       echo "<p>and these are your games that are already done:<br />Game: \n";
+	       $output=array();
 	       $result = mysql_query("SELECT hash,game_id from Hand WHERE user_id='$uid' AND status='gameover'" );
 	       while( $r = mysql_fetch_array($result,MYSQL_NUM))
-		 echo "<a href=\"".$host."?me=".$r[0]."\">#".$r[1]." </a>, ";
-	       echo "</p>\n";
+		 $output[]= "<a href=\"".$host."?me=".$r[0]."\">#".$r[1]." </a>";
+	       echo implode(", ",$output)."</p>\n";
 	       
 	       $names = DB_get_all_names();
 	       echo "<p>registered players:<br />\n";
-	       foreach ($names as $name)
-		 echo "$name, \n";
+	       echo implode(", ",$names)."\n";
 	       echo "</p>\n";
 	       
 	       echo "<p>Want to start a new game? Visit <a href=\"".$host."?new\">this page.</a></p>";
@@ -1687,6 +1694,7 @@ else if(myisset("me"))
 	 }
      };
      output_footer();
+     DB_close();
      exit();
    }
 /* page for registration */
diff --git a/output.php b/output.php
index 0871bcc..9cadcb0 100644
--- a/output.php
+++ b/output.php
@@ -64,8 +64,7 @@ function output_user_settings($email,$password)
 function output_register()
 {
   echo "IMPORTANT: passwords are going over the net as clear text, so pick an easy password. No need to pick anything complicated here ;)<br /><br />";
-  echo "TODO: convert timezone into a menu<br />\n";
-  echo "TODO: figure out a way to handle passwords <br />\n";
+  echo "TODO: figure out a better way to handle passwords <br />\n";
   ?>
         <form action="index.php" method="post">
           <fieldset>
@@ -83,7 +82,12 @@ function output_register()
               </tr><tr>
 	       <td><label for="Rtimezone">Timezone:</label></td>
                <td>
-                  <input type="text" id="Rtimezone" name="Rtimezone" size="4" maxsize="4" value="+1" />
+		  <select id="Rtimezone" name="Rtimezone" size="1">
+                     <option value="1">Berlin</option>
+                     <option value="-8">Berkeley</option>
+                     <option value="13">Wellington</option>
+		  </select>
+	         (If your timezone is not listed, just select whatever you want and email the admin your correct time zone.)
 	       </td>
               </tr><tr>
                <td colspan="2"> <input type="submit" value="register" /></td>