summaryrefslogtreecommitdiff |
diff options
author | Nathanael Sensfelder <SpamShield0@MultiAgentSystems.org> | 2019-09-10 21:44:45 +0200 |
---|---|---|
committer | Nathanael Sensfelder <SpamShield0@MultiAgentSystems.org> | 2019-09-10 21:44:45 +0200 |
commit | 37be15261c35d732e6a254b75b77654ad0ad9d97 (patch) | |
tree | 34102799ff0f16982d3b7b4c87f5f1b6a4bdd923 /src/battle/query/btl_character_turn.erl | |
parent | cfdd37c261bfb814740aa8cec30177f9c5f85d36 (diff) |
Transforms some "assert" into error throws.
Diffstat (limited to 'src/battle/query/btl_character_turn.erl')
-rw-r--r-- | src/battle/query/btl_character_turn.erl | 29 |
1 files changed, 20 insertions, 9 deletions
diff --git a/src/battle/query/btl_character_turn.erl b/src/battle/query/btl_character_turn.erl index 4f96552..9534945 100644 --- a/src/battle/query/btl_character_turn.erl +++ b/src/battle/query/btl_character_turn.erl @@ -65,9 +65,18 @@ assert_user_is_current_player (Update, Request) -> CurrentPlayerIX = btl_player_turn:get_player_ix(CurrentPlayerTurn), CurrentPlayer = btl_battle:get_player(CurrentPlayerIX, Battle), - true = (PlayerID == btl_player:get_id(CurrentPlayer)), - - ok. + case (PlayerID == btl_player:get_id(CurrentPlayer)) of + true -> ok; + _ -> + error + ( + { + player_id, + btl_player:get_id(CurrentPlayer), + PlayerID + } + ) + end. -spec assert_user_owns_played_character ( @@ -83,9 +92,10 @@ assert_user_owns_played_character (Update, Request) -> CharacterPlayer = btl_battle:get_player(CharacterPlayerIX, Battle), CharacterPlayerID = btl_player:get_id(CharacterPlayer), - true = (PlayerID == CharacterPlayerID), - - ok. + case (PlayerID == CharacterPlayerID) of + true -> ok; + _ -> error({player_id, CharacterPlayerID, PlayerID}) + end. -spec assert_character_can_be_played (btl_character_turn_update:type()) -> 'ok'. assert_character_can_be_played (Update) -> @@ -93,9 +103,10 @@ assert_character_can_be_played (Update) -> CharacterIX = btl_character_turn_update:get_character_ix(Update), Character = btl_battle:get_character(CharacterIX, Battle), - true = btl_character:get_is_active(Character), - - ok. + case btl_character:get_is_active(Character) of + true -> ok; + _ -> error({character, active, CharacterIX}) + end. -spec assert_user_permissions ( |