Skip to content

Add EntitySpinAttackEvent#12808

Closed
Clexus wants to merge 3 commits into
PaperMC:mainfrom
Clexus:spinattack
Closed

Add EntitySpinAttackEvent#12808
Clexus wants to merge 3 commits into
PaperMC:mainfrom
Clexus:spinattack

Conversation

@Clexus
Copy link
Copy Markdown
Contributor

@Clexus Clexus commented Jul 6, 2025

Useful when user want to detect "an entity collides with another entity using a trident" or what

@Clexus Clexus requested a review from a team as a code owner July 6, 2025 10:04
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Jul 6, 2025
* calls {@link org.bukkit.event.entity.EntityDamageByEntityEvent}.
*/
@NullMarked
public class EntitySpinAttackEvent extends EntityEvent implements Cancellable {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EntityRiptideAttackEvent given we call it riptiding in most other places

Comment on lines +1549 to +1550
+ if (entity instanceof LivingEntity livingEntity) { // Paper - move LivingEntity conversion ahead
+ // Paper start - fire EntitySpinAttackEvent
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one paper start comment above the if statement is enough

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one paper start comment above the if statement is enough

my repo was broken so I made a new pr #12842

@github-project-automation github-project-automation Bot moved this from Awaiting review to Changes required in Paper PR Queue Jul 13, 2025
@Clexus Clexus closed this by deleting the head repository Jul 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Changes required

Development

Successfully merging this pull request may close these issues.

2 participants