README.md

@@ -11,6 +11,7 @@ TODO:
 
 
 Fix für Hanno auf Marvin:
-export "OPENSHIFT_IP=192.168.2.40"
+set environment varibale OPENSHIFT_IP=192.168.2.40
+or source .env
 
 Ähnliches falls jemand Netzwerkfehler des daemon erhält

gradle.properties

@@ -9,7 +9,7 @@ yarn_mappings=1.19.2+build.28
 loader_version=0.14.23
 
 # Mod Properties
-mod_version=1.0.1
+mod_version=1.0.0
 maven_group=modchest
 archives_base_name=reservermod
 

src/client/java/modchest/REServerModClient.java

@@ -13,6 +13,6 @@ public class REServerModClient implements ClientModInitializer {
 
         modNetworkingClient.registerC2SPackets();   //Identifier unter denen der Client zuhoert werden registriert
 
-        LOGGER.info("Modchest-Client successfully loaded!");
+        LOGGER.info("Modchest successfully loaded!");
     }
 }

src/client/java/modchest/networking/packet/respawnRequestS2CPacket.java

@@ -9,7 +9,7 @@ import net.minecraft.network.PacketByteBuf;
 public class respawnRequestS2CPacket {
 
     public static void receive(MinecraftClient minecraftClient, ClientPlayNetworkHandler clientPlayNetworkHandler, PacketByteBuf packetByteBuf, PacketSender packetSender) {
-        //REServerModClient.LOGGER.info("Server asked for respawn after pressing a saved respawn button!");
+        REServerModClient.LOGGER.info("Server asked for respawn after pressing a saved respawn button!");
         minecraftClient.player.requestRespawn();
     }
 }

src/main/java/modchest/REServerMod.java

@@ -11,12 +11,10 @@ public class REServerMod implements ModInitializer {
 	public static final String MOD_ID = "modchest";
 	public static final Logger LOGGER = LoggerFactory.getLogger("modchest"); // Erster Error Logger
 
-	//TODO: beim bett wird noch im zwei block-radius gesucht, damit der respawn point auch gelöscht wird, wenn man nur einen der beiden hälften zerstört
-
 	@Override
 	public void onInitialize() {	//Der uebersicht halber sind jetzt alle initializer in Klassen in util in initializer
-		initializer.itemGroups();
+		//initializer.itemGroups();
-		initializer.itemsAndBlocks();
+		//initializer.itemsAndBlocks();
 		initializer.events();
 		initializer.networking();
 

src/main/java/modchest/mixin/bedBlock.java

@@ -16,7 +16,6 @@ import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
 
 import java.util.Arrays;
 import java.util.List;
-import java.util.Objects;
 
 @Mixin(BedBlock.class)
 public class bedBlock {
@@ -24,35 +23,6 @@ public class bedBlock {
     @Inject(method = "onBreak", at = @At("HEAD"))
     public void onBreak(World world, BlockPos pos, BlockState state, PlayerEntity player, CallbackInfo ci) {    //Ueberprueft, wenn ein Bett zerstoert wird, ob jemand dieses Bett als Spawnpunkt hat
         if (!world.isClient) {      //Stellt sicher, dass das ganze nciht auf dem CLient rechnet wird, weil der Code sonst abstuertzt
-            int posBedX= pos.getX();
-            int posBedY= pos.getY();
-            int posBedZ= pos.getZ();
-
-            String bedData = state.toString();
-            if (Objects.equals(bedData.split("part=")[1], "foot]")) {        //das fussende wurde zerstoert, damit der Code also geht, muss in richtung kopfende eins gegangen werden
-                switch (bedData.split("facing=")[1].split(",")[0]) {
-                    case "east":
-                        posBedX++;
-                        player.sendMessage(Text.of("east"));
-                        break;
-                    case "west":
-                        posBedX--;
-                        player.sendMessage(Text.of("west"));
-                        break;
-                    case "south":
-                        posBedZ++;
-                        player.sendMessage(Text.of("south"));
-                        break;
-                    case "north":
-                        posBedZ--;
-                        player.sendMessage(Text.of("north"));
-                        break;
-                    default:
-                        player.sendMessage(Text.of(bedData.split("facing=")[1].split(",")[0]));
-                        break;
-                }
-            }
-
             setAndGetMultiButtonsSpawn setAndGet = new setAndGetMultiButtonsSpawn();    //Damit die Mehtoden benutzt werden koennen, muss ein Objekt erstellt werden; ist aber egal, weil eh keine Daten in dieser Klasse gespeichert werden
             List<? extends PlayerEntity> playerArray = null;
             try {           //Sollte der Code doch ausversehen auf dem Client ausgefuehrt werden, wuerde die folgende Zeile einen Error schmeissen
@@ -68,7 +38,7 @@ public class bedBlock {
                     } catch (Exception e) {
                         REServerMod.LOGGER.info("Error casting playerArray to ServerPlayerEntity: " + e);
                     }
-                    if (posBedX == posXYZ[0] && posBedY == posXYZ[1] && posBedZ == posXYZ[2]) {  //TODO: change range to 1      //Gleicht ab, ob die Koordinaten uebereinstimmen; die Spawnpunkte sind nicht immer ganz sauber, daher wird in Blockrange 1 geschaut
+                    if (inRange(pos.getX(), posXYZ[0], 1) && inRange(pos.getY(), posXYZ[1], 1) && inRange(pos.getZ(), posXYZ[2], 1)) {      //Gleicht ab, ob die Koordinaten uebereinstimmen; die Spawnpunkte sind nicht immer ganz sauber, daher wird in Blockrange 1 geschaut
                         try {           //Sollte der Code doch ausversehen auf dem Client ausgefuehrt werden, wuerde die folgende Zeile einen Error schmeissen
                             setAndGet.setMultiSpawnBlank((ServerPlayerEntity) playerArray.get(i), j);   //Wird ein bett zerstoert, wird der Spawnpunkt geblanked
                         } catch (Exception e) {

src/main/java/modchest/networking/packet/deathScreenMultiButtonsC2SPacket.java

@@ -38,7 +38,7 @@ public class deathScreenMultiButtonsC2SPacket {
                 player.setSpawnPoint(World.OVERWORLD, new BlockPos(posXYZ[0], posXYZ[1], posXYZ[2]), 0, true, true);    //neuer Spawn wird gesetzt
                 ServerPlayNetworking.send(player, modNetworkingServer.request_respawn, PacketByteBufs.create());        //Anfrage an den Client zum respawnen wird gesendet
                 player.sendMessage((Text.translatable("chat.modchest.multispawn.respawned", num)));
-                REServerMod.LOGGER.info("Respawned Player " + player.getEntityName() + " at " + Arrays.toString(setAndGet.getMultiSpawn(player, num)));
+                REServerMod.LOGGER.info("Respawned Player: " + player.getDisplayName() + "at " + Arrays.toString(setAndGet.getMultiSpawn(player, num)));
                 setAndGet.setMultiSpawn(player, 6);
             } catch (Exception e) {
                 REServerMod.LOGGER.info("Sorry! Following error, trying to respawn player " + player.getDisplayName() + ": " + e);
@@ -50,5 +50,6 @@ public class deathScreenMultiButtonsC2SPacket {
                 player.sendMessage(Text.translatable("chat.modchest.multispawn.spawnnotset"));
             }
         }
+
     }
 }

src/main/java/modchest/util/setAndGetMultiButtonsSpawn.java

@@ -26,16 +26,12 @@ public class setAndGetMultiButtonsSpawn {
         fullArray[2] = (int) player.getZ();
 
         try {
-            if (subNbt != null) {
+            assert subNbt != null;
             subNbt.putIntArray(String.valueOf(spawn), fullArray);   //speichert unseren array an der entsprechenden nummer ab
             topNbt.put("multi_sleep", subNbt);  //speichert den tag unter multi_sleep im top-tag ab
             castedPlayer.setDataSaver(topNbt);      //speichert den top-tag wieder im spieler ab
-            } else {
-                REServerMod.LOGGER.info("Error when trying to write spawn to file!");
-                player.sendMessage(Text.translatable("chat.modchest.multispawn.spawnnotsaved"));
-            }
         } catch (Exception e) {
-            REServerMod.LOGGER.info("Error when saving Spawnpoint! Exception: " + e);
+            REServerMod.LOGGER.info("Sorry! Error when saving Spawnpoint! Exception: " + e);
             player.sendMessage(Text.translatable("chat.modchest.multispawn.spawnnotsaved"));
         }
     }
@@ -123,7 +119,7 @@ public class setAndGetMultiButtonsSpawn {
 
         if (subNbt != null) {
             NbtCompound nbt = (NbtCompound) subNbt.get("vanillaSpawnData"); //hollt sich die daten
-            if (nbt != null) {
+            assert nbt != null;
             if (!nbt.getBoolean("used")) {
                 nbt.putBoolean("used", true);       //setzt den verwendet boolean auf true; nicht ganz perfekt, sollte ein fehler beim speichern auftreten wird es nicht nochmal versucht, speichert aber massiv an coding und leistung
                 subNbt.put("vanillaSpawnData", nbt);
@@ -133,11 +129,9 @@ public class setAndGetMultiButtonsSpawn {
             } else {
                 return null;
             }
-            }
         } else {
             return null;
         }
-        return null;
     }
 
     public synchronized void setMultiSpawnBlank(ServerPlayerEntity player, int spawn) {     //wird aufgerufen wenn das Bett dieses Spielers zerstoert wurde

src/main/resources/fabric.mod.json

@@ -16,7 +16,7 @@
 		"sources": "https://github.com/FabricMC/fabric-example-mod"
 	},
 	"license": "CC0-1.0",
-	"icon": "assets/modchest/icon.png",
+	"icon": "assets/reservermod/icon.png",
 	"environment": "*",
 	"entrypoints": {
 		"main": [