Add PlayerUseBowWithoutProjectileEvent #11774
Conversation
paper-api/src/main/java/io/papermc/paper/event/player/PlayerUseBowWithoutProjectileEvent.java
Outdated
Show resolved
Hide resolved
| return item.clone(); | ||
| } |
There was a problem hiding this comment.
Because this event is only specific to bows, I think that this should be getBowItem() or something more specific
There was a problem hiding this comment.
tries to draw a bow or load a crossbow
There was a problem hiding this comment.
We might be able to find a better name I guess, but certainly not related to bows xD
There was a problem hiding this comment.
I meant bows, which applies to both crossbows/bow. But not sure, maybe leave as is then.
There was a problem hiding this comment.
Guess something like weapon item?
There was a problem hiding this comment.
Cant you use anything as "bow" now?
|
Maybe ProjectileWeaponItem, because thats the superclass of both bow and crossbow, although that's a really long name. PlayerMissingProjectileItem? (And isn't PlayerReadyArrowEvent confusing then, because its fired for crossbows and firework rockets aswell?) |
… adjust javadocs, fix using old parameter name when calling event in Player#getProjectile
#11669
(Still dont have a better name for the event)