Example of using the WIP Screen handler API

Jenkinsfile

@@ -1,23 +0,0 @@
-pipeline {
-   agent any
-   stages {
-
-      stage('Init') {
-         steps {
-            sh "rm -rf build/libs/"
-            sh "chmod +x gradlew"
-         }
-      }
-
-      stage ('Build') {
-         when {
-                branch '1.16'
-         }
-         steps {
-            sh "./gradlew clean build publish curseTools --refresh-dependencies --stacktrace"
-
-            archiveArtifacts artifacts: '**/build/libs/*.jar', fingerprint: true
-         }
-      }
-   }
-}

build.gradle

@@ -44,6 +44,7 @@ repositories {
 		//trinkets
 		url = "https://maven.abusedmaster.xyz/"
 	}
+	mavenLocal()
 	maven {
 		url = "https://jitpack.io"
 	}
@@ -76,6 +77,8 @@ dependencies {
 	//Fabric api
 	modImplementation "net.fabricmc.fabric-api:fabric-api:0.10.10+build.347-1.16"
 
+	modImplementation "net.fabricmc.fabric-api:fabric-screen-handler-api-v1:1.0.0+9ceae8129b"
+
 	optionalDependency ("me.shedaniel:RoughlyEnoughItems:4.3.7-unstable")
 	disabledOptionalDependency ('io.github.cottonmc:LibCD:2.3.0+1.15.2')
 	disabledOptionalDependency ('com.github.emilyploszaj:trinkets:2.4.2')

src/main/java/techreborn/client/GuiType.java

@@ -24,22 +24,29 @@
 
 package techreborn.client;
 
+import io.netty.buffer.Unpooled;
 import net.fabricmc.api.EnvType;
 import net.fabricmc.api.Environment;
-import net.fabricmc.fabric.api.client.screen.ScreenProviderRegistry;
+import net.fabricmc.fabric.api.client.screenhandler.v1.ScreenRegistry;
-import net.fabricmc.fabric.api.container.ContainerFactory;
+import net.fabricmc.fabric.api.screenhandler.v1.ExtendedScreenHandlerFactory;
-import net.fabricmc.fabric.api.container.ContainerProviderRegistry;
+import net.fabricmc.fabric.api.screenhandler.v1.ScreenHandlerRegistry;
 import net.minecraft.block.entity.BlockEntity;
 import net.minecraft.client.gui.screen.ingame.HandledScreen;
 import net.minecraft.entity.player.PlayerEntity;
+import net.minecraft.entity.player.PlayerInventory;
 import net.minecraft.network.PacketByteBuf;
+import net.minecraft.screen.ScreenHandler;
+import net.minecraft.screen.ScreenHandlerType;
+import net.minecraft.server.network.ServerPlayerEntity;
+import net.minecraft.text.LiteralText;
+import net.minecraft.text.Text;
 import net.minecraft.util.Identifier;
 import net.minecraft.util.math.BlockPos;
 import net.minecraft.world.World;
 import reborncore.RebornCore;
 import reborncore.api.blockentity.IMachineGuiHandler;
 import reborncore.client.screen.BuiltScreenHandlerProvider;
-import reborncore.client.screen.builder.ScreenHandlerBuilder;
+import reborncore.client.screen.builder.BuiltScreenHandler;
 import techreborn.blockentity.data.DataDrivenBEProvider;
 import techreborn.blockentity.data.DataDrivenGui;
 import techreborn.blockentity.generator.PlasmaGeneratorBlockEntity;
@@ -129,6 +136,7 @@ import techreborn.client.gui.GuiThermalGenerator;
 import techreborn.client.gui.GuiVacuumFreezer;
 import techreborn.client.gui.GuiWireMill;
 
+import javax.annotation.Nullable;
 import java.util.HashMap;
 import java.util.Map;
 import java.util.function.Supplier;
@@ -197,32 +205,59 @@ public final class GuiType<T extends BlockEntity> implements IMachineGuiHandler
 	}
 
 	private final Identifier identifier;
-	private final Supplier<Supplier<GuiFactory<T>>> factorySupplierMeme;
+	private final Supplier<Supplier<GuiFactory<T>>> guiFactory;
+	private final ScreenHandlerRegistry.ExtendedClientHandlerFactory<BuiltScreenHandler> screenHandlerFactory;
+	private final ScreenHandlerType<BuiltScreenHandler> screenHandlerType;
 
 	private GuiType(Identifier identifier, Supplier<Supplier<GuiFactory<T>>> factorySupplierMeme) {
 		this.identifier = identifier;
-		this.factorySupplierMeme = factorySupplierMeme;
+		this.guiFactory = factorySupplierMeme;
-		register();
+		this.screenHandlerFactory = getScreenHandlerFactory();
+		this.screenHandlerType = ScreenHandlerRegistry.registerExtended(identifier, screenHandlerFactory);
+		RebornCore.clientOnly(() -> () -> ScreenRegistry.register(screenHandlerType, getGuiFactory()));
+	}
+
+	private ScreenHandlerRegistry.ExtendedClientHandlerFactory<BuiltScreenHandler> getScreenHandlerFactory() {
+		return (syncId, playerInventory, packetByteBuf) -> {
+			final BlockEntity blockEntity = playerInventory.player.world.getBlockEntity(packetByteBuf.readBlockPos());
+			BuiltScreenHandler screenHandler = ((BuiltScreenHandlerProvider) blockEntity).createScreenHandler(syncId, playerInventory.player);
+
+			//Set the screen handler type, not ideal but works lol
+			screenHandler.setType(screenHandlerType);
+
+			return screenHandler;
+		};
 	}
 
 	@Environment(EnvType.CLIENT)
-	private GuiFactory<T> getFactory() {
+	private GuiFactory<T> getGuiFactory() {
-		return factorySupplierMeme.get().get();
+		return guiFactory.get().get();
 	}
 
 	@Override
 	public void open(PlayerEntity player, BlockPos pos, World world) {
 		if(!world.isClient){
-			ContainerProviderRegistry.INSTANCE.openContainer(identifier, player, packetByteBuf -> packetByteBuf.writeBlockPos(pos));
+			//This is awful
-		}
+			player.openHandledScreen(new ExtendedScreenHandlerFactory() {
+				@Override
+				public void writeScreenOpeningData(ServerPlayerEntity serverPlayerEntity, PacketByteBuf packetByteBuf) {
+					packetByteBuf.writeBlockPos(pos);
 				}
 
-	private void register() {
+				@Override
-		ContainerProviderRegistry.INSTANCE.registerFactory(identifier, (syncID, identifier, playerEntity, packetByteBuf) -> {
+				public Text getDisplayName() {
-			final BlockEntity blockEntity = playerEntity.world.getBlockEntity(packetByteBuf.readBlockPos());
+					return new LiteralText("What is this for?");
-			return ((BuiltScreenHandlerProvider) blockEntity).createScreenHandler(syncID, playerEntity);
+				}
+
+				@Nullable
+				@Override
+				public ScreenHandler createMenu(int syncId, PlayerInventory inv, PlayerEntity player) {
+					PacketByteBuf buf = new PacketByteBuf(Unpooled.buffer());
+					buf.writeBlockPos(pos);
+					return screenHandlerFactory.create(syncId, inv, buf);
+				}
 			});
-		RebornCore.clientOnly(() -> () -> ScreenProviderRegistry.INSTANCE.registerFactory(identifier, getFactory()));
+		}
 	}
 
 	public Identifier getIdentifier() {
@@ -230,14 +265,15 @@ public final class GuiType<T extends BlockEntity> implements IMachineGuiHandler
 	}
 
 	@Environment(EnvType.CLIENT)
-	public interface GuiFactory<T extends BlockEntity> extends ContainerFactory<HandledScreen> {
+	public interface GuiFactory<T extends BlockEntity> extends ScreenRegistry.Factory<BuiltScreenHandler, HandledScreen<BuiltScreenHandler>> {
 		HandledScreen<?> create(int syncId, PlayerEntity playerEntity, T blockEntity);
 
 		@Override
-		default HandledScreen create(int syncId, Identifier identifier, PlayerEntity playerEntity, PacketByteBuf packetByteBuf) {
+		default HandledScreen create(BuiltScreenHandler builtScreenHandler, PlayerInventory playerInventory, Text text) {
+			PlayerEntity playerEntity = playerInventory.player;
 			//noinspection unchecked
-			T blockEntity = (T) playerEntity.world.getBlockEntity(packetByteBuf.readBlockPos());
+			T blockEntity = (T) builtScreenHandler.getBlockEntity();
-			return create(syncId, playerEntity, blockEntity);
+			return create(builtScreenHandler.syncId, playerEntity, blockEntity);
 		}
 	}