[CHANGE] make it so return to village ai has higher priority than attacking mob ai so guards are able to return to the village more effectively

Author: seymourimadeit

src/main/java/tallestegg/guardvillagers/configuration/GuardConfig.java

@@ -192,7 +192,7 @@ public CommonConfig(ForgeConfigSpec.Builder builder) {
             builder.push("guard stuff");
             guardSpawnInVillage = builder.defineInRange("How many guards should spawn in a village?", 6, 0, 100000000);
             guardSinkToFightUnderWater = builder.define("Allow guards to sink temporarily to fight mobs that are under water?", true);
-            depthGuardHuntUnderwater = builder.comment("If a guard is fighting a mob underwater and the vertical distance between that mob and the guard is larger than this, the guard will instead float up to not take the risk of drowning").defineInRange("Depth value for guards fighting underwater mobs",  8, 0, 100000000);
+            depthGuardHuntUnderwater = builder.comment("If a guard is fighting a mob underwater and the vertical distance between that mob and the guard is larger than this, the guard will instead float up to not take the risk of drowning").defineInRange("Depth value for guards fighting underwater mobs",  5, 0, 100000000);
             mobsGuardsProtectTargeted = builder.defineListAllowEmpty("Mobs that guards actively protect when they get targeted", Lists.newArrayList("minecraft:villager", "guardvillagers:guard", "minecraft:iron_golem"), obj -> true);
             mobsGuardsProtectHurt = builder.comment("Mobs in this list also won't get hurt by a guard's arrow if the config option to disable guard arrows hurting villagers is enabled.").defineListAllowEmpty("Mobs that guards actively protect when they get hurt", Lists.newArrayList("minecraft:villager", "guardvillagers:guard", "minecraft:iron_golem"),  obj -> true);
             structuresThatSpawnGuards = builder.comment("Guards are placed in the middle, thus more advanced placement should be done via datapacks").define("Structure pieces that spawn guards", Lists.newArrayList("minecraft:village/common/iron_golem"), obj -> true);

src/main/java/tallestegg/guardvillagers/entities/Guard.java

@@ -646,12 +646,12 @@ public boolean canUse() {
         this.targetSelector.addGoal(3, new HeroHurtTargetGoal(this));
         this.targetSelector.addGoal(5, new Guard.DefendVillageGuardGoal(this));
         if (GuardConfig.AttackAllMobs) {
-            this.targetSelector.addGoal(3, new NearestAttackableTargetGoal<>(this, Mob.class, 5, true, true, (mob) -> mob instanceof Enemy && !GuardConfig.MobBlackList.contains(mob.getEncodeId())));
+            this.targetSelector.addGoal(5, new NearestAttackableTargetGoal<>(this, Mob.class, 5, true, true, (mob) -> mob instanceof Enemy && !GuardConfig.COMMON.MobBlackList.get().contains(mob.getEncodeId())));
         } else {
-            this.targetSelector.addGoal(2, new NearestAttackableTargetGoal<>(this, Ravager.class, true)); // To make witches and ravagers have a priority than other mobs this has to be done
-            this.targetSelector.addGoal(2, new NearestAttackableTargetGoal<>(this, Witch.class, true));
-            this.targetSelector.addGoal(3, new NearestAttackableTargetGoal<>(this, Raider.class, true));
-            this.targetSelector.addGoal(4, new NearestAttackableTargetGoal<>(this, Zombie.class, true, (mob) -> !(mob instanceof NeutralMob)));
+            this.targetSelector.addGoal(3, new NearestAttackableTargetGoal<>(this, Ravager.class, true)); // To make witches and ravagers have a priority than other mobs this has to be done
+            this.targetSelector.addGoal(4, new NearestAttackableTargetGoal<>(this, Witch.class, true));
+            this.targetSelector.addGoal(5, new NearestAttackableTargetGoal<>(this, Raider.class, true));
+            this.targetSelector.addGoal(5, new NearestAttackableTargetGoal<>(this, Zombie.class, true, (mob) -> !(mob instanceof ZombifiedPiglin)));
         }
         this.targetSelector.addGoal(3, new NearestAttackableTargetGoal<>(this, Mob.class, 5, true, true, (mob) -> GuardConfig.COMMON.MobWhiteList.get().contains(mob.getEncodeId())));
         this.targetSelector.addGoal(3, new NearestAttackableTargetGoal<>(this, Player.class, 10, true, false, this::isAngryAt));