Skip to content
This repository has been archived by the owner on Aug 3, 2020. It is now read-only.

Placeholders not working on /info #2

Closed
SnowBlitzz opened this issue Jan 20, 2018 · 4 comments
Closed

Placeholders not working on /info #2

SnowBlitzz opened this issue Jan 20, 2018 · 4 comments

Comments

@SnowBlitzz
Copy link

I can’t seem to get any placeholders to work on the /info text files. They just become blank. The placeholders work on every other plugins so it’s not the placeholder’s issue. @dualspiral

And yes I’m using the correct format {%placeholder%}

Tested on:
Sponge version 2800 and 2833
Nucleus latest 1.2.2
Gluon 1.0.3
PlaceholderAPI 4.4

@dualspiral
Copy link
Member

gluon-1.0.4.zip

Try this version. I suspect it wasn't registering properly.

@SnowBlitzz
Copy link
Author

Ok this one works

@SnowBlitzz
Copy link
Author

Oops sorry for raising a false alarm.

It apparently only works on some placeholders that are not from the PlaceholderAPI itself. The placeholders from the PlaceholderAPI itself always give a blank in the /info files, even though /papi parse gives the correct value.

@dualspiral
Copy link
Member

Yep, I've just worked through it, it's actually a problem with Placeholder API itself, having just worked through it. Placeholder API Commands seem to bypass the normal parse method, hence they work. For Nucleus, which uses the parse method, this line sends an empty optional to the Player expansion which rejects it - it thinks it's only a relational parameter when it's both, so it fails at https://github.com/rojo8399/PlaceholderAPI/blob/master/src/main/java/me/rojo8399/placeholderapi/impl/placeholder/Defaults.java#L817.

This is not a Gluon problem, this is a Placeholder API problem.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants