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.5
 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/ManasSkill.java

@@ -226,8 +226,9 @@ public void addHeldAttributeModifiers(ManasSkillInstance instance, LivingEntity
      * Removes the attribute modifiers of this skill from the {@link LivingEntity} holding the skill activation button.
      *
      * @param entity   Affected {@link LivingEntity} owning this Skill.
+     * @param instance Affected {@link ManasSkillInstance}
      */
-    public void removeHeldAttributeModifiers(LivingEntity entity) {
+    public void removeHeldAttributeModifiers(ManasSkillInstance instance, LivingEntity entity) {
         for(Map.Entry<Attribute, AttributeModifier> entry : this.onHeldAttributeModifiers.entrySet()) {
             AttributeInstance attributeinstance = entity.getAttributes().getInstance(entry.getKey());
             if (attributeinstance != null) attributeinstance.removeModifier(entry.getValue());

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

@@ -393,6 +393,15 @@ public void addHeldAttributeModifiers(LivingEntity entity) {
         this.getSkill().addHeldAttributeModifiers(this, entity);
     }
 
+    /**
+     * Removes the attribute modifiers of this instance from the {@link LivingEntity} holding the skill activation button.
+     *
+     * @param entity   Affected {@link LivingEntity} owning this Skill.
+     */
+    public void removeHeldAttributeModifiers(LivingEntity entity) {
+        this.getSkill().removeHeldAttributeModifiers(this, entity);
+    }
+
     /**
      * Called when the {@link LivingEntity} owning this Skill toggles this {@link ManasSkill} type of this instance on.
      *

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

@@ -3,31 +3,38 @@
 import com.github.manasmods.manascore.api.skills.capability.SkillStorage;
 import lombok.Getter;
 import net.minecraft.world.entity.LivingEntity;
+import org.jetbrains.annotations.Nullable;
 
 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);
-        if (optional.isEmpty()) return false;
+    public @Nullable ManasSkillInstance getSkillInstance(SkillStorage storage, LivingEntity entity) {
+        Optional<ManasSkillInstance> optional = storage.getSkill(this.getSkill());
+        return optional.orElse(null);
+    }
 
-        ManasSkillInstance instance = optional.get();
+    public boolean tick(SkillStorage storage, LivingEntity entity) {
+        ManasSkillInstance instance = this.getSkillInstance(storage, entity);
+        if (instance == null) return false;
         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();
     }
@@ -216,8 +222,25 @@ public static void onLogOut(final PlayerEvent.PlayerLoggedOutEvent e) {
         Player player = e.getEntity();
         Multimap<UUID, TickingSkill> multimap = TickEventListenerHandler.tickingSkills;
         if (multimap.containsKey(player.getUUID())) {
-            for (TickingSkill tickingSkill : multimap.get(player.getUUID()))
-                tickingSkill.getSkill().removeHeldAttributeModifiers(player);
+            for (TickingSkill tickingSkill : multimap.get(player.getUUID())) {
+                ManasSkillInstance instance = tickingSkill.getSkillInstance(SkillAPI.getSkillsFrom(player), player);
+                if (instance == null) continue;
+                instance.removeHeldAttributeModifiers(player);
+            }
+            multimap.removeAll(player.getUUID());
+        }
+    }
+
+    @SubscribeEvent
+    public static void onChangingDimension(final PlayerEvent.PlayerChangedDimensionEvent e) {
+        Player player = e.getEntity();
+        Multimap<UUID, TickingSkill> multimap = TickEventListenerHandler.tickingSkills;
+        if (multimap.containsKey(player.getUUID())) {
+            for (TickingSkill tickingSkill : multimap.get(player.getUUID())) {
+                ManasSkillInstance instance = tickingSkill.getSkillInstance(SkillAPI.getSkillsFrom(player), player);
+                if (instance == null) continue;
+                instance.removeHeldAttributeModifiers(player);
+            }
             multimap.removeAll(player.getUUID());
         }
     }

src/main/java/com/github/manasmods/manascore/network/toserver/RequestSkillReleasePacket.java

@@ -52,12 +52,13 @@ public void handle(Supplier<NetworkEvent.Context> ctx) {
                     if (optional.isEmpty()) continue;
                     ManasSkillInstance skillInstance = optional.get();
 
-                    if (!skillInstance.canInteractSkill(player)) continue;
-                    if (skillInstance.onCoolDown() && !skillInstance.canIgnoreCoolDown(player)) continue;
-
-                    skillInstance.onRelease(player, this.heldTick);
-                    skillInstance.getSkill().removeHeldAttributeModifiers(player);
+                    if (skillInstance.canInteractSkill(player)) {
+                        if (!skillInstance.onCoolDown() || skillInstance.canIgnoreCoolDown(player)) {
+                            skillInstance.onRelease(player, this.heldTick);
+                        }
+                    }
 
+                    skillInstance.removeHeldAttributeModifiers(player);
                     Multimap<UUID, TickingSkill> multimap = TickEventListenerHandler.tickingSkills;
                     if (multimap.containsKey(player.getUUID())) multimap.get(player.getUUID()).removeIf(tickingSkill -> tickingSkill.getSkill() == skillInstance.getSkill());
                 }

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": {