Compatibility Focus

build.gradle

@@ -35,7 +35,7 @@ minecraft {
     copyIdeResources = true
     // generateRunFolders = true
 
-    // accessTransformer = file('src/main/resources/META-INF/accesstransformer.cfg')
+    accessTransformer = file('src/main/resources/META-INF/accesstransformer.cfg')
 
     runs {
         client {

gradle.properties

@@ -7,10 +7,10 @@ forgeVersion=43.3.5
 # Parchment Version
 parchmentVersion=2022.11.27
 # Mod Information see https://mcforge.readthedocs.io/en/1.18.x/gettingstarted/versioning/#examples
-modVersion=2.1.4.0
+modVersion=2.1.4.4
 modId=manascore
 # Mixin Extras
-mixinExtrasVersion=0.3.2
+mixinExtrasVersion=0.2.0
 # Photon: https://modrinth.com/mod/photon-editor/versions?l=forge&g=1.19.2
 photonVersion=1.0.7.a
 # LDLib: https://modrinth.com/mod/ldlib/versions?l=forge&g=1.19.2

src/main/java/com/github/manasmods/manascore/ManasCore.java

@@ -5,14 +5,11 @@
 package com.github.manasmods.manascore;
 
 import com.github.manasmods.manascore.attribute.ManasCoreAttributes;
-import com.github.manasmods.manascore.config.ManasCoreConfig;
 import com.github.manasmods.manascore.network.ManasCoreNetwork;
 import com.github.manasmods.manascore.skill.SkillRegistry;
 import lombok.Getter;
 import net.minecraftforge.eventbus.api.IEventBus;
-import net.minecraftforge.fml.ModLoadingContext;
 import net.minecraftforge.fml.common.Mod;
-import net.minecraftforge.fml.config.ModConfig.Type;
 import net.minecraftforge.fml.event.lifecycle.FMLCommonSetupEvent;
 import net.minecraftforge.fml.javafmlmod.FMLJavaModLoadingContext;
 import org.apache.logging.log4j.LogManager;
@@ -27,7 +24,6 @@ public final class ManasCore {
     private static final Logger logger = LogManager.getLogger();
 
     public ManasCore() {
-        ModLoadingContext.get().registerConfig(Type.COMMON, ManasCoreConfig.SPEC);
         IEventBus modEventBus = FMLJavaModLoadingContext.get().getModEventBus();
         ManasCoreAttributes.register(modEventBus);
         SkillRegistry.register(modEventBus);

src/main/java/com/github/manasmods/manascore/api/skills/TickingSkill.java

@@ -6,28 +6,31 @@
 
 import java.util.Optional;
 
+@Getter
 public class TickingSkill {
     private int duration = 0;
-    @Getter
     private final ManasSkill skill;
     public TickingSkill(ManasSkill skill) {
         this.skill = skill;
     }
 
     public boolean tick(SkillStorage storage, LivingEntity entity) {
-        Optional<ManasSkillInstance> optional = storage.getSkill(skill);
+        Optional<ManasSkillInstance> optional = storage.getSkill(this.getSkill());
         if (optional.isEmpty()) return false;
 
         ManasSkillInstance instance = optional.get();
         if (reachedMaxDuration(instance, entity)) return false;
 
-        if (!instance.canInteractSkill(entity)) return false;
+        if (!instance.canInteractSkill(entity)) {
+            instance.onRelease(entity, this.getDuration());
+            return false;
+        }
         return instance.onHeld(entity, this.duration++);
     }
 
     public boolean reachedMaxDuration(ManasSkillInstance instance, LivingEntity entity) {
         int maxDuration = instance.getMaxHeldTime(entity);
         if (maxDuration == -1) return false;
-        return duration >= maxDuration;
+        return this.getDuration() >= maxDuration;
     }
 }

src/main/java/com/github/manasmods/manascore/attribute/ManasCoreAttributeHandler.java

@@ -87,14 +87,15 @@ public static void applyEntityCrit(final LivingHurtEvent e) {
 
     @SubscribeEvent(priority = EventPriority.HIGH)
     public static void modifyCrit(final CriticalHitEvent e) {
+        float vanillaMultiplier = e.getDamageModifier() / e.getOldDamageModifier();
+        float critMultiplier = (float) e.getEntity().getAttributeValue(ManasCoreAttributes.CRIT_MULTIPLIER.get());
         if (e.isVanillaCritical()) {
-            e.setDamageModifier(e.getDamageModifier() * (float) e.getEntity().getAttributeValue(ManasCoreAttributes.CRIT_MULTIPLIER.get()));
+            e.setDamageModifier(vanillaMultiplier * critMultiplier);
             return;
         }
 
         double critChance = e.getEntity().getAttributeValue(ManasCoreAttributes.CRIT_CHANCE.get()) / 100; // convert to %
-        float critMultiplier = (float) e.getEntity().getAttributeValue(ManasCoreAttributes.CRIT_MULTIPLIER.get());
-        CriticalChanceEvent event = new CriticalChanceEvent(e.getEntity(), e.getTarget(), e.getDamageModifier() * critMultiplier, critChance);
+        CriticalChanceEvent event = new CriticalChanceEvent(e.getEntity(), e.getTarget(), vanillaMultiplier * critMultiplier, critChance);
         if (MinecraftForge.EVENT_BUS.post(event)) return;
 
         RandomSource random = e.getEntity().getRandom();

src/main/java/com/github/manasmods/manascore/capability/skill/TestSkill.java

@@ -13,7 +13,7 @@
 import net.minecraft.world.entity.monster.Creeper;
 import net.minecraft.world.entity.monster.Spider;
 import net.minecraft.world.entity.player.Player;
-import net.minecraft.world.entity.projectile.ThrownTrident;
+import net.minecraft.world.entity.projectile.AbstractArrow;
 import net.minecraft.world.item.Items;
 import net.minecraft.world.phys.BlockHitResult;
 import net.minecraftforge.common.MinecraftForge;
@@ -104,7 +104,7 @@ public void onTakenDamage(ManasSkillInstance instance, LivingDamageEvent event)
     }
 
     public void onProjectileHit(ManasSkillInstance instance, LivingEntity living, ProjectileImpactEvent event) {
-        if (event.getProjectile() instanceof ThrownTrident) {
+        if (event.getProjectile() instanceof AbstractArrow) {
             log.debug("Dodged");
             event.setCanceled(true);
         }

src/main/java/com/github/manasmods/manascore/capability/skill/event/ServerEventListenerHandler.java

@@ -8,9 +8,11 @@
 import com.github.manasmods.manascore.api.skills.event.SkillDamageEvent;
 import com.github.manasmods.manascore.api.skills.event.UnlockSkillEvent;
 import com.google.common.collect.Multimap;
+import it.unimi.dsi.fastutil.ints.IntOpenHashSet;
 import net.minecraft.world.entity.Entity;
 import net.minecraft.world.entity.LivingEntity;
 import net.minecraft.world.entity.player.Player;
+import net.minecraft.world.entity.projectile.AbstractArrow;
 import net.minecraft.world.phys.EntityHitResult;
 import net.minecraftforge.common.MinecraftForge;
 import net.minecraftforge.event.entity.ProjectileImpactEvent;
@@ -149,6 +151,10 @@ public static void onProjectileHit(final ProjectileImpactEvent e) {
             if (!skillInstance.canInteractSkill(living)) continue;
 
             skillInstance.onProjectileHit(living, e);
+            if (e.isCanceled() && e.getProjectile()  instanceof AbstractArrow arrow) {
+                if (arrow.piercingIgnoreEntityIds == null) arrow.piercingIgnoreEntityIds = new IntOpenHashSet(10);
+                arrow.piercingIgnoreEntityIds.add(living.getId());
+            }
         }
         skillStorage.syncChanges();
     }

src/main/resources/META-INF/accesstransformer.cfg

@@ -0,0 +1 @@
+public net.minecraft.world.entity.projectile.AbstractArrow f_36701_ # piercingIgnoreEntityIds

src/main/resources/META-INF/mods.toml

@@ -3,7 +3,7 @@ loaderVersion = "${forgeLoaderVersion}"
 license = "GPL-3"
 issueTrackerURL = "https://github.com/ManasMods/ManasCore/issues" #optional
 [[mods]] #mandatory
-modId = "${modId}" #mandatory
+modId = "manascore" #mandatory
 version = "${modVersion}"
 displayName = "ManasCore" #mandatory
 updateJSONURL="https://api.modrinth.com/updates/wRRSWpd1/forge_updates.json" #optional
@@ -15,24 +15,18 @@ description = '''
 Utility and Core Library for Manas Mods
 '''
 #@formatter:off
-[[dependencies.${modId}]] #optional
+[[dependencies.manascore]] #optional
    modId="forge" #mandatory
    mandatory=true #mandatory
    versionRange="${forgeLoaderVersion}" #mandatory
    ordering="NONE"
    side="BOTH"
 # Here's another dependency
-[[dependencies.${modId}]]
+[[dependencies.manascore]]
    modId="minecraft"
    mandatory=true
    # This version range declares a minimum of the current minecraft version up to but not including the next major version
    versionRange="${minecraftVersionCheck}"
    ordering="NONE"
    side="BOTH"
-[[dependencies.${modId}]]
-   modId="mixinextras"
-   mandatory=true
-   versionRange="[${mixinExtrasVersion},)"
-   ordering="NONE"
-   side="BOTH"
 ##@formatter:on

src/main/resources/manascore.mixins.json

@@ -6,15 +6,13 @@
   "refmap": "manascore.refmap.json",
   "mixins": [
     "BlockLootAccessor",
-    "MixinAnvilMenu",
     "MixinLivingEntity",
     "MixinServerLevel",
     "RegistryBuilderAccessor",
     "ShapedRecipeBuilderAccessor"
   ],
   "client": [
     "AbstractContainerScreenAccessor",
-    "MixinAnvilScreen",
     "MixinMinecraft"
   ],
   "injectors": {