From 824ff375d4740fd407f46a3ba90d7185342f603f Mon Sep 17 00:00:00 2001
From: EnZaXD <florian.michael07@gmail.com>
Date: Sun, 16 Jun 2024 22:28:50 +0200
Subject: [PATCH] Fix GameMode#getById behaviour in 1.8->1.9 (#3947)

Vanilla fallbacks to SURVIVAL and doesn't throw any errors,
also NOT_SET was missing previously.
---
 .../com/viaversion/viaversion/api/minecraft/GameMode.java    | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/api/src/main/java/com/viaversion/viaversion/api/minecraft/GameMode.java b/api/src/main/java/com/viaversion/viaversion/api/minecraft/GameMode.java
index 1fa30424b..1751e0c25 100644
--- a/api/src/main/java/com/viaversion/viaversion/api/minecraft/GameMode.java
+++ b/api/src/main/java/com/viaversion/viaversion/api/minecraft/GameMode.java
@@ -24,6 +24,7 @@ package com.viaversion.viaversion.api.minecraft;
 
 public enum GameMode {
 
+    NOT_SET(""),
     SURVIVAL("Survival Mode"),
     CREATIVE("Creative Mode"),
     ADVENTURE("Adventure Mode"),
@@ -41,11 +42,11 @@ public enum GameMode {
 
     public static GameMode getById(int id) {
         return switch (id) {
-            case 0 -> SURVIVAL;
+            case -1 -> NOT_SET;
             case 1 -> CREATIVE;
             case 2 -> ADVENTURE;
             case 3 -> SPECTATOR;
-            default -> throw new IllegalArgumentException("Unknown gamemode id: " + id);
+            default /*0*/ -> SURVIVAL;
         };
     }
 }