Jump to content

[1.18.2] Datapack registry not populating


SeerUnseeing

Recommended Posts

I'm fiddling with a fork of Create Ore Excavation to tailor it to my tastes, and I've hit an issue. The idea is to separate ore veins from the recipes, to allow for more depth. To this end, I've added a new data-driven registry to store the vein types, which are then used to populate a chunk capability for later use.

When the server starts up, however, the data pack entries are loaded and the registry entry constructor called, but no entries seem to actually be added to the registry. There are no errors in the log, and from debugging it looks like the entries should be being added. Despite that, when I check the size and content of the registry with a debug item, it's empty.

I'm not sure if I horribly misunderstood the way these are set up, or if I'm just missing something; any help would be greatly appreciated.

The code in my main class (irrelevant lines omitted):

  //<...>
  
  public static final DeferredRegister<VeinType> VEINS_REGISTRY = DeferredRegister.create(new ResourceLocation(MODID, "vein_types"), MODID);
  public static final Supplier<IForgeRegistry<VeinType>> VEINS = VEINS_REGISTRY.makeRegistry(VeinType.class, ()->{
    LOGGER.info("Building ore vein registry...");
    return new RegistryBuilder<VeinType>()
      .onAdd((owner, stage, id, obj, oldObj)->LOGGER.debug("Registered ore vein: " + obj.getRegistryName())) //This is never hit
      .dataPackRegistry(VeinType.VeinCodex,VeinType.VeinCodex);
  });

  public CreateOreExcavation() {

    //<...>

    MinecraftForge.EVENT_BUS.register(this);

    VEINS_REGISTRY.register(FMLJavaModLoadingContext.get().getModEventBus());

  }
  
  public static IForgeRegistry<VeinType> getVeinRegistry(){return VEINS.get();}

  //<...>

The registry entity class, VeinType:

public class VeinType extends ForgeRegistryEntry<VeinType> {

  int weight;

  public TagKey<Biome> biomeAllowlist, biomeDenylist;

  public VeinType(int weight, Optional<TagKey<Biome>> allowlist, Optional<TagKey<Biome>> denylist){
    this.weight = weight;
    this.biomeAllowlist=allowlist.orElse(null);
    this.biomeDenylist =denylist.orElse(null);
    CreateOreExcavation.LOGGER.info("VeinType constructor was called with weight " + weight); // This triggers
  }

  public int getWeight(){return weight;}
  public Optional<TagKey<Biome>> getAllowList(){return Optional.ofNullable(biomeAllowlist);}
  public Optional<TagKey<Biome>> getDenyList(){return Optional.ofNullable(biomeDenylist);}

  public boolean canGenerate(ServerLevel lvl, Holder<Biome> b) {
    if(biomeDenylist != null  && isInTag(biomeDenylist, lvl, b))return false;
    return biomeAllowlist == null || isInTag(biomeAllowlist, lvl, b);
    
  }

  private static boolean isInTag(TagKey<Biome> tag, ServerLevel lvl, Holder<Biome> b) {
    return lvl.getServer().registryAccess().registryOrThrow(Registry.BIOME_REGISTRY).getTag(tag).map(t -> t.contains(b)).orElse(false);
  }

  public static final Codec<VeinType> VeinCodex = RecordCodecBuilder.create(instance -> instance.group(
      Codec.INT.fieldOf("weight").forGetter(VeinType::getWeight),
      TagKey.codec(Registry.BIOME_REGISTRY).optionalFieldOf("biome_allowlist").forGetter(VeinType::getAllowList),
      TagKey.codec(Registry.BIOME_REGISTRY).optionalFieldOf("biome_denylist") .forGetter(VeinType::getDenyList)
          ).apply(instance, VeinType::new));
}
Link to comment
Share on other sites

Please in future put your code on github so we can see all the relevant context and make it easier to try it for ourselves.

 

The registration is working fine for me.

I changed your code a bit to make it easier to use (but I don't think this makes any difference):

    public static final ResourceKey<Registry<VeinType>> VEINS_KEY = ResourceKey.<VeinType>createRegistryKey(new ResourceLocation(MODID, "vein_types"));
    public static final DeferredRegister<VeinType> VEINS_REGISTRY = DeferredRegister.create(VEINS_KEY, MODID);

I also added a programmatically registered entry:

Quote

    public static RegistryObject<VeinType> VEIN = VEINS_REGISTRY.register("test", () -> new VeinType(20, Optional.empty(), Optional.empty()));

Then I list the registry at server started

@Mod.EventBusSubscriber(modid = MODID)
public class Events {
    @SubscribeEvent
    public static void serverStarting(ServerStartedEvent event) {
        var registry = event.getServer().registryAccess().registryOrThrow(VEINS_KEY);
        LOGGER.info("------------> " + registry.key());
        registry.forEach(entry -> LOGGER.info(registry.getKey(entry) + " " + entry.weight));
    }

Then with a data/examplemod/examplemod/vein_types/myvein.json

Quote

{
    "weight": 10
}

Which gives:

[09:32:27] [Server thread/INFO] [co.ex.ex.ExampleMod/]: ------------> ResourceKey[minecraft:root / examplemod:vein_types]
[09:32:27] [Server thread/INFO] [co.ex.ex.ExampleMod/]: examplemod:test 20
[09:32:27] [Server thread/INFO] [co.ex.ex.ExampleMod/]: examplemod:myvein 10

 

I guess the onAdd() doesn't work for datapack registries because they are "vanilla" and not real ForgeRegistrys?

But I might be wrong about that.

Edited by warjort

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.