Zrips / CMI

114 stars 97 forks source link

Spawner Silktouch #8700

Open WhiteProject1 opened 4 months ago

WhiteProject1 commented 4 months ago

Description of Issue

https://github.com/Zrips/CMI/assets/103301275/f195017d-f939-42a2-b621-23b01f9b1769

Version Information

on video

Errors

on video

Relevant Config Sections

Spawners:
  # If you experiencing issues with spawner handling, set this to true to avoid any spawner manipulations from CMI side
  # This will disable features like, spawner placement, spawner drops, spawner charges and so on
  FullDisable: false
  Break:
    # ATTENTION! cmi.dropspawner will be required to get spawner dropping for the player
    # Enable or disable spawner handler for spawner break
    # If enabled player will get spawner if using silktouch pickaxe and have cmi.dropspawner permission node
    # If player has cmi.dropspawner.nosilk permission node, player is not required to use silk touch pickaxe to get droped spawner
    Enabled: true
    # If set to true, player will need to have particular permission node to break and get particular spawner.
    # In example: player should have cmi.dropspawner.pig to break pig spawner and get it dropped, or cmi.dropspawner.zombie to get zombie spawner
    RequiresExactPermission: false
    # When set to false, exp will not be dropped from broken spawner independent if spawner it self is being dropped
    # When set to true exp will be dropped only if spawner is not
    DropExp: false
    # Minimal silktouch level required to get spawner back
    SilkTouchLevel: 1
    # Number in percentage from 0 up to 100 for a spawner to be dropped when you break one
    BaseDropChance: 100
  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: false
    # 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
    # If set to false, then player will need to have basic cmi.placespawner permission to place any type of spawner
    RequiresExactPermission: false
  Interact:
    # When set to true, players trying to change spawner with monster egg will require appropriate permission node
    # In example: player should have cmi.egginteract.pig to change spawner into pig, or cmi.egginteract.zombie to change into zombie spawner
    EggRequiresPermission: false

Relevant Plugins

Vault 1.7.3

Agreements

Zrips commented 3 months ago

Can you test this one without any other plugin in case there is another one which drops additional spawner when you break it with silk touch? And I need output from /cmi version to have better idea of your setup

WhiteProject1 commented 3 months ago

image there is nothing about spawner break image

Zrips commented 3 months ago

Try disabling CMI spawner break handling and check if you still get spawner back when you mine one

WhiteProject1 commented 3 months ago

I turned off the spawner modules, it only gives xp image

charlieslost commented 3 months ago

I am still getting this issue on the latest version running 1.20.6, when disabled no spawner drops, when enabled the same effect as the video happens

Siqued commented 3 months ago

Should be fixed with next update. Posted this issue some days ago

WhiteProject1 commented 3 months ago

month a go brother I just want support and fix but zrips not doing

Siqued commented 3 months ago

month a go brother I just want support and fix but zrips not doing

Try and be him. He is doing everything he can to help us. He can’t do all bug fixes in a day.

And as i Said he should just have fixed the bug. So it will hopefully work next update

WhiteProject1 commented 3 months ago

Months bro not day

Siqued commented 3 months ago

Months bro not day

Just be happy that he wants to help with that attitude