Fix crash when binding HTTP variable, Closes #25

CyclopsMC · May 6, 2023 · 34e2b9f · 34e2b9f
1 parent 9292b54
commit 34e2b9f
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 7 deletions.
diff --git a/src/main/java/org/cyclops/integratedrest/IntegratedRest.java b/src/main/java/org/cyclops/integratedrest/IntegratedRest.java
@@ -4,6 +4,7 @@
 import net.minecraft.world.item.ItemStack;
 import net.minecraftforge.api.distmarker.Dist;
 import net.minecraftforge.api.distmarker.OnlyIn;
+import net.minecraftforge.client.event.ModelEvent;
 import net.minecraftforge.common.MinecraftForge;
 import net.minecraftforge.event.server.ServerStartedEvent;
 import net.minecraftforge.event.server.ServerStoppingEvent;
@@ -55,6 +56,9 @@ public IntegratedRest() {
         MinecraftForge.EVENT_BUS.addListener(this::onApiServerStarted);
         MinecraftForge.EVENT_BUS.addListener(this::onApiServerStopping);
         FMLJavaModLoadingContext.get().getModEventBus().addListener(this::afterSetup);
+        if (MinecraftHelpers.isClientSide()) {
+            FMLJavaModLoadingContext.get().getModEventBus().addListener(this::onModelLoading);
+        }
     }
 
     @Override
@@ -69,10 +73,6 @@ protected void setup(FMLCommonSetupEvent event) {
     protected void afterSetup(FMLLoadCompleteEvent event) {
         IntegratedDynamics._instance.getRegistryManager().getRegistry(IVariableFacadeHandlerRegistry.class).registerHandler(HttpVariableFacadeHandler.getInstance());
 
-        if (MinecraftHelpers.isClientSide()) {
-            HttpVariableModelProviders.load();
-        }
-
         RequestHandlers.load();
         ValueTypeJsonHandlers.load();
 
@@ -83,6 +83,10 @@ protected void afterSetup(FMLLoadCompleteEvent event) {
                         "/data/" + Reference.MOD_ID + "/info/rest_info.xml");
     }
 
+    protected void onModelLoading(ModelEvent.RegisterAdditional event) {
+        HttpVariableModelProviders.load();
+    }
+
     /**
      * Register the things that are related to server starting.
      * @param event The Forge event required for this.

diff --git a/src/main/java/org/cyclops/integratedrest/client/model/HttpVariableModelProviders.java b/src/main/java/org/cyclops/integratedrest/client/model/HttpVariableModelProviders.java
@@ -13,7 +13,7 @@
 public class HttpVariableModelProviders {
 
     public static final IVariableModelProviderRegistry REGISTRY = IntegratedDynamics._instance.getRegistryManager().getRegistry(IVariableModelProviderRegistry.class);
-    public static final SingleVariableModelProvider HTTP = REGISTRY.addProvider(new SingleVariableModelProvider(new ResourceLocation(Reference.MOD_ID + ":customoverlay/http")));
+    public static final SingleVariableModelProvider HTTP = REGISTRY.addProvider(new SingleVariableModelProvider(new ResourceLocation(Reference.MOD_ID, "customoverlay/http")));
 
     public static void load() {}
 

diff --git a/src/main/java/org/cyclops/integratedrest/item/HttpVariableFacade.java b/src/main/java/org/cyclops/integratedrest/item/HttpVariableFacade.java
@@ -57,7 +57,7 @@ protected Component getProxyTooltip() {
     @Override
     public void addModelOverlay(IVariableModelBaked variableModelBaked, List<BakedQuad> quads, RandomSource rand, ModelData modelData) {
         if(isValid()) {
-            quads.addAll(variableModelBaked.getSubModels(HttpVariableModelProviders.HTTP).getBakedModel().getQuads(null, null, rand));
+            quads.addAll(variableModelBaked.getSubModels(HttpVariableModelProviders.HTTP).getBakedModel().getQuads(null, null, rand, modelData, null));
         }
     }
 }
diff --git a/src/main/resources/assets/integratedrest/models/customoverlay/http.json b/src/main/resources/assets/integratedrest/models/customoverlay/http.json
@@ -1,5 +1,5 @@
 {
-  "parent": "builtin/generated",
+  "parent": "item/generated",
   "textures": {
     "layer0": "integratedrest:customoverlay/http"
   }