-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] Skript Compatibility #20
Comments
No such field def error probably caused by not updated Sponge version. ItemTypes.AIR was added "recently". And for setOp see here : |
Not implementing setOp makes sense, I think that option is not necessary. Also I've made a small update of the issue. (Most of events doesn't fire any error or message) Edit: Because most of them are not registered or even implemented, I get it. |
Implementing events are a royal pain in the butt, so I'd be fine if you just took them from Craftbukkit and turned them into mixins (e.g https://github.com/Maxqia/PoreRT/blob/master/src/main/java/blue/lapis/pore/mixin/network/MixinNetHandlerPlayServer.java), also apparently SpongeForge didn't update the SpongeAPI for the ItemTypes.AIR update until 1.11.2 :P |
I'll look into that. |
Is there anything which I (as Skript maintainer) could do to improve Pore compatibility? |
Probably not from the Skript side of things :P |
I'll keep this issue up to date, and I'll add more features to the list in next coming days.
Tested with:
Skript - dev23
skAliases - 2.0.5
PoreRT - 1.11 build 112
SpongeForge - 1.11-2195-6.0.0-BETA-2004
Forge - 1.11-13.19.1.2199
How to reproduce:
Tested features:
Effects:
give a stone to the player
PoreHumanEntity.setOp()
Events:
MaterialConverter java.lang.NoSuchFieldError: AIR
Types:
TODO
I'll add more features to the list later, most of them are causing error with MaterialConverter like above.
The text was updated successfully, but these errors were encountered: