feat: add support for external placeholder

[Zarkness2]

add support for using placeholders from the placeholderapi in the exp formula system, as it could be useful to use external placeholders.

[Zarkness2]

because I am trying to make a level system, in which to level up you need x exp points + x island level, and it would be useful if you want to use, also the exp, vanilla too, or money or any other placeholder that returns a number, for the exp formula.

[CPJiNan]

Sorry, due to plugin design limitations, implementing this feature seems a bit difficult.

[Zarkness2]

@CPJiNan Would it be possible to implement this in version 5.0?

[CPJiNan]

@Zarkness2 This feature will be added in version 5.0.

[Zarkness2]

thanks

[Zarkness2]

Hello @CPJiNan, how do I use external placeholders, that is, from placeholderapi in the exp system?

[Zarkness2]

I am using placeholders such as + %coinsengine_balance_raw_crystal% -> returns the money you have in that economy, or %easyph_prueba4% -> returns the value I assign to it since it is from a custom placeholder plugin, %player_level% -> returns the vanilla exp level and N/A in the placeholders to see the xp I need for that level.

I'm not using the version that came out today, but the one that came out on September 30, so I don't know what the problem is.

[Zarkness2]

@CPJiNan ?

[CPJiNan]

Honestly, I'm not entirely sure how this issue came up either...

It seemed to work fine during my initial testing.

Also, due to some reasons, I probably won't be able to maintain this project for a while...

[Zarkness2]

Don't mess with me, it's the best levels plugin out there right now. I hope you can get back to it after that time. Good luck.

From what I've been testing and looking at the code, it seems to me that the problem lies in the placeholders that call the getLevelExp method, which doesn't have a member that can parse the placeholders of the exp functions.

@CPJiNan

[CPJiNan]

@Zarkness2

Indeed, the getLevelExp method in the PlaceholderAPIHook class was missing the member parameter.

This has been fixed in the latest commit.

Thanks for pointing that out.