Implement Releaseable::getMinUseDuration()

[remminiscent]

Implemented a new function getMinUseDuration in the Releaseable interface which will return the minimum needed ticks to use the item for an action to occur.

Also, regarding the Bow, should it be 5 ticks, or 0?

Related issues & PRs

• Fixes Min use duration for Releasable #6653

Changes

API changes

• Added Releaseable::getMinUseDuration

Backwards compatibility

Nope, It's BC Breaking. good for major-next branch

Tests

Works 👍
https://streamable.com/2l8zru

[kostamax27]

It might be worth moving this to PlayerItemConsumeEvent, similar to how hasItemCooldown is handled below.

[dktapps]

I'm not sure how I want to handle this tbh. Interfaces like this have been a constant pain in the ass for BC breaks.

Considering we already have onReleaseUsing() in Item anyway, and #6928 is adding continueUsing() to Item directly for similar reasons, we should probably make sure the approaches for this are consistent.

[remminiscent]

I'm not sure how I want to handle this tbh. Interfaces like this have been a constant pain in the ass for BC breaks.

Considering we already have onReleaseUsing() in Item anyway, and #6928 is adding continueUsing() to Item directly for similar reasons, we should probably make sure the approaches for this are consistent.

so do i /close this PR? ☹️

or, in my opinion, we have a parent ChargeableItem class for food, bows, crossbows, trident, etc. no need for bunch of interfaces like Chargeable & Releaseable

[dktapps]

Bow would have to extend both Durable and Chargeable, that's not gonna work

[remminiscent]

Bow would have to extend both Durable and Chargeable, that's not gonna work

You're right. well, i'll move onto making other PR's until this gets a plan