Skip to content
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

Placeholders with Essentials #4936

Closed
Smurfboi007 opened this issue Jun 11, 2022 · 10 comments
Closed

Placeholders with Essentials #4936

Smurfboi007 opened this issue Jun 11, 2022 · 10 comments
Labels
status: invalid Not a valid issue.

Comments

@Smurfboi007
Copy link

Type of bug

Error in console

/ess dump all output

https://essentialsx.net/dump.html?id=564254946a384985b23185612c5c591a

Error log (if applicable)

No response

Bug description

Hi! When trying to use placeholders in the essentials config I get this error, I got the placeholder installed and everything.

[14:50:59 ERROR]: Could not pass event AsyncPlayerChatEvent to EssentialsChat v2.19.5-dev+7-1c5a7e9
java.util.UnknownFormatConversionException: Conversion = '§'
at org.bukkit.event.player.AsyncPlayerChatEvent.setFormat(AsyncPlayerChatEvent.java:105) ~[paper-api-1.18.2-R0.1-SNAPSHOT.jar:?]
at com.earth2me.essentials.chat.EssentialsChatPlayerListenerLowest.onPlayerChat(EssentialsChatPlayerListenerLowest.java:69) ~[EssentialsXChat-2.19.5-dev+7-1c5a7e9.jar:?]
at com.destroystokyo.paper.event.executor.asm.generated.GeneratedEventExecutor967.execute(Unknown Source) ~[?:?]
at org.bukkit.plugin.EventExecutor.lambda$create$1(EventExecutor.java:75) ~[paper-api-1.18.2-R0.1-SNAPSHOT.jar:?]
at co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:76) ~[paper-api-1.18.2-R0.1-SNAPSHOT.jar:git-Paper-361]
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:70) ~[paper-api-1.18.2-R0.1-SNAPSHOT.jar:?]
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:669) ~[paper-api-1.18.2-R0.1-SNAPSHOT.jar:?]
at io.papermc.paper.adventure.ChatProcessor.post(ChatProcessor.java:195) ~[paper-1.18.2.jar:git-Paper-361]
at io.papermc.paper.adventure.ChatProcessor.process(ChatProcessor.java:67) ~[paper-1.18.2.jar:git-Paper-361]
at net.minecraft.server.network.ServerGamePacketListenerImpl.chat(ServerGamePacketListenerImpl.java:2223) ~[?:?]
at net.minecraft.server.network.ServerGamePacketListenerImpl.handleChat(ServerGamePacketListenerImpl.java:2134) ~[?:?]
at net.minecraft.server.network.ServerGamePacketListenerImpl.handleChat(ServerGamePacketListenerImpl.java:2100) ~[?:?]
at net.minecraft.network.protocol.game.ServerboundChatPacket$1.run(ServerboundChatPacket.java:40) ~[?:?]
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) ~[?:?]
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) ~[?:?]
at java.lang.Thread.run(Thread.java:833) ~[?:?]

Steps to reproduce

This error comes up whenever typing in chat and it goes back to the default chat look.

Expected behaviour

The placeholder should be working.

Actual behaviour

It doesn't work.

@Smurfboi007 Smurfboi007 added the bug: unconfirmed Potential bugs that need replicating to verify. label Jun 11, 2022
@osfanbuff63
Copy link

osfanbuff63 commented Jun 11, 2022

Try updating EssentialsChat to the same version as Essentials? It looks like there's a version mismatch. As well, make sure to run /papi ecloud download Essentials and then /papi reload. If neither of those work, since we don't develop the PlaceholderAPI expansion, report it to them at the expansion repository.

@Chew
Copy link
Member

Chew commented Jun 11, 2022

I didn't see the essentials placeholder load in the logs which confused me, if you are referring to PAPI placeholders, see above, otherwise, can you please clarify?

@osfanbuff63
Copy link

I didn't see the essentials placeholder load in the logs which confused me, if you are referring to PAPI placeholders, see above, otherwise, can you please clarify?

They have PAPI, but I didn't see the placeholder - hence why I suggested installing it.

@Smurfboi007
Copy link
Author

Right, so those fixes didn't work, I have all the dependencies installed. Will check with placeholderAPI. But thanks for helping out! I will update if I find the issue or/and solve the issue.

@osfanbuff63
Copy link

I will say - can you reproduce this without PlaceholderAPI? If you can - then don't ask them, since it could be an Essentials issue.

@Smurfboi007
Copy link
Author

Smurfboi007 commented Jun 11, 2022

Wdym? Like just removing PlaceholderAPI from the plugins folder?

@osfanbuff63
Copy link

Wdym? Like just removing PlaceholderAPI from the plugins folder?

Yep!

@Smurfboi007
Copy link
Author

It works when removing it, but it obviously just says the %PLACEHOLDER% in text and doesn't show it. That means it's a problem with placeholderAPI then, right?

@osfanbuff63
Copy link

It works when removing it, but it obviously just says the %PLACEHOLDER% in text and doesn't show it. That means it's a problem with placeholderAPI then, right?

Probably, I'd bring it up to them then

@triagonal
Copy link
Member

EssentialsXChat does not support PlaceholderAPI placeholders. Not only is there no built-in support, but the %placeholder% format entirely breaks the way Essentials parses the chat format, leading to the error you're seeing.

If you want to use PAPI placeholders in your chat format, you need to install ChatInjector as well - make sure to carefully follow the instructions on the page to transform your placeholders to the proper format (%placeholder% -> {placeholder})

@triagonal triagonal added status: invalid Not a valid issue. and removed bug: unconfirmed Potential bugs that need replicating to verify. labels Jun 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: invalid Not a valid issue.
Projects
None yet
Development

No branches or pull requests

4 participants