search

EbonJaeger / perworldinventory-kt

Multi-world inventory plugin for Spigot written in Kotlin.
MIT License
46 stars 32 forks source link

Conflicts with keeping the EXP value on death with WorldguardExtraFlags, CMI and EssentialsX #228

Open TheoRetisch1 opened 3 years ago

TheoRetisch1 commented 3 years ago

Describe the bug Perworldiventory is generally not compatible with other plugins that want to set the EXP values. It does doesn't work with the Keep-EXP feature from Essentials (#227) and from CMI. The XP will simply be reseted on death if per-world-inventory is active. It is also incompatible with the keep-exp flag of WorldguardExtraFlag. If you set the keep-exp flag to true when PerWorldInventory is loaded, the EXP are simply reset on death. Interestingly, this feature works, if the inventory with the keep-inventory true flag is kept at the same time. So it would be fantastic if you would allow other plugins to set the appropriate EXP value.

To Reproduce Steps to reproduce the behavior: WorldguardExtraFlags

  1. Install Worldguard + WorldguardExtraFlags
  2. Create a region.
  3. Set the the flag keep-exp to true for the region
  4. Die inside the region -> you will lose your EXP

  5. Try it without PWI -> you will keep your EXP

    CMI

  6. Install CMI
  7. Set the permission cmi.keepexp to true
  8. Die -> you will lose your EXP

  9. Try it without PWI -> you will keep your EXP

    EssentialsX: There is already a Bug report on your issue tracker: #227

Expected behavior CMI, WorldguardExtraFlags and CMI should be able to prevent the loss of EXP on death by a permission/flag.

Plugin version PerWorldInventory 2.3.2 WorldGuardExtraFlags 4.1.4 (https://github.com/aromaa/WorldGuardExtraFlagsPlugin) EssentialsX 2.18.2 (https://github.com/EssentialsX/Essentials) CMI 8.7.11.2 (https://www.spigotmc.org/resources/cmi-270-commands-insane-kits-portals-essentials-economy-mysql-sqlite-much-more.3742/) Worldguard 7.0.5-SNAPSHOT build 2084 (https://builds.enginehub.org/job/worldguard)

Server version Paper 1.16.4 - 319

Additional context and logs PWI config: https://pastebin.com/nk97p9nY PWI world settings: https://pastebin.com/iDfFPiQH There are no errors in the log.

TheoRetisch1 commented 3 years ago

This is still an issue using Paper 1.16.5-440!

TheoRetisch1 commented 3 years ago

This is still an issue - Paper 1.16.5 -468 :/

TheoRetisch1 commented 3 years ago

This is still an issue - Paper 1.16.5 - 762 :/