diff options
author | Arun Persaud <arun@nubati.net> | 2012-05-08 20:55:57 -0700 |
---|---|---|
committer | Arun Persaud <arun@nubati.net> | 2012-05-08 21:31:19 -0700 |
commit | d760c835fcf5444dd1a0b2997e6eb5744e0be952 (patch) | |
tree | 02779b6c0ffeb46b096781bc7aa4b180ac833315 /include | |
parent | f11c2bcac54384c4154324e4c72a2b62af35995c (diff) | |
download | e-DoKo-d760c835fcf5444dd1a0b2997e6eb5744e0be952.tar.gz e-DoKo-d760c835fcf5444dd1a0b2997e6eb5744e0be952.tar.bz2 e-DoKo-d760c835fcf5444dd1a0b2997e6eb5744e0be952.zip |
BUGFIX: a typo in the variable name card_status triggered some error messages.
fixed the typo and cleaned up the code a bit with a better default for the variable
Diffstat (limited to 'include')
-rw-r--r-- | include/game.php | 29 |
1 files changed, 8 insertions, 21 deletions
diff --git a/include/game.php b/include/game.php index 79cdfa6..3890c02 100644 --- a/include/game.php +++ b/include/game.php @@ -520,8 +520,9 @@ switch($mystatus) /* Depending on the situation we set * cards_status (see functions.php for possible options) + * most of the times we need to just show the cards, so we make this the default */ -$cards_status = CARDS_EMPTY; +$card_status = CARDS_SHOW; /* Also collect message that should be displayed to the user, so that we can show * them after showing the table. This makes the html flow more consistent and easier @@ -545,7 +546,10 @@ switch($mystatus) { /* asks the player, if he wants to join the game */ output_check_want_to_play($me); + + /* don't show the cards before the user joined the game */ $card_status = CARDS_EMPTY; + break; } else @@ -563,6 +567,8 @@ switch($mystatus) mymail($user,$gameid,GAME_CANCELED,$email_message); } + $card_status = CARDS_EMPTY; + /* update game status */ cancel_game('noplay',$gameid); break; @@ -588,7 +594,7 @@ switch($mystatus) DB_set_player_by_gameid($gameid,$user); break; } - }; + } } } case 'init': @@ -600,8 +606,6 @@ switch($mystatus) $mycards = DB_get_hand($me); output_check_for_sickness($me,$mycards); - $card_status = CARDS_SHOW; - break; } else @@ -619,8 +623,6 @@ switch($mystatus) $messages[] = "You selected more than one sickness, please go back ". "and answer the <a href=\"$INDEX?action=game&me=$me&in=yes\">question</a> again."; - $card_status = CARDS_SHOW; - break; } else @@ -715,15 +717,6 @@ switch($mystatus) 'Seems like this is not the case, so you need to wait a bit... '. 'you will get an email once that is the case, please use the link in '. 'that email to continue the game.'); - - /* display cards, if player was just at the init-phase he will still see the cards from there - * we can put this one here, since the last player to finish the init state won't get here and - * will still see his card anyway from the init-phase - */ - if($mystatus=='check') - { - $card_status = CARDS_SHOW; - } } else { @@ -962,7 +955,6 @@ switch($mystatus) } } } - $card_status = CARDS_SHOW; } break; @@ -1007,9 +999,6 @@ switch($mystatus) /* get hand */ $mycards = DB_get_hand($me); - /* default: show cards, will be overwritten, if we need to give back cards */ - $card_status = CARDS_SHOW; - /* check if user need to give more cards back */ if( ($myparty=='re' || $myparty=='contra') && count($mycards)>12) { @@ -1329,8 +1318,6 @@ switch($mystatus) $messages[] = _('You finished the setup, but not everyone else finished it... '. 'You need to wait for the others. Just wait for an email.'); - $card_status = CARDS_SHOW; - break; /* not sure this works... the idea is that you can * only play a card after everyone is ready to play */ } |