Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Knockback #2094

Merged
merged 9 commits into from
Dec 21, 2019
Merged
Show file tree
Hide file tree
Changes from 6 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ public static void init() {
ScriptEvent.registerScriptEvent(new PlayerEquipsArmorScriptEvent());
ScriptEvent.registerScriptEvent(new PlayerJumpsPaperScriptEventImpl());
ScriptEvent.registerScriptEvent(new PlayerSpectatesEntityScriptEvent());
ScriptEvent.registerScriptEvent(new EntityKnocksbackEntityScriptEvent());
Copy link
Member

Choose a reason for hiding this comment

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

This list should be alphabetized (just look at the order of events in your IntelliJ class listing)

ScriptEvent.registerScriptEvent(new PlayerStopsSpectatingScriptEvent());
ScriptEvent.registerScriptEvent(new ProjectileCollideScriptEvent());
ScriptEvent.registerScriptEvent(new TNTPrimesScriptEvent());
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,130 @@
package com.denizenscript.denizen.paper.events;

import com.denizenscript.denizen.events.BukkitScriptEvent;
import com.denizenscript.denizen.objects.EntityTag;
import com.denizenscript.denizen.objects.LocationTag;
import com.denizenscript.denizen.utilities.implementation.BukkitScriptEntryData;
import com.denizenscript.denizencore.objects.ObjectTag;
import com.denizenscript.denizen.objects.ItemTag;
import com.denizenscript.denizencore.scripts.ScriptEntryData;
import com.denizenscript.denizencore.scripts.containers.ScriptContainer;
import com.denizenscript.denizencore.utilities.CoreUtilities;
import com.destroystokyo.paper.event.entity.EntityKnockbackByEntityEvent;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;

public class EntityKnocksbackEntityScriptEvent extends BukkitScriptEvent implements Listener {

// <--[event]
// @Events
// entity knocks back entity
// entity knocks back <entity>
// <entity> knocks back entity
// <entity> knocks back <entity>
//
// @Regex ^on [^\s]+ knocks back [^\s]+$
//
// @Switch in:<area> to only process the event if it occurred within a specified area.
// @Switch with:<item> to only process the event when the item used to cause damage (in the damager's hand) is a specified item.
//
// @Plugin Paper
//
// @Cancellable true
//
// @Triggers when an entity is knocked back from the hit of another entity.
//
// @Context
// <context.entity> returns the EntityTag that was knocked back.
// <context.damager> returns the EntityTag of the one who knocked.
// <context.acceleration> returns the knockback applied as a vector.
//
// @Player when the damager or damaged entity is a player. Cannot be both.
//
// @NPC when the damager or damaged entity is an NPC. Cannot be both.
//
// -->

public EntityKnocksbackEntityScriptEvent() {
instance = this;
}

public static EntityKnocksbackEntityScriptEvent instance;

public EntityTag entity;
public EntityTag hitBy;
public ItemTag held;
public EntityKnockbackByEntityEvent event;

@Override
public boolean couldMatch(ScriptPath path) {
return path.eventArgLowerAt(1).equals("knocks") &&
path.eventArgLowerAt(2).equals("back");
}

@Override
public boolean matches(ScriptPath path) {
String attacker = path.eventArgLowerAt(0);
String target = path.eventArgLowerAt(3);
if (!tryEntity(hitBy, attacker) || (!tryEntity(entity, target))) {
return false;
}
if (!runInCheck(path, entity.getLocation())) {
return false;
}
if (!runWithCheck(path, held)) {
return false;
}
return super.matches(path);
}

@Override
public String getName() {
return "EntityKnocksbackEntity";
}

@Override
public boolean applyDetermination(ScriptPath path, ObjectTag determinationObj) {
String determination = determinationObj.toString();
Copy link
Member

Choose a reason for hiding this comment

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

Goes inside the if, not outside it (only used inside)

if (!isDefaultDetermination(determinationObj)) {
if (determinationObj instanceof LocationTag && LocationTag.matches(determination)) {
Copy link
Member

Choose a reason for hiding this comment

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

Nope. ||. If it's already a LocationTag or it can become a LocationTag.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ohhhh! I get it now!

event.getAcceleration().copy(((LocationTag) determinationObj).toVector());
Copy link
Member

Choose a reason for hiding this comment

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

need a valueOf, at least if instanceOf failed and matches passed instead.

Alternately you could remove the instanceOf and only do matches + valueOf (would be cleaner tbh)

return true;
}
}
return super.applyDetermination(path, determinationObj);
}

@Override
public ScriptEntryData getScriptEntryData() {
return new BukkitScriptEntryData(
hitBy.isPlayer() ? hitBy.getDenizenPlayer() :
(entity.isPlayer() ? entity.getDenizenPlayer() : null),

Copy link
Member

Choose a reason for hiding this comment

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

Stray doublenewline ^
Also the newlines on the individual parts

I'm confused why this isn't formatted exactly the same as the EntityDamaged event, which I think is what you were referencing anyway?

        return new BukkitScriptEntryData(damager != null && damager.isPlayer() ? damager.getDenizenPlayer() : entity.isPlayer() ? entity.getDenizenPlayer() : null,
                damager != null && damager.isCitizensNPC() ? damager.getDenizenNPC() : entity.isCitizensNPC() ? entity.getDenizenNPC() : null);

hitBy.isCitizensNPC() ? hitBy.getDenizenNPC() :
(entity.isCitizensNPC() ? entity.getDenizenNPC() : null)
);
}

@Override
public ObjectTag getContext(String name) {
if (name.equals("entity")) {
return entity.getDenizenObject();
}
else if (name.equals("damager")) {
return hitBy.getDenizenObject();
}
else if (name.equals("acceleration")) {
return new LocationTag(event.getAcceleration());
}
return super.getContext(name);
}

@EventHandler
public void onEntityKnockbackEntity(EntityKnockbackByEntityEvent event) {
entity = new EntityTag(event.getEntity());
hitBy = new EntityTag(event.getHitBy());
held = hitBy.getItemInHand();
this.event = event;
fire(event);
}
}