From 9239ba22c382383cd258ad3f36b2bc0dc99664b2 Mon Sep 17 00:00:00 2001 From: Arun Persaud Date: Thu, 8 May 2008 20:40:43 -0700 Subject: CLEANUP: reorganized the 'init' phase of a game moved some code around, so that we test the result of the init phase in the init case statement. Made things a bit clearer and removed one page reload by doing this. Signed-off-by: Arun Persaud --- include/db.php | 17 ++++++-- include/game.php | 126 +++++++++++++++++++----------------------------------- include/stats.php | 2 +- include/user.php | 2 +- 4 files changed, 59 insertions(+), 88 deletions(-) diff --git a/include/db.php b/include/db.php index 8c04828..a9bdffd 100644 --- a/include/db.php +++ b/include/db.php @@ -668,8 +668,6 @@ function DB_set_party_by_hash($hash,$party) function DB_get_PREF($myid) { - global $PREF; - /* Cardset */ $r = DB_query_array("SELECT value from User_Prefs". " WHERE user_id='$myid' AND pref_key='cardset'" ); @@ -696,7 +694,20 @@ function DB_get_PREF($myid) else $PREF["email"]="emailnonaddict"; - return; + return $PREF; +} + +function DB_get_RULES($gameid) +{ + $r = DB_query_array("SELECT * FROM Rulesets". + " LEFT JOIN Game ON Game.ruleset=Rulesets.id ". + " WHERE Game.id='$gameid'" ); + + $RULES["dullen"] = $r[2]; + $RULES["schweinchen"] = $r[3]; + $RULES["call"] = $r[4]; + + return $RULES; } function DB_get_email_pref_by_hash($hash) diff --git a/include/game.php b/include/game.php index 0d4f0a9..35b94cd 100644 --- a/include/game.php +++ b/include/game.php @@ -5,6 +5,7 @@ if(!isset($HOST)) exit; +/* calling game.php only makes sense when we give it a hash for a game */ if(!myisset("me")) { echo "Hmm, you really shouldn't mess with the urls.
\n"; @@ -12,10 +13,9 @@ if(!myisset("me")) DB_close(); exit(); } - $me = $_REQUEST["me"]; -/* test for valid ID */ +/* Ok, got a hash, but is it valid? */ $myid = DB_get_userid('hash',$me); if(!$myid) { @@ -30,7 +30,7 @@ if(!$myid) if(isset($_SESSION["name"])) output_status($_SESSION["name"]); -/* the user had done something, update the timestamp */ +/* the user has done something, update the timestamp */ DB_update_user_timestamp($myid); /* get some information from the DB */ @@ -41,17 +41,11 @@ $mypos = DB_get_pos_by_hash($me); $myhand = DB_get_handid('hash',$me); $session = DB_get_session_by_gameid($gameid); -/* get prefs and save them */ -DB_get_PREF($myid); +/* get prefs and save them in a variable*/ +$PREF = DB_get_PREF($myid); /* get rule set for this game */ -$r = DB_query_array("SELECT * FROM Rulesets". - " LEFT JOIN Game ON Game.ruleset=Rulesets.id ". - " WHERE Game.id='$gameid'" ); - -$RULES["dullen"] = $r[2]; -$RULES["schweinchen"] = $r[3]; -$RULES["call"] = $r[4]; +$RULES = DB_get_RULES($gameid); /* get some infos about the game */ $gametype = DB_get_gametype_by_gameid($gameid); @@ -189,59 +183,52 @@ switch($mystatus) } } case 'init': - + /* here we ask the player if he is sick */ $mycards = DB_get_hand($me); sort($mycards); - - output_check_for_sickness($me,$mycards); - - echo "

Your cards are:
\n"; - foreach($mycards as $card) - display_card($card,$PREF["cardset"]); - echo "

\n"; - - /* move on to the next stage*/ - DB_set_hand_status_by_hash($me,'check'); - break; - - case 'check': - /* ok, user is in the game, saw his cards and selected his vorbehalt - * so first we check what he selected - */ + if(!myisset("solo","wedding","poverty","nines") ) { - /* all these variables have a pre-selected default, - * so we should never get here, - * unless a user tries to cheat ;) - * can also happen if user reloads the page! - */ - echo "

You need to answer the questions.

"; - DB_set_hand_status_by_hash($me,'init'); + output_check_for_sickness($me,$mycards); + + echo "

Your cards are:
\n"; + foreach($mycards as $card) + display_card($card,$PREF["cardset"]); + echo "

\n"; + + break; } else { - /* check if someone selected more than one vorbehalt */ - $Nvorbehalt = 0; - if($_REQUEST["solo"]!="No") $Nvorbehalt++; - if($_REQUEST["wedding"] == "yes") $Nvorbehalt++; - if($_REQUEST["poverty"] == "yes") $Nvorbehalt++; - if($_REQUEST["nines"] == "yes") $Nvorbehalt++; - - if($Nvorbehalt>1) + /* check if someone selected more than one sickness */ + $Nsickness = 0; + if($_REQUEST["solo"]!="No") $Nsickness++; + if($_REQUEST["wedding"] == "yes") $Nsickness++; + if($_REQUEST["poverty"] == "yes") $Nsickness++; + if($_REQUEST["nines"] == "yes") $Nsickness++; + + if($Nsickness>1) { - echo "

You selected more than one vorbehalt, please go back ". + echo "

You selected more than one sickness, please go back ". "and answer the question again.

"; - DB_set_hand_status_by_hash($me,'init'); + + echo "

Your cards are:
\n"; + foreach($mycards as $card) + display_card($card,$PREF["cardset"]); + echo "

\n"; + + break; } else { + /* everything is ok, save what user said and proceed */ echo "

Processing what you selected in the last step..."; /* check if this sickness needs to be handled first */ $gametype = DB_get_gametype_by_gameid($gameid); - $startplayer = DB_get_startplayer_by_gameid($gameid); + $startplayer = DB_get_startplayer_by_gameid($gameid); /* need this to check which solo goes first */ - if( $_REQUEST["solo"]!="No") + if( $_REQUEST["solo"]!="No" ) { /* user wants to play a solo */ @@ -280,45 +267,18 @@ switch($mystatus) " is playing solo, this game will be canceled.
\n"; DB_set_sickness_by_hash($me,"nines"); } - - echo " Ok, done with checking, please go to the next step of the setup.

"; - + + echo "

\n"; + /* move on to the next stage*/ DB_set_hand_status_by_hash($me,'poverty'); - - /* check if everyone has reached this stage, send out email */ - $userids = DB_get_all_userid_by_gameid($gameid); - $ok = 1; - foreach($userids as $user) - { - $userstat = DB_get_hand_status_by_userid_and_gameid($user,$gameid); - if($userstat!='poverty' && $userstat!='play') - { - $ok = 0; - DB_set_player_by_gameid($gameid,$user); - } - }; - if($ok) - { - /* reset player = everyone has to do something now */ - DB_set_player_by_gameid($gameid,NULL); - - foreach($userids as $user) - { - $To = DB_get_email('userid',$user); - $userhash = DB_get_hash_from_gameid_and_userid($gameid,$user); - if($userhash != $me) - { - $message = "Everyone finish the questionary in game ".DB_format_gameid($gameid).", ". - "please visit this link now to continue: \n". - " ".$HOST.$INDEX."?action=game&me=".$userhash."\n\n" ; - mymail($To,$EmailName." finished setup in game ".DB_format_gameid($gameid),$message); - } - }; - }; }; }; - break; + + case 'check': + /* hmm, by reorganizing things a bit, this stage is empty at the moment */ + /* move on to the next stage, but user shouldn't get here anymore anyway */ + DB_set_hand_status_by_hash($me,'poverty'); case 'poverty': /* here we need to check if there is a solo or some other form of sickness. diff --git a/include/stats.php b/include/stats.php index cf9b412..eac8a8e 100644 --- a/include/stats.php +++ b/include/stats.php @@ -14,7 +14,7 @@ if(!$myid) output_status(); -DB_get_PREF($myid); +$PREF = DB_get_PREF($myid); DB_update_user_timestamp($myid); diff --git a/include/user.php b/include/user.php index fa4653d..00426aa 100644 --- a/include/user.php +++ b/include/user.php @@ -101,7 +101,7 @@ if(myisset("forgot")) $_SESSION["name"] = $myname; output_status(); - DB_get_PREF($myid); + $PREF = DB_get_PREF($myid); /* does the user want to change some preferences? */ if(myisset("setpref")) -- cgit v1.2.3-18-g5258