Hex Casting

Hex Casting

Mods
6M Downloads

Quilt: Brainswept villagers stop being brainswept when released from a Supplementaries cage

object-Object opened this issue ยท 1 comments

commented

On Quilt, when a brainswept villager is captured then released with a cage from Supplementaries, it is no longer brainswept. On Fabric, brainswept villagers can't be caged at all. (Edit: Or in my dev environment on Fabric it can be caged just fine. I have no idea what's happening anymore.)

Specifically, the tag cardinal_components: {"hexcasting:brainswept": {brainswept: 1b}} on the initial villager becomes cardinal_components: {} on the data in the cage item, so apparently the CC data isn't being saved.

Here's the line in Supplementaries where, I think, the mob data is fetched:
https://github.com/MehVahdJukaar/Supplementaries/blob/fb7073e114a8a94f166c28098b840b40d81d510f/src/main/java/net/mehvahdjukaar/supplementaries/common/capabilities/mobholder/MobContainer.java#L353

Repro:

  • Set "allow_all_mobs": true in the Supplementaries config.
  • Brainsweep a villager.
  • Right click it with a cage, then right click the ground to release it.

Modlist:

  • Cardinal Components API 5.0.2
  • Cloth Config 8.2.88
  • Hex Casting 0.10.3
  • Moonlight 1.19.2-2.2.37
  • Patchouli 1.19.2-77-FABRIC
  • PAUCAL 0.5.0
  • Quilt Kotlin Libraries 1.0.2
  • Quilted Fabric API 4.0.0-beta-30+0.76.0-1.19.2
  • Supplementaries 1.19.2-2.3.16

More details here: MehVahdJukaar/Supplementaries#631

commented

I still have no idea what causes this, but we should check and see if it still happens on 1.20.