Fabric API

Fabric API

Mods
111M Downloads

ModelLoadingEventDispatcher.BlockStateResolverContext#setModel() does not null-check the given blockstate

XFactHD opened this issue ยท 0 comments

commented

ModelLoadingEventDispatcher.BlockStateResolverContext#setModel() null-checks the given model a second time instead of null-checking the given blockstate:

fabric/fabric-model-loading-api-v1/src/client/java/net/fabricmc/fabric/impl/client/model/loading/ModelLoadingEventDispatcher.java

Lines 180 to 182 in d70d2c0

public void setModel(BlockState state, GroupableModel model) {
Objects.requireNonNull(model, "state cannot be null");
Objects.requireNonNull(model, "model cannot be null");

FAPI version: 0.112.2+1.21.4