search

Jikoo / OpenInv

Open anyone's inventory as a chest, real-time!
GNU General Public License v3.0
118 stars 37 forks source link

Broken with Magma Forge Server #140

Closed Relaxing9 closed 1 year ago

Relaxing9 commented 1 year ago

OpenInv Version: 4.3.0 Server Version: Magma version 1.18.2-43e59c84 (Implementing API version 1.18.2-R0.1-SNAPSHOT, Forge Version 40.2.1)

From Logs: https://paste.gg/p/anonymous/56ce379337a24e9ca72f0475d8f9c2b1 From Console: https://paste.gg/p/anonymous/0d773276a7b844a89acdd1408a9b5420

image

Anytime the commands openinv and openender are ran on a Forge server there is an error, and the intended result doesn't happen just an error.

Jikoo commented 1 year ago

OI requires Spigot internals. I highly recommend you run a Forge mod for things like this; OI does not have support for concepts like dynamic additional inventory slots e.g. Baubles or whatever the modern equivalent is. These combo servers have also been notoriously unreliable because mods have a tendency to accomplish their goals in various inconsistent ways that result in missed cases for Bukkit events and whatnot.

Re: what I am assuming is a response from a Magma dev: They can add a Commodore entry to remap methods when loading plugins. It does require them to manually maintain the bytecode rewrite, but that's about it. The CB plugin loader that I assume they're piggybacking should handle the rest.