multiplayer fix

comdex-official · Feb 15, 2024 · b31efe0 · b31efe0
1 parent a7cd99c
commit b31efe0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/x/common/keeper/keeper.go b/x/common/keeper/keeper.go
@@ -95,25 +95,25 @@ func (k Keeper) SinglePlayer(ctx sdk.Context, contractAddress string, ResolveSin
 	logger := k.Logger(ctx)
 	err := k.SudoContractCall(ctx, contractAddress, ResolveSinglePlayer)
 	if err != nil {
-		logger.Error(fmt.Sprintf("Game %s contract call error for single player", gameName))
+		logger.Error(fmt.Sprintf("Game %s contract call error for single-player", gameName))
 	} else {
-		logger.Info(fmt.Sprintf("Game %s contract call for single player success", gameName))
+		logger.Info(fmt.Sprintf("Game %s contract call for single-player success", gameName))
 	}
 }
 
 func (k Keeper) MultiPlayer(ctx sdk.Context, contractAddress string, SetupMultiPlayer []byte, ResolveMultiPlayer []byte, gameName string) {
 	logger := k.Logger(ctx)
 	err := k.SudoContractCall(ctx, contractAddress, SetupMultiPlayer)
 	if err != nil {
-		logger.Error(fmt.Sprintf("Game %s contract call error for setup multi player", gameName))
+		logger.Error(fmt.Sprintf("Game %s contract call error for setup multi-player", gameName))
 	} else {
-		logger.Info(fmt.Sprintf("Game %s contract call for setup multi player success", gameName))
+		logger.Info(fmt.Sprintf("Game %s contract call for setup multi-player success", gameName))
 	}
 
 	err = k.SudoContractCall(ctx, contractAddress, ResolveMultiPlayer)
 	if err != nil {
-		logger.Error(fmt.Sprintf("Game %s contract call error for resolve multi player", gameName))
+		logger.Error(fmt.Sprintf("Game %s contract call error for resolve multi-player", gameName))
 	} else {
-		logger.Info(fmt.Sprintf("Game %s contract call for single resolve multi success", gameName))
+		logger.Info(fmt.Sprintf("Game %s contract call for resolve multi-player success", gameName))
 	}
 }
diff --git a/x/common/types/message_register_contract.go b/x/common/types/message_register_contract.go
@@ -20,6 +20,7 @@ func NewMsgRegisterContract(
 		SecurityAddress: securityAddress,
 		GameName: gameName,
 		ContractAddress: contractAddress,
+		GameType: gameType,
 	}
 }
 
@@ -56,7 +57,7 @@ func (msg *MsgRegisterContract) ValidateBasic() error {
 	}
 	gameType := []uint64{1, 2, 3}
 	if !slices.Contains(gameType, msg.GameType) {
-		return sdkerrors.Wrap(sdkerrors.ErrInvalidType, "invalid game type (%s)")
+		return sdkerrors.Wrap(sdkerrors.ErrInvalidType, "invalid game type, should be 1,2 or 3")
 	}
 
 	return nil