Slimefun

Slimefun

Bukkit Plugins
3M Downloads

Better NBT handling for spawners (CMI compatibility)

bobhenl opened this issue · 0 comments

commented

📍 Description (REQUIRED)

Hi, as reported here Zrips/CMI#4605 on CMI issue tracker, there was problem when I had enabled place section in their config and then wanted to place specific type of Reinforced spawner, it didn't work.

But CMI isn't needed probably, as Zrips mentioned there, this is how REINFORCED SPAWNER nbt look:
image
And this is, how it should look, there's missing some important tags.
image

📑 Steps to reproduce the Issue (REQUIRED)

  1. You don't need to use CMI? just look at nbt tags and try to fix.

IF you decide to reproduce with CMI, then:

  1. Set
  Place:
    # Enable or disable spawner handler for spawner place
    # If enabled player will place spawner depending from what it is by its type
    # If disabled then spawner will be placed in normal way and it will allow other plugins to handle its placement
    Enabled: true
    # If set to true, player will need to have appropriate permission node to place spawner by its type
    RequiresPermission: true
    # RequiresPermission should be set to true for this to work. If set to true, player will need to have particular permission node to place particular spawner.
    # In example: player should have cmi.placespawner.pig to place pig spawner, or cmi.placespawner.zombie to place zombie spawner

and then try to place some reinforced spawner... you will see it's pig always, even when you place spider spawner.

💡 Expected behavior (REQUIRED)

Spider spawner should be placed as spider spawner, not the pig one. Better NBT mentioned in nbt tags.

🧭 Environment (REQUIRED)

  • Server Software (Spigot/Paper): Paper
  • Minecraft Version: 1.16.4
  • Slimefun Version: 757
  • CS-CoreLib Version: 91
    CMI: 8.7.11.1