Easy NPC

Easy NPC

2M Downloads

[Crash Report] Incompatibility with Arc Lib

tooopt opened this issue ยท 4 comments

commented

Minecraft and Mod version ๐Ÿž
1.20.1 Forge
5.4.1 EasyNPC version

Description of the crash ๐Ÿ’ฅ
If EasyNPC and arc lib are installed, when you spawn an npc, the game crashes

Steps to reproduce the crash โš™๏ธ

  1. Install easynpc
  2. Install ArcLib with dependencies (Arclib, Architecturyapi, UI Lib, SuperMartijn core lib and SuperMartijn config lib)
  3. Launch the game, create a new world and spawn an npc

Expected behavior before the crash โœ”๏ธ
Expected behavior would be to spawn an npc

Logs ๐Ÿ“

debug log - https://pastebin.com/yTZtG5v4
crash.log - https://pastebin.com/xtCvhDgC

Additional context ๐Ÿ“‹
Side note that the mod is incompatible only with arclib in this situation, If you delete arclib but leave the dependencies, everything works normally

commented

Thank you for the report.

The issue lies with ArcLib itself and its use of the mixin definition.
https://github.com/DAQEM/ArcLib/blob/f602714beb0b5b6a4c941fd4fb892e353fc0d3ef/common/src/main/java/com/daqem/arc/mixin/MixinLivingEntity.java#L24

In this mixin, they define a getLivingEntity function, which overrides the getLivingEntity method of NPC entities and other entities that implement this function.

Since the NPC entities are not part of their library, they should avoid making such changes in the mixin.
Instead, they should use their own namespace rather than a general term like getLivingEntity.
Alternatively, they could remove this function entirely to improve compatibility with other mods and implementations that they do not own.

As mentioned earlier, this will likely cause issues with other mods that use getLivingEntity for their entities.

Please report this issue to ArcLib, and feel free to use this information as a reference. Thank you.

commented

Thank you! I have reported this issue to ArcLib

commented

This issue is stale because it has been open 7 days with no activity. Remove stale label or comment or this will be closed in 3 days.

commented

This issue was closed because it has been stalled for 5 days with no activity.