Minecraft General

Worked that was what I needed.... added to MyModTags class where I handle all my tag group stuff [yeah, stuff, I could not think of a better word]. public static Tag<Item> makeWrapperTag(String p_199901_0_) { return new ItemTags.Wrapper(new ResourceLocation(p_199901_0_)); } then where I needed to use the tagged items i did something like this.... Ingredient CARNIVORE_TAGED_ITEMS = Ingredient.fromTag(makeWrapperTag("carnivore")); goalSelector.addGoal(4, new TemptGoal(this, 1.2D, false, CARNIVORE_TAGED_ITEMS)); Well I went around the world to get here, thanks again Draco18s and thanks Alpvax. Doing this all the tagged items were found from inside the mod and from the datapack. Both those I put under minecraft and under my mod [see earlier comments above]

Yes, that's what a soup is...

The event provides the entity that is being hit, not the entity that is holding my item. Storing the data like that is working, but once the item is dropped it will still block the damage to the player

maybe is because its container is bowl? if (stack_container == Items.BOWL) { ReturnUseAction = UseAction.DRINK;}  

You can't really do this. When the class is first called by the event bus loader, it will declare these variables prior to the registry phase. So, it will initialize both of these values to null. Just create the variables locally in the event since you're not using them anywhere else.    Two more things, if you need to cast the value to BlockState, you're doing BlockStates wrong. Second, everything must be wrapped in a DeferredWorkQueue as everything you're doing is not thread-safe.