core
/
testserver
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
testserver/Spigot-Server-Patches/0283-Add-SentientNPC-Interf...

166 lines
7.2 KiB
Diff
Raw Normal View History

[Auto] Update Upstream Upstream has released updates that appear to apply compile correctly. This update has not been tested by PaperMC and as with ANY update, please do your own testing. Bukkit Changes: d2834556 SPIGOT-4219: Event for PigZombies angering. CraftBukkit Changes: a9c796f1 SPIGOT-4184: Fix furnaces not matching Vanilla smelt or animations 195f071e SPIGOT-4219: Event for PigZombies angering. 5e3082c7 SPIGOT-4230: Improve legacy block types
2018-08-05 23:46:43 +00:00
From 08a589cb3acd47ab831c3c1e1236821f94b86de0 Mon Sep 17 00:00:00 2001
Add Critical missing Bukkit API - setTarget/getTarget moved down to SentientNPC This fixes a CRITICAL missing part of the Bukkit API due to mistakes on upstream refusing to implement the Sentient NPC baseclass of all NPC's. Until now, the Bukkit API has not provided a way for accessing and setting a non creature entities target. Although Flying, Slime, Ambient, and Water mobs all supported targets internally, you were unable to get/set it. Now with the SentientNPC API and these API's moved down, every sentient NPC has access to target data.
2018-07-02 02:07:35 +00:00
From: Aikar <aikar@aikar.co>
Date: Sun, 1 Jul 2018 22:06:29 -0400
Subject: [PATCH] Add SentientNPC Interface to Entities
Used to determine ACTUAL Living NPC's. Spigot mistakenly inversed the conditions for LivingEntity, and
used LivingEntity for Insentient Entities, and named the actual EntityLiving class EntityInsentient.
This should of all been inversed on the implementation side. To make matters worse, Spigot never
exposed the differentiator that there are entities with AI that are not sentient/alive such as
Armor stands and Players are the only things that do not implement the REAL EntityLiving class (named Insentient internally)
This interface lets you identify NPC entities capable of sentience, and able to move about and react to the world.
diff --git a/src/main/java/com/destroystokyo/paper/entity/CraftSentientNPC.java b/src/main/java/com/destroystokyo/paper/entity/CraftSentientNPC.java
new file mode 100644
Update B/CB/S
2018-07-23 08:39:55 +00:00
index 0000000000..a60ba13495
Add Critical missing Bukkit API - setTarget/getTarget moved down to SentientNPC This fixes a CRITICAL missing part of the Bukkit API due to mistakes on upstream refusing to implement the Sentient NPC baseclass of all NPC's. Until now, the Bukkit API has not provided a way for accessing and setting a non creature entities target. Although Flying, Slime, Ambient, and Water mobs all supported targets internally, you were unable to get/set it. Now with the SentientNPC API and these API's moved down, every sentient NPC has access to target data.
2018-07-02 02:07:35 +00:00
--- /dev/null
+++ b/src/main/java/com/destroystokyo/paper/entity/CraftSentientNPC.java
@@ -0,0 +1,25 @@
+package com.destroystokyo.paper.entity;
+
+import net.minecraft.server.EntityInsentient;
+import org.bukkit.craftbukkit.entity.CraftLivingEntity;
+import org.bukkit.entity.LivingEntity;
+
+public interface CraftSentientNPC <T extends EntityInsentient> extends SentientNPC {
+ T getHandle();
+
+ default public void setTarget(LivingEntity target) {
+ T entity = getHandle();
+ if (target == null) {
+ entity.setGoalTarget(null, null, false);
+ } else if (target instanceof CraftLivingEntity) {
+ entity.setGoalTarget(((CraftLivingEntity) target).getHandle(), null, false);
+ }
+ }
+
+ default public LivingEntity getTarget() {
+ if (getHandle().getGoalTarget() == null) return null;
+
+ return (CraftLivingEntity) getHandle().getGoalTarget().getBukkitEntity();
+ }
+
+}
diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftAmbient.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftAmbient.java
Update B/CB/S
2018-07-23 08:39:55 +00:00
index 086980e76d..ccce080ab8 100644
Add Critical missing Bukkit API - setTarget/getTarget moved down to SentientNPC This fixes a CRITICAL missing part of the Bukkit API due to mistakes on upstream refusing to implement the Sentient NPC baseclass of all NPC's. Until now, the Bukkit API has not provided a way for accessing and setting a non creature entities target. Although Flying, Slime, Ambient, and Water mobs all supported targets internally, you were unable to get/set it. Now with the SentientNPC API and these API's moved down, every sentient NPC has access to target data.
2018-07-02 02:07:35 +00:00
--- a/src/main/java/org/bukkit/craftbukkit/entity/CraftAmbient.java
+++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftAmbient.java
@@ -1,11 +1,12 @@
package org.bukkit.craftbukkit.entity;
+import com.destroystokyo.paper.entity.CraftSentientNPC;
import net.minecraft.server.EntityAmbient;
import org.bukkit.craftbukkit.CraftServer;
import org.bukkit.entity.Ambient;
import org.bukkit.entity.EntityType;
-public class CraftAmbient extends CraftLivingEntity implements Ambient {
+public class CraftAmbient extends CraftLivingEntity implements Ambient, CraftSentientNPC { // Paper
public CraftAmbient(CraftServer server, EntityAmbient entity) {
super(server, entity);
}
diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftComplexLivingEntity.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftComplexLivingEntity.java
Update B/CB/S
2018-07-23 08:39:55 +00:00
index cc115cc368..3a4e6f0c7e 100644
Add Critical missing Bukkit API - setTarget/getTarget moved down to SentientNPC This fixes a CRITICAL missing part of the Bukkit API due to mistakes on upstream refusing to implement the Sentient NPC baseclass of all NPC's. Until now, the Bukkit API has not provided a way for accessing and setting a non creature entities target. Although Flying, Slime, Ambient, and Water mobs all supported targets internally, you were unable to get/set it. Now with the SentientNPC API and these API's moved down, every sentient NPC has access to target data.
2018-07-02 02:07:35 +00:00
--- a/src/main/java/org/bukkit/craftbukkit/entity/CraftComplexLivingEntity.java
+++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftComplexLivingEntity.java
@@ -1,17 +1,19 @@
package org.bukkit.craftbukkit.entity;
+import com.destroystokyo.paper.entity.CraftSentientNPC;
+import net.minecraft.server.EntityInsentient;
import net.minecraft.server.EntityLiving;
import org.bukkit.craftbukkit.CraftServer;
import org.bukkit.entity.ComplexLivingEntity;
-public abstract class CraftComplexLivingEntity extends CraftLivingEntity implements ComplexLivingEntity {
+public abstract class CraftComplexLivingEntity extends CraftLivingEntity implements ComplexLivingEntity, CraftSentientNPC { // Paper
public CraftComplexLivingEntity(CraftServer server, EntityLiving entity) {
super(server, entity);
}
@Override
- public EntityLiving getHandle() {
- return (EntityLiving) entity;
+ public EntityInsentient getHandle() { // Paper
+ return (EntityInsentient) entity; // Paper
}
@Override
diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftCreature.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftCreature.java
Update B/CB/S
2018-07-23 08:39:55 +00:00
index 09d42141fb..30004e5e8c 100644
Add Critical missing Bukkit API - setTarget/getTarget moved down to SentientNPC This fixes a CRITICAL missing part of the Bukkit API due to mistakes on upstream refusing to implement the Sentient NPC baseclass of all NPC's. Until now, the Bukkit API has not provided a way for accessing and setting a non creature entities target. Although Flying, Slime, Ambient, and Water mobs all supported targets internally, you were unable to get/set it. Now with the SentientNPC API and these API's moved down, every sentient NPC has access to target data.
2018-07-02 02:07:35 +00:00
--- a/src/main/java/org/bukkit/craftbukkit/entity/CraftCreature.java
+++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftCreature.java
@@ -1,16 +1,18 @@
package org.bukkit.craftbukkit.entity;
+import com.destroystokyo.paper.entity.CraftSentientNPC;
import net.minecraft.server.EntityCreature;
import org.bukkit.craftbukkit.CraftServer;
import org.bukkit.entity.Creature;
import org.bukkit.entity.LivingEntity;
-public class CraftCreature extends CraftLivingEntity implements Creature {
+public class CraftCreature extends CraftLivingEntity implements Creature, CraftSentientNPC { // Paper
public CraftCreature(CraftServer server, EntityCreature entity) {
super(server, entity);
}
- public void setTarget(LivingEntity target) {
+ // Paper start - move down to SentientNPC
+ /*public void setTarget(LivingEntity target) {
EntityCreature entity = getHandle();
if (target == null) {
entity.setGoalTarget(null, null, false);
@@ -23,7 +25,8 @@ public class CraftCreature extends CraftLivingEntity implements Creature {
if (getHandle().getGoalTarget() == null) return null;
return (CraftLivingEntity) getHandle().getGoalTarget().getBukkitEntity();
- }
+ }*/
+ // Paper end
@Override
public EntityCreature getHandle() {
diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftFlying.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftFlying.java
Update B/CB/S
2018-07-23 08:39:55 +00:00
index f374c7b880..9e6f523bf1 100644
Add Critical missing Bukkit API - setTarget/getTarget moved down to SentientNPC This fixes a CRITICAL missing part of the Bukkit API due to mistakes on upstream refusing to implement the Sentient NPC baseclass of all NPC's. Until now, the Bukkit API has not provided a way for accessing and setting a non creature entities target. Although Flying, Slime, Ambient, and Water mobs all supported targets internally, you were unable to get/set it. Now with the SentientNPC API and these API's moved down, every sentient NPC has access to target data.
2018-07-02 02:07:35 +00:00
--- a/src/main/java/org/bukkit/craftbukkit/entity/CraftFlying.java
+++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftFlying.java
@@ -1,10 +1,11 @@
package org.bukkit.craftbukkit.entity;
+import com.destroystokyo.paper.entity.CraftSentientNPC;
import net.minecraft.server.EntityFlying;
import org.bukkit.craftbukkit.CraftServer;
import org.bukkit.entity.Flying;
-public class CraftFlying extends CraftLivingEntity implements Flying {
+public class CraftFlying extends CraftLivingEntity implements Flying, CraftSentientNPC { // Paper
public CraftFlying(CraftServer server, EntityFlying entity) {
super(server, entity);
diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftSlime.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftSlime.java
Update B/CB/S
2018-07-23 08:39:55 +00:00
index 6bf30c834c..3768b9573a 100644
Add Critical missing Bukkit API - setTarget/getTarget moved down to SentientNPC This fixes a CRITICAL missing part of the Bukkit API due to mistakes on upstream refusing to implement the Sentient NPC baseclass of all NPC's. Until now, the Bukkit API has not provided a way for accessing and setting a non creature entities target. Although Flying, Slime, Ambient, and Water mobs all supported targets internally, you were unable to get/set it. Now with the SentientNPC API and these API's moved down, every sentient NPC has access to target data.
2018-07-02 02:07:35 +00:00
--- a/src/main/java/org/bukkit/craftbukkit/entity/CraftSlime.java
+++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftSlime.java
@@ -1,5 +1,6 @@
package org.bukkit.craftbukkit.entity;
+import com.destroystokyo.paper.entity.CraftSentientNPC;
import net.minecraft.server.EntitySlime;
import org.bukkit.craftbukkit.CraftServer;
@@ -7,7 +8,7 @@ import org.bukkit.entity.EntityType;
import org.bukkit.entity.LivingEntity;
import org.bukkit.entity.Slime;
-public class CraftSlime extends CraftLivingEntity implements Slime {
+public class CraftSlime extends CraftLivingEntity implements Slime, CraftSentientNPC { // Paper
public CraftSlime(CraftServer server, EntitySlime entity) {
super(server, entity);
--
2.18.0