Suggestion Open

grindstone, 2 bugs.

#6472 opened by Unknown 1 year ago

Unknownreported this suggestion

Summary

bug 1 Grindstone ignores the "Repairable: false" item Option

bug 2 Grindstone resets "Repair Cost" this is a vanilla feature | please add an option to disable the use of grind stone on said item like the "PreventAnvil" option

Steps to reproduce

Bug 1:

[Legacy attachment omitted]

item config:

  Id: bow
  Display: Repair Bug Bow
  MaxDurability: 300
  Durability: 200
  Options:
    PreventCrafting: true
    PreventAnvil: true
    Repairable: false
  Skills:
  - message{message=hi} @self ~onHold

This will make a bow that

  • Can't be repair via Crafting, or Anvil, but can be repaired with the Grindstone
  • Once it goes through the grind stone it can be repaired at an Anvil
  • Skills will continue to work

Bug 2

[Legacy attachment omitted]

item config:

  Id: bow
  Display: Repair 10 Cost Bow
  MaxDurability: 300
  Durability: 200
  RepairCost: 10
  Options:
    PreventCrafting: true
    PreventSmithing: true
    PreventAnvil: false
    PreventEnchanting: true
  Skills:
  - message{message="hi again"} @self ~onHold

This will make a bow that

  • Can't be repair via Crafting, can be repaired on an Anvil at a high exp cost. But can be repaired with the Grindstone for free; AND it will reset the repair cost of the item as it's the vanilla behavior of Grindstones third line below "Repairing and disenchanting"
  • Skills will continue to work

Current behavior

Bug 1

Makes what should be "unrepairable" items repairable

Bug 2

Makes what should be highly expensive to repair items free to repair

Intended correct behavior & Proposed fixes

Bug 1

Grindstone respects the "Repairable: false" option, and isn't repairable

Bug 2

Request to add an "PreventGrindstone:true" to make items that can ONLY be repaired at an Anvil, and another usage of making it unable to be disenchanted

Versions

[Legacy attachment omitted]

Attached Files

[Legacy attachment omitted] [Legacy attachment omitted] [Legacy attachment omitted]

Unknowncommented

This should be on higher priority.

Unknowncommented

agree, hope they fix this soon, there was another post about grindstones 4 years ago. But it was for adding the "preventgrindstone:true/false" option, which would indirectly solve this issue but oh well.

Unknowncommented

Grindstones can still remove the no-enchanting/repairing tag, which makes me really angry because ItemsAdder items should prevent this by default—at least I don't remember setting this up specifically. I've looked through a lot of documentation in MythicMobs but still couldn't solve it, only finding an unresolved issue from four years ago.

Sign in to join the discussion.