diff --git a/fabric-model-loading-api-v1/build.gradle b/fabric-model-loading-api-v1/build.gradle
index ca3eb5eb1..3d2819264 100644
--- a/fabric-model-loading-api-v1/build.gradle
+++ b/fabric-model-loading-api-v1/build.gradle
@@ -4,6 +4,7 @@ moduleDependencies(project, ['fabric-api-base'])
testDependencies(project, [
':fabric-renderer-api-v1',
+ ':fabric-renderer-indigo',
':fabric-rendering-v1',
':fabric-resource-loader-v0'
])
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/BlockStateResolver.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/BlockStateResolver.java
index dacd2f1a4..4f350e43d 100644
--- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/BlockStateResolver.java
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/BlockStateResolver.java
@@ -22,7 +22,6 @@ import net.minecraft.block.Block;
import net.minecraft.block.BlockState;
import net.minecraft.client.render.model.ModelLoader;
import net.minecraft.client.render.model.UnbakedModel;
-import net.minecraft.client.util.ModelIdentifier;
import net.minecraft.util.Identifier;
/**
@@ -72,7 +71,7 @@ public interface BlockStateResolver {
void setModel(BlockState state, UnbakedModel model);
/**
- * Loads a model using an {@link Identifier} or {@link ModelIdentifier}, or gets it if it was already loaded.
+ * Loads a model using an {@link Identifier}, or gets it if it was already loaded.
*
* @param id the model identifier
* @return the unbaked model, or a missing model if it is not present
@@ -81,9 +80,6 @@ public interface BlockStateResolver {
/**
* The current model loader instance, which changes between resource reloads.
- *
- *
Do not call {@link ModelLoader#getOrLoadModel} as it does not supported nested model resolution;
- * use {@link #getOrLoadModel} from the context instead.
*/
ModelLoader loader();
}
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/DelegatingUnbakedModel.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/DelegatingUnbakedModel.java
index ee37263fd..081f5f6a0 100644
--- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/DelegatingUnbakedModel.java
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/DelegatingUnbakedModel.java
@@ -27,7 +27,6 @@ import net.minecraft.client.render.model.Baker;
import net.minecraft.client.render.model.ModelBakeSettings;
import net.minecraft.client.render.model.UnbakedModel;
import net.minecraft.client.texture.Sprite;
-import net.minecraft.client.util.ModelIdentifier;
import net.minecraft.client.util.SpriteIdentifier;
import net.minecraft.util.Identifier;
@@ -43,7 +42,7 @@ public final class DelegatingUnbakedModel implements UnbakedModel {
/**
* Constructs a new delegating model.
*
- * @param delegate The identifier (can be a {@link ModelIdentifier}) of the underlying baked model.
+ * @param delegate The identifier of the underlying baked model.
*/
public DelegatingUnbakedModel(Identifier delegate) {
this.delegate = delegate;
@@ -57,11 +56,12 @@ public final class DelegatingUnbakedModel implements UnbakedModel {
@Override
public void setParents(Function modelLoader) {
+ modelLoader.apply(delegate).setParents(modelLoader);
}
- @Nullable
@Override
- public BakedModel bake(Baker baker, Function textureGetter, ModelBakeSettings rotationContainer, Identifier modelId) {
+ @Nullable
+ public BakedModel bake(Baker baker, Function textureGetter, ModelBakeSettings rotationContainer) {
return baker.bake(delegate, rotationContainer);
}
}
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/FabricBakedModelManager.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/FabricBakedModelManager.java
index dbc5e0c4c..e0b8caba0 100644
--- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/FabricBakedModelManager.java
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/FabricBakedModelManager.java
@@ -16,8 +16,6 @@
package net.fabricmc.fabric.api.client.model.loading.v1;
-import org.jetbrains.annotations.Nullable;
-
import net.minecraft.client.render.model.BakedModel;
import net.minecraft.client.render.model.BakedModelManager;
import net.minecraft.client.util.ModelIdentifier;
@@ -30,20 +28,18 @@ import net.minecraft.util.Identifier;
*/
public interface FabricBakedModelManager {
/**
- * An alternative to {@link BakedModelManager#getModel(ModelIdentifier)} that accepts an
- * {@link Identifier} instead. Models loaded using {@link ModelLoadingPlugin.Context#addModels}
- * do not have a corresponding {@link ModelIdentifier}, so the vanilla method cannot be used to
- * retrieve them. The {@link Identifier} that was used to load them can be used in this method
- * to retrieve them.
+ * Similar to {@link BakedModelManager#getModel(ModelIdentifier)}, but accepts an {@link Identifier} instead of a
+ * {@link ModelIdentifier}. Use this method to retrieve models loaded using
+ * {@link ModelLoadingPlugin.Context#addModels}, since those models do not have corresponding
+ * {@link ModelIdentifier}s.
*
* This method, as well as its vanilla counterpart, should only be used after the
* {@link BakedModelManager} has completed reloading. Otherwise, the result will be
- * outdated or null.
+ * outdated or an exception will be thrown.
*
* @param id the id of the model
* @return the model
*/
- @Nullable
default BakedModel getModel(Identifier id) {
throw new UnsupportedOperationException("Implemented via mixin.");
}
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/ModelLoadingPlugin.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/ModelLoadingPlugin.java
index 002deb2fa..5d15c8428 100644
--- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/ModelLoadingPlugin.java
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/ModelLoadingPlugin.java
@@ -22,7 +22,6 @@ import org.jetbrains.annotations.ApiStatus;
import net.minecraft.block.Block;
import net.minecraft.client.render.model.json.JsonUnbakedModel;
-import net.minecraft.client.util.ModelIdentifier;
import net.minecraft.resource.ResourceManager;
import net.minecraft.util.Identifier;
@@ -53,13 +52,14 @@ public interface ModelLoadingPlugin {
@ApiStatus.NonExtendable
interface Context {
/**
- * Adds one or more models (can be {@link ModelIdentifier}s) to the list of models that will be loaded and
- * baked.
+ * Adds one or more models that will be loaded, baked, and made available through
+ * {@link FabricBakedModelManager#getModel(Identifier)}.
*/
void addModels(Identifier... ids);
/**
- * Adds multiple models (can be {@link ModelIdentifier}s) to the list of models that will be loaded and baked.
+ * Adds multiple models that will be loaded, baked, and made available through
+ * {@link FabricBakedModelManager#getModel(Identifier)}.
*/
void addModels(Collection extends Identifier> ids);
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/ModelModifier.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/ModelModifier.java
index da6777996..0d8596b6d 100644
--- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/ModelModifier.java
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/ModelModifier.java
@@ -20,6 +20,7 @@ import java.util.function.Function;
import org.jetbrains.annotations.ApiStatus;
import org.jetbrains.annotations.Nullable;
+import org.jetbrains.annotations.UnknownNullability;
import net.minecraft.client.render.model.BakedModel;
import net.minecraft.client.render.model.Baker;
@@ -39,7 +40,7 @@ import net.fabricmc.fabric.api.event.Event;
*
*
Example use cases:
*
- * - Overriding a model for a particular block state - check if the given identifier is a {@link ModelIdentifier},
+ *
- Overriding a model for a particular block state - check if the given top-level identifier is not null,
* and then check if it has the appropriate variant for that block state. If so, return your desired model,
* otherwise return the given model.
* - Wrapping a model to override certain behaviors - simply return a new model instance and delegate calls
@@ -91,16 +92,26 @@ public final class ModelModifier {
@ApiStatus.NonExtendable
interface Context {
/**
- * The identifier of this model (may be a {@link ModelIdentifier}).
+ * Models with a resource ID are loaded directly from JSON or a {@link ModelModifier}.
*
- *
For item models, only the {@link ModelIdentifier} with the {@code inventory} variant is passed, and
- * not the corresponding plain identifier.
+ * @return the identifier of the given model as an {@link Identifier}, or null if {@link #topLevelId()} is
+ * not null
*/
- Identifier id();
+ @UnknownNullability("#topLevelId() != null")
+ Identifier resourceId();
/**
- * Loads a model using an {@link Identifier} or {@link ModelIdentifier}, or gets it if it was already
- * loaded.
+ * Models with a top-level ID are loaded from blockstate files, {@link BlockStateResolver}s, or by copying
+ * a previously loaded model.
+ *
+ * @return the identifier of the given model as a {@link ModelIdentifier}, or null if {@link #resourceId()}
+ * is not null
+ */
+ @UnknownNullability("#resourceId() != null")
+ ModelIdentifier topLevelId();
+
+ /**
+ * Loads a model using an {@link Identifier}, or gets it if it was already loaded.
*
* @param id the model identifier
* @return the unbaked model, or a missing model if it is not present
@@ -109,9 +120,6 @@ public final class ModelModifier {
/**
* The current model loader instance, which changes between resource reloads.
- *
- *
Do not call {@link ModelLoader#getOrLoadModel} as it does not supported nested model
- * resolution; use {@link #getOrLoadModel} from the context instead.
*/
ModelLoader loader();
}
@@ -135,9 +143,23 @@ public final class ModelModifier {
@ApiStatus.NonExtendable
interface Context {
/**
- * The identifier of this model (may be a {@link ModelIdentifier}).
+ * Models with a resource ID are loaded directly from JSON or a {@link ModelModifier}.
+ *
+ * @return the identifier of the given model as an {@link Identifier}, or null if {@link #topLevelId()} is
+ * not null
*/
- Identifier id();
+ @UnknownNullability("#topLevelId() != null")
+ Identifier resourceId();
+
+ /**
+ * Models with a top-level ID are loaded from blockstate files, {@link BlockStateResolver}s, or by copying
+ * a previously loaded model.
+ *
+ * @return the identifier of the given model as a {@link ModelIdentifier}, or null if {@link #resourceId()}
+ * is not null
+ */
+ @UnknownNullability("#resourceId() != null")
+ ModelIdentifier topLevelId();
/**
* The function that can be used to retrieve sprites.
@@ -189,9 +211,23 @@ public final class ModelModifier {
@ApiStatus.NonExtendable
interface Context {
/**
- * The identifier of this model (may be a {@link ModelIdentifier}).
+ * Models with a resource ID are loaded directly from JSON or a {@link ModelModifier}.
+ *
+ * @return the identifier of the given model as an {@link Identifier}, or null if {@link #topLevelId()} is
+ * not null
*/
- Identifier id();
+ @UnknownNullability("#topLevelId() != null")
+ Identifier resourceId();
+
+ /**
+ * Models with a top-level ID are loaded from blockstate files, {@link BlockStateResolver}s, or by copying
+ * a previously loaded model.
+ *
+ * @return the identifier of the given model as a {@link ModelIdentifier}, or null if {@link #resourceId()}
+ * is not null
+ */
+ @UnknownNullability("#resourceId() != null")
+ ModelIdentifier topLevelId();
/**
* The unbaked model that is being baked.
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/ModelResolver.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/ModelResolver.java
index 8236af031..7f0843e2c 100644
--- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/ModelResolver.java
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/api/client/model/loading/v1/ModelResolver.java
@@ -21,7 +21,6 @@ import org.jetbrains.annotations.Nullable;
import net.minecraft.client.render.model.ModelLoader;
import net.minecraft.client.render.model.UnbakedModel;
-import net.minecraft.client.util.ModelIdentifier;
import net.minecraft.util.Identifier;
/**
@@ -64,7 +63,7 @@ public interface ModelResolver {
Identifier id();
/**
- * Loads a model using an {@link Identifier} or {@link ModelIdentifier}, or gets it if it was already loaded.
+ * Loads a model using an {@link Identifier}, or gets it if it was already loaded.
*
* @param id the model identifier
* @return the unbaked model, or a missing model if it is not present
@@ -73,9 +72,6 @@ public interface ModelResolver {
/**
* The current model loader instance, which changes between resource reloads.
- *
- *
Do not call {@link ModelLoader#getOrLoadModel} as it does not supported nested model resolution;
- * use {@link #getOrLoadModel} from the context instead.
*/
ModelLoader loader();
}
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/LegacyModelVariantProvider.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/BakerImplHooks.java
similarity index 66%
rename from fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/LegacyModelVariantProvider.java
rename to fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/BakerImplHooks.java
index ecc5ddc0b..8d088dee1 100644
--- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/LegacyModelVariantProvider.java
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/BakerImplHooks.java
@@ -16,15 +16,11 @@
package net.fabricmc.fabric.impl.client.model.loading;
-import org.jetbrains.annotations.Nullable;
+import java.util.function.Function;
-import net.minecraft.client.render.model.UnbakedModel;
-import net.minecraft.client.util.ModelIdentifier;
+import net.minecraft.client.texture.Sprite;
+import net.minecraft.client.util.SpriteIdentifier;
-/**
- * Legacy v0 bridge - remove if the legacy v0 module is removed.
- */
-public interface LegacyModelVariantProvider {
- @Nullable
- UnbakedModel loadModelVariant(ModelIdentifier modelId);
+public interface BakerImplHooks {
+ Function fabric_getTextureGetter();
}
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/BlockStatesLoaderHooks.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/BlockStatesLoaderHooks.java
new file mode 100644
index 000000000..eda910323
--- /dev/null
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/BlockStatesLoaderHooks.java
@@ -0,0 +1,30 @@
+/*
+ * Copyright (c) 2016, 2017, 2018, 2019 FabricMC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package net.fabricmc.fabric.impl.client.model.loading;
+
+import net.minecraft.block.Block;
+import net.minecraft.block.BlockState;
+import net.minecraft.state.StateManager;
+import net.minecraft.util.Identifier;
+
+public interface BlockStatesLoaderHooks {
+ void fabric_setLoadingOverride(LoadingOverride override);
+
+ interface LoadingOverride {
+ boolean loadBlockStates(Identifier id, StateManager stateManager);
+ }
+}
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoaderHooks.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoaderHooks.java
index 67e1dfd26..434988699 100644
--- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoaderHooks.java
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoaderHooks.java
@@ -17,7 +17,7 @@
package net.fabricmc.fabric.impl.client.model.loading;
import net.minecraft.client.render.model.UnbakedModel;
-import net.minecraft.client.render.model.json.JsonUnbakedModel;
+import net.minecraft.client.util.ModelIdentifier;
import net.minecraft.util.Identifier;
public interface ModelLoaderHooks {
@@ -27,11 +27,5 @@ public interface ModelLoaderHooks {
UnbakedModel fabric_getOrLoadModel(Identifier id);
- void fabric_putModel(Identifier id, UnbakedModel model);
-
- void fabric_putModelDirectly(Identifier id, UnbakedModel model);
-
- void fabric_queueModelDependencies(UnbakedModel model);
-
- JsonUnbakedModel fabric_loadModelFromJson(Identifier id);
+ void fabric_add(ModelIdentifier id, UnbakedModel model);
}
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoadingConstants.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoadingConstants.java
new file mode 100644
index 000000000..7d27e12e0
--- /dev/null
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoadingConstants.java
@@ -0,0 +1,42 @@
+/*
+ * Copyright (c) 2016, 2017, 2018, 2019 FabricMC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package net.fabricmc.fabric.impl.client.model.loading;
+
+import net.minecraft.client.util.ModelIdentifier;
+import net.minecraft.util.Identifier;
+
+public final class ModelLoadingConstants {
+ /**
+ * This variant is used to convert user-provided Identifiers for extra models to ModelIdentifiers, since top-level
+ * models that will be baked must have a ModelIdentifier. Models corresponding to the Identifiers will go through
+ * ModelModifier.OnLoad, but models corresponding to the ModelIdentifiers will not.
+ *
+ * This variant must be non-empty, must not contain "=", and must not be equal to "inventory" or "missingno".
+ */
+ public static final String RESOURCE_SPECIAL_VARIANT = "fabric_resource";
+
+ private ModelLoadingConstants() {
+ }
+
+ public static ModelIdentifier toResourceModelId(Identifier id) {
+ return new ModelIdentifier(id, RESOURCE_SPECIAL_VARIANT);
+ }
+
+ public static boolean isResourceModelId(ModelIdentifier id) {
+ return id.variant().equals(RESOURCE_SPECIAL_VARIANT);
+ }
+}
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoadingEventDispatcher.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoadingEventDispatcher.java
index c0a62ce67..7d6a3dcc5 100644
--- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoadingEventDispatcher.java
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoadingEventDispatcher.java
@@ -25,9 +25,8 @@ import com.google.common.collect.ImmutableList;
import it.unimi.dsi.fastutil.objects.ObjectArrayList;
import it.unimi.dsi.fastutil.objects.Reference2ReferenceMap;
import it.unimi.dsi.fastutil.objects.Reference2ReferenceOpenHashMap;
-import it.unimi.dsi.fastutil.objects.ReferenceOpenHashSet;
-import it.unimi.dsi.fastutil.objects.ReferenceSet;
import org.jetbrains.annotations.Nullable;
+import org.jetbrains.annotations.UnknownNullability;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -42,6 +41,7 @@ import net.minecraft.client.render.model.UnbakedModel;
import net.minecraft.client.texture.Sprite;
import net.minecraft.client.util.ModelIdentifier;
import net.minecraft.client.util.SpriteIdentifier;
+import net.minecraft.state.StateManager;
import net.minecraft.util.Identifier;
import net.fabricmc.fabric.api.client.model.loading.v1.BlockStateResolver;
@@ -53,12 +53,10 @@ public class ModelLoadingEventDispatcher {
private static final Logger LOGGER = LoggerFactory.getLogger(ModelLoadingEventDispatcher.class);
private final ModelLoader loader;
- private final ModelLoaderPluginContextImpl pluginContext;
+ private final ModelLoadingPluginContextImpl pluginContext;
private final ObjectArrayList modelResolverContextStack = new ObjectArrayList<>();
-
- private final ObjectArrayList blockStateResolverContextStack = new ObjectArrayList<>();
- private final ReferenceSet resolvingBlocks = new ReferenceOpenHashSet<>();
+ private final BlockStateResolverContext blockStateResolverContext = new BlockStateResolverContext();
private final ObjectArrayList onLoadModifierContextStack = new ObjectArrayList<>();
private final ObjectArrayList beforeBakeModifierContextStack = new ObjectArrayList<>();
@@ -66,7 +64,7 @@ public class ModelLoadingEventDispatcher {
public ModelLoadingEventDispatcher(ModelLoader loader, List plugins) {
this.loader = loader;
- this.pluginContext = new ModelLoaderPluginContextImpl(((ModelLoaderHooks) loader)::fabric_getOrLoadModel);
+ this.pluginContext = new ModelLoadingPluginContextImpl();
for (ModelLoadingPlugin plugin : plugins) {
try {
@@ -83,107 +81,19 @@ public class ModelLoadingEventDispatcher {
}
}
- /**
- * @return {@code true} to cancel the vanilla method
- */
- public boolean loadModel(Identifier id) {
- if (id instanceof ModelIdentifier modelId) {
- if ("inventory".equals(modelId.getVariant())) {
- // We ALWAYS override the vanilla inventory model code path entirely, even for vanilla item models.
- // See loadItemModel for an explanation.
- loadItemModel(modelId);
- return true;
- } else {
- // Prioritize block state resolver over legacy variant provider
- BlockStateResolverHolder resolver = pluginContext.getBlockStateResolver(modelId);
+ public boolean loadBlockStateModels(Identifier id, StateManager stateManager) {
+ BlockStateResolver resolver = pluginContext.blockStateResolvers.get(id);
- if (resolver != null) {
- loadBlockStateModels(resolver.resolver(), resolver.block(), resolver.blockId());
- return true;
- }
-
- UnbakedModel legacyModel = legacyLoadModelVariant(modelId);
-
- if (legacyModel != null) {
- ((ModelLoaderHooks) loader).fabric_putModel(id, legacyModel);
- return true;
- }
-
- return false;
- }
+ if (resolver != null) {
+ resolveBlockStates(resolver, stateManager.getOwner(), id);
+ return true;
} else {
- UnbakedModel model = resolveModel(id);
-
- if (model != null) {
- ((ModelLoaderHooks) loader).fabric_putModel(id, model);
- return true;
- }
-
return false;
}
}
- @Nullable
- private UnbakedModel legacyLoadModelVariant(ModelIdentifier modelId) {
- return pluginContext.legacyVariantProviders().invoker().loadModelVariant(modelId);
- }
-
- /**
- * This function handles both modded item models and vanilla item models.
- * The vanilla code path for item models is never used.
- * See the long comment in the function for an explanation.
- */
- private void loadItemModel(ModelIdentifier modelId) {
- ModelLoaderHooks loaderHooks = (ModelLoaderHooks) loader;
-
- Identifier id = modelId.withPrefixedPath("item/");
-
- // Legacy variant provider
- UnbakedModel model = legacyLoadModelVariant(modelId);
-
- // Model resolver
- if (model == null) {
- model = resolveModel(id);
- }
-
- // Load from the vanilla code path otherwise.
- if (model == null) {
- model = loaderHooks.fabric_loadModelFromJson(id);
- }
-
- // This is a bit tricky:
- // We have a single UnbakedModel now, but there are two identifiers:
- // the ModelIdentifier (...#inventory) and the Identifier (...:item/...).
- // So we call the on load modifier now and then directly add the model to the ModelLoader,
- // reimplementing the behavior of ModelLoader#put.
- // Calling ModelLoader#put is not an option as the model for the Identifier would not be replaced by an on load modifier.
- // This is why we override the vanilla code path entirely.
- model = modifyModelOnLoad(modelId, model);
-
- loaderHooks.fabric_putModelDirectly(modelId, model);
- loaderHooks.fabric_putModelDirectly(id, model);
- loaderHooks.fabric_queueModelDependencies(model);
- }
-
- private void loadBlockStateModels(BlockStateResolver resolver, Block block, Identifier blockId) {
- if (!resolvingBlocks.add(block)) {
- throw new IllegalStateException("Circular reference while resolving models for block " + block);
- }
-
- try {
- resolveBlockStates(resolver, block, blockId);
- } finally {
- resolvingBlocks.remove(block);
- }
- }
-
private void resolveBlockStates(BlockStateResolver resolver, Block block, Identifier blockId) {
- // Get and prepare context
- if (blockStateResolverContextStack.isEmpty()) {
- blockStateResolverContextStack.add(new BlockStateResolverContext());
- }
-
- BlockStateResolverContext context = blockStateResolverContextStack.pop();
+ BlockStateResolverContext context = blockStateResolverContext;
context.prepare(block);
Reference2ReferenceMap resolvedModels = context.models;
@@ -204,14 +114,14 @@ public class ModelLoadingEventDispatcher {
for (BlockState state : allStates) {
ModelIdentifier modelId = BlockModels.getModelId(blockId, state);
- ((ModelLoaderHooks) loader).fabric_putModelDirectly(modelId, missingModel);
+ ((ModelLoaderHooks) loader).fabric_add(modelId, missingModel);
}
} else if (resolvedModels.size() == allStates.size()) {
// If there are as many resolved models as total states, all states have
// been resolved and models do not need to be null-checked.
resolvedModels.forEach((state, model) -> {
ModelIdentifier modelId = BlockModels.getModelId(blockId, state);
- ((ModelLoaderHooks) loader).fabric_putModel(modelId, model);
+ ((ModelLoaderHooks) loader).fabric_add(modelId, model);
});
} else {
UnbakedModel missingModel = ((ModelLoaderHooks) loader).fabric_getMissingModel();
@@ -223,21 +133,18 @@ public class ModelLoadingEventDispatcher {
if (model == null) {
LOGGER.error("Block state resolver did not provide a model for state {} in block {}. Using missing model.", state, block);
- ((ModelLoaderHooks) loader).fabric_putModelDirectly(modelId, missingModel);
+ ((ModelLoaderHooks) loader).fabric_add(modelId, missingModel);
} else {
- ((ModelLoaderHooks) loader).fabric_putModel(modelId, model);
+ ((ModelLoaderHooks) loader).fabric_add(modelId, model);
}
}
}
resolvedModels.clear();
-
- // Store context for reuse
- blockStateResolverContextStack.add(context);
}
@Nullable
- private UnbakedModel resolveModel(Identifier id) {
+ public UnbakedModel resolveModel(Identifier id) {
if (modelResolverContextStack.isEmpty()) {
modelResolverContextStack.add(new ModelResolverContext());
}
@@ -251,13 +158,13 @@ public class ModelLoadingEventDispatcher {
return model;
}
- public UnbakedModel modifyModelOnLoad(Identifier id, UnbakedModel model) {
+ public UnbakedModel modifyModelOnLoad(UnbakedModel model, @UnknownNullability Identifier resourceId, @UnknownNullability ModelIdentifier topLevelId) {
if (onLoadModifierContextStack.isEmpty()) {
onLoadModifierContextStack.add(new OnLoadModifierContext());
}
OnLoadModifierContext context = onLoadModifierContextStack.pop();
- context.prepare(id);
+ context.prepare(resourceId, topLevelId);
model = pluginContext.modifyModelOnLoad().invoker().modifyModelOnLoad(model, context);
@@ -265,13 +172,13 @@ public class ModelLoadingEventDispatcher {
return model;
}
- public UnbakedModel modifyModelBeforeBake(UnbakedModel model, Identifier id, Function textureGetter, ModelBakeSettings settings, Baker baker) {
+ public UnbakedModel modifyModelBeforeBake(UnbakedModel model, @UnknownNullability Identifier resourceId, @UnknownNullability ModelIdentifier topLevelId, Function textureGetter, ModelBakeSettings settings, Baker baker) {
if (beforeBakeModifierContextStack.isEmpty()) {
beforeBakeModifierContextStack.add(new BeforeBakeModifierContext());
}
BeforeBakeModifierContext context = beforeBakeModifierContextStack.pop();
- context.prepare(id, textureGetter, settings, baker);
+ context.prepare(resourceId, topLevelId, textureGetter, settings, baker);
model = pluginContext.modifyModelBeforeBake().invoker().modifyModelBeforeBake(model, context);
@@ -280,13 +187,13 @@ public class ModelLoadingEventDispatcher {
}
@Nullable
- public BakedModel modifyModelAfterBake(@Nullable BakedModel model, Identifier id, UnbakedModel sourceModel, Function textureGetter, ModelBakeSettings settings, Baker baker) {
+ public BakedModel modifyModelAfterBake(@Nullable BakedModel model, @UnknownNullability Identifier resourceId, @UnknownNullability ModelIdentifier topLevelId, UnbakedModel sourceModel, Function textureGetter, ModelBakeSettings settings, Baker baker) {
if (afterBakeModifierContextStack.isEmpty()) {
afterBakeModifierContextStack.add(new AfterBakeModifierContext());
}
AfterBakeModifierContext context = afterBakeModifierContextStack.pop();
- context.prepare(id, sourceModel, textureGetter, settings, baker);
+ context.prepare(resourceId, topLevelId, sourceModel, textureGetter, settings, baker);
model = pluginContext.modifyModelAfterBake().invoker().modifyModelAfterBake(model, context);
@@ -357,15 +264,26 @@ public class ModelLoadingEventDispatcher {
}
private class OnLoadModifierContext implements ModelModifier.OnLoad.Context {
- private Identifier id;
+ @UnknownNullability
+ private Identifier resourceId;
+ @UnknownNullability
+ private ModelIdentifier topLevelId;
- private void prepare(Identifier id) {
- this.id = id;
+ private void prepare(@UnknownNullability Identifier resourceId, @UnknownNullability ModelIdentifier topLevelId) {
+ this.resourceId = resourceId;
+ this.topLevelId = topLevelId;
}
@Override
- public Identifier id() {
- return id;
+ @UnknownNullability("#topLevelId() != null")
+ public Identifier resourceId() {
+ return resourceId;
+ }
+
+ @Override
+ @UnknownNullability("#resourceId() != null")
+ public ModelIdentifier topLevelId() {
+ return topLevelId;
}
@Override
@@ -380,21 +298,32 @@ public class ModelLoadingEventDispatcher {
}
private class BeforeBakeModifierContext implements ModelModifier.BeforeBake.Context {
- private Identifier id;
+ @UnknownNullability
+ private Identifier resourceId;
+ @UnknownNullability
+ private ModelIdentifier topLevelId;
private Function textureGetter;
private ModelBakeSettings settings;
private Baker baker;
- private void prepare(Identifier id, Function textureGetter, ModelBakeSettings settings, Baker baker) {
- this.id = id;
+ private void prepare(@UnknownNullability Identifier resourceId, @UnknownNullability ModelIdentifier topLevelId, Function textureGetter, ModelBakeSettings settings, Baker baker) {
+ this.resourceId = resourceId;
+ this.topLevelId = topLevelId;
this.textureGetter = textureGetter;
this.settings = settings;
this.baker = baker;
}
@Override
- public Identifier id() {
- return id;
+ @UnknownNullability("#topLevelId() != null")
+ public Identifier resourceId() {
+ return resourceId;
+ }
+
+ @Override
+ @UnknownNullability("#resourceId() != null")
+ public ModelIdentifier topLevelId() {
+ return topLevelId;
}
@Override
@@ -419,14 +348,18 @@ public class ModelLoadingEventDispatcher {
}
private class AfterBakeModifierContext implements ModelModifier.AfterBake.Context {
- private Identifier id;
+ @UnknownNullability
+ private Identifier resourceId;
+ @UnknownNullability
+ private ModelIdentifier topLevelId;
private UnbakedModel sourceModel;
private Function textureGetter;
private ModelBakeSettings settings;
private Baker baker;
- private void prepare(Identifier id, UnbakedModel sourceModel, Function textureGetter, ModelBakeSettings settings, Baker baker) {
- this.id = id;
+ private void prepare(@UnknownNullability Identifier resourceId, @UnknownNullability ModelIdentifier topLevelId, UnbakedModel sourceModel, Function textureGetter, ModelBakeSettings settings, Baker baker) {
+ this.resourceId = resourceId;
+ this.topLevelId = topLevelId;
this.sourceModel = sourceModel;
this.textureGetter = textureGetter;
this.settings = settings;
@@ -434,8 +367,15 @@ public class ModelLoadingEventDispatcher {
}
@Override
- public Identifier id() {
- return id;
+ @UnknownNullability("#topLevelId() != null")
+ public Identifier resourceId() {
+ return resourceId;
+ }
+
+ @Override
+ @UnknownNullability("#resourceId() != null")
+ public ModelIdentifier topLevelId() {
+ return topLevelId;
}
@Override
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoaderPluginContextImpl.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoadingPluginContextImpl.java
similarity index 66%
rename from fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoaderPluginContextImpl.java
rename to fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoadingPluginContextImpl.java
index ba4ceec64..3231be50b 100644
--- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoaderPluginContextImpl.java
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoadingPluginContextImpl.java
@@ -23,15 +23,12 @@ import java.util.Map;
import java.util.Objects;
import java.util.Optional;
import java.util.Set;
-import java.util.function.Function;
-import org.jetbrains.annotations.Nullable;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import net.minecraft.block.Block;
import net.minecraft.client.render.model.UnbakedModel;
-import net.minecraft.client.util.ModelIdentifier;
import net.minecraft.registry.Registries;
import net.minecraft.registry.RegistryKey;
import net.minecraft.util.Identifier;
@@ -43,13 +40,11 @@ import net.fabricmc.fabric.api.client.model.loading.v1.ModelResolver;
import net.fabricmc.fabric.api.event.Event;
import net.fabricmc.fabric.api.event.EventFactory;
-public class ModelLoaderPluginContextImpl implements ModelLoadingPlugin.Context {
- private static final Logger LOGGER = LoggerFactory.getLogger(ModelLoaderPluginContextImpl.class);
+public class ModelLoadingPluginContextImpl implements ModelLoadingPlugin.Context {
+ private static final Logger LOGGER = LoggerFactory.getLogger(ModelLoadingPluginContextImpl.class);
final Set extraModels = new LinkedHashSet<>();
-
- private final Map blockStateResolvers = new HashMap<>();
- private final BlockKey lookupKey = new BlockKey();
+ final Map blockStateResolvers = new HashMap<>();
private final Event modelResolvers = EventFactory.createArrayBacked(ModelResolver.class, resolvers -> context -> {
for (ModelResolver resolver : resolvers) {
@@ -103,13 +98,7 @@ public class ModelLoaderPluginContextImpl implements ModelLoadingPlugin.Context
return model;
}, MODEL_MODIFIER_PHASES);
- /**
- * This field is used by the v0 wrapper to avoid constantly wrapping the context in hot code.
- */
- public final Function modelGetter;
-
- public ModelLoaderPluginContextImpl(Function modelGetter) {
- this.modelGetter = modelGetter;
+ public ModelLoadingPluginContextImpl() {
}
@Override
@@ -136,23 +125,12 @@ public class ModelLoaderPluginContextImpl implements ModelLoadingPlugin.Context
}
Identifier blockId = optionalKey.get().getValue();
- BlockKey key = new BlockKey(blockId.getNamespace(), blockId.getPath());
- BlockStateResolverHolder holder = new BlockStateResolverHolder(resolver, block, blockId);
- if (blockStateResolvers.put(key, holder) != null) {
- throw new IllegalArgumentException("Duplicate block state resolver for block " + blockId);
+ if (blockStateResolvers.put(blockId, resolver) != null) {
+ throw new IllegalArgumentException("Duplicate block state resolver for " + block);
}
}
- @Nullable
- BlockStateResolverHolder getBlockStateResolver(ModelIdentifier modelId) {
- BlockKey key = lookupKey;
- key.namespace = modelId.getNamespace();
- key.path = modelId.getPath();
-
- return blockStateResolvers.get(key);
- }
-
@Override
public Event resolveModel() {
return modelResolvers;
@@ -172,52 +150,4 @@ public class ModelLoaderPluginContextImpl implements ModelLoadingPlugin.Context
public Event modifyModelAfterBake() {
return afterBakeModifiers;
}
-
- private static class BlockKey {
- private String namespace;
- private String path;
-
- private BlockKey() {
- }
-
- private BlockKey(String namespace, String path) {
- this.namespace = namespace;
- this.path = path;
- }
-
- @Override
- public boolean equals(Object o) {
- if (this == o) return true;
- if (o == null || getClass() != o.getClass()) return false;
- BlockKey blockKey = (BlockKey) o;
- return namespace.equals(blockKey.namespace) && path.equals(blockKey.path);
- }
-
- @Override
- public int hashCode() {
- return 31 * namespace.hashCode() + path.hashCode();
- }
- }
-
- // Legacy v0 bridge - remove if the legacy v0 module is removed.
-
- private final Event legacyVariantProviders = EventFactory.createArrayBacked(LegacyModelVariantProvider.class, providers -> modelId -> {
- for (LegacyModelVariantProvider provider : providers) {
- try {
- UnbakedModel model = provider.loadModelVariant(modelId);
-
- if (model != null) {
- return model;
- }
- } catch (Exception exception) {
- LOGGER.error("Failed to run legacy model variant provider", exception);
- }
- }
-
- return null;
- });
-
- public Event legacyVariantProviders() {
- return legacyVariantProviders;
- }
}
diff --git a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/BakedModelManagerMixin.java b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/BakedModelManagerMixin.java
index f5c9b6488..27e7ce596 100644
--- a/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/BakedModelManagerMixin.java
+++ b/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/mixin/client/model/loading/BakedModelManagerMixin.java
@@ -28,11 +28,12 @@ import org.spongepowered.asm.mixin.Shadow;
import org.spongepowered.asm.mixin.injection.At;
import org.spongepowered.asm.mixin.injection.Redirect;
-import net.minecraft.class_9824;
import net.minecraft.client.render.model.BakedModel;
import net.minecraft.client.render.model.BakedModelManager;
+import net.minecraft.client.render.model.BlockStatesLoader;
import net.minecraft.client.render.model.ModelLoader;
import net.minecraft.client.render.model.json.JsonUnbakedModel;
+import net.minecraft.client.util.ModelIdentifier;
import net.minecraft.resource.ResourceManager;
import net.minecraft.resource.ResourceReloader;
import net.minecraft.util.Identifier;
@@ -41,16 +42,17 @@ import net.minecraft.util.profiler.Profiler;
import net.fabricmc.fabric.api.client.model.loading.v1.FabricBakedModelManager;
import net.fabricmc.fabric.api.client.model.loading.v1.ModelLoadingPlugin;
+import net.fabricmc.fabric.impl.client.model.loading.ModelLoadingConstants;
import net.fabricmc.fabric.impl.client.model.loading.ModelLoadingPluginManager;
@Mixin(BakedModelManager.class)
-public class BakedModelManagerMixin implements FabricBakedModelManager {
+abstract class BakedModelManagerMixin implements FabricBakedModelManager {
@Shadow
- private Map models;
+ private Map models;
@Override
public BakedModel getModel(Identifier id) {
- return models.get(id);
+ return models.get(ModelLoadingConstants.toResourceModelId(id));
}
@Redirect(
@@ -63,8 +65,8 @@ public class BakedModelManagerMixin implements FabricBakedModelManager {
allow = 1)
private CompletableFuture loadModelPluginData(
CompletableFuture