Fixes #3 (animal theft)

Author: AeroStun

src/main/java/org/scbrm/fidelity/WhipItem.java

@@ -41,7 +41,7 @@ public ActionResult useOnEntity(ItemStack stack, PlayerEntity user, LivingEntity
                     iequine.setMaster(user);
                 } else if(iequine.getMaster() != user) {
                     user.sendSystemMessage(new LiteralText("You do not own this animal"), Util.NIL_UUID);
-                    return ActionResult.FAIL;
+                    return ActionResult.success(user.world.isClient);
                 }
 
                 final IHorseBaseEntity.State state = iequine.getState().next();

src/main/java/org/scbrm/fidelity/mixin/HorseBaseEntityMixin.java

@@ -1,7 +1,9 @@
 package org.scbrm.fidelity.mixin;
 
+import net.minecraft.entity.player.PlayerEntity;
 import net.minecraft.nbt.CompoundTag;
 import net.minecraft.server.ServerConfigHandler;
+import net.minecraft.text.LiteralText;
 import org.scbrm.fidelity.bridge.IHorseBaseEntity;
 import org.scbrm.fidelity.entity.ai.goal.ObeyMasterGoal;
 import net.minecraft.entity.EntityType;
@@ -22,6 +24,8 @@
 import java.util.Optional;
 import java.util.UUID;
 
+import static net.minecraft.util.Util.NIL_UUID;
+
 @Mixin(HorseBaseEntity.class)
 public abstract class HorseBaseEntityMixin extends AnimalEntity implements IHorseBaseEntity {
 
@@ -97,4 +101,13 @@ public void _readCustomDataFromTag(CompoundTag tag, CallbackInfo info) {
 		}
 	}
 
+	@Inject(at = @At("HEAD"), method = "putPlayerOnBack(Lnet/minecraft/entity/player/PlayerEntity;)V", cancellable = true)
+	void preventTheft(PlayerEntity player, CallbackInfo info){
+		if(!this.isMaster(player)) {
+			if(!player.world.isClient)
+				player.sendSystemMessage(new LiteralText("You do not own this animal"), NIL_UUID);
+			info.cancel();
+		}
+	}
+
 }

src/main/java/org/scbrm/fidelity/mixin/HorseEntityMixin.java

@@ -0,0 +1,17 @@
+package org.scbrm.fidelity.mixin;
+
+import net.minecraft.entity.EntityType;
+import net.minecraft.entity.passive.HorseBaseEntity;
+import net.minecraft.entity.passive.HorseEntity;
+import net.minecraft.world.World;
+import org.spongepowered.asm.mixin.Mixin;
+
+@Mixin(HorseEntity.class)
+public abstract class HorseEntityMixin extends HorseBaseEntity {
+
+    protected HorseEntityMixin(EntityType<? extends HorseBaseEntity> entityType, World world) {
+        super(entityType, world);
+    }
+
+
+}