EssentialsX

EssentialsX

2M Downloads

Blank currency symbol

Henriiig opened this issue ยท 7 comments

commented

Feature request

Feature description:

The ability to have a none and blank currency for balance. This was possible before the newest dev update but now it replaces the " " with a $ sign. Not quite sure if this is actually a bug.

How the feature is useful:

The future is usefull for people who wants to use their own type of currency through skript or other plugins using balance.

commented

Caused by c6de771.

commented

Can you explain where Essentials is showing the $ symbol and how setting a blank currency symbol affects Skript? I don't think it would be a big deal to allow this, just curious about what problem this has on your end, so if you could share some examples (and maybe some screenshots) that would be nice

Note to self:
change the length check to > 1 rather than == 1 c6de771#diff-b8e0c618de42d4f3393256c3c8e2cc43b20b16b0633c772d3284d2e4c85ab40cR760

commented

Its simply because the blank spot has been replaced with a dollar sign. This was not the case in earlier versions of Essentials. I linked you screenshots to show you exactly what the problem is.

image
image

commented

Can you explain where Essentials is showing the $ symbol and how setting a blank currency symbol affects Skript? I don't think it would be a big deal to allow this, just curious about what problem this has on your end, so if you could share some examples (and maybe some screenshots) that would be nice

Note to self:
change the length check to > 1 rather than == 1 c6de771#diff-b8e0c618de42d4f3393256c3c8e2cc43b20b16b0633c772d3284d2e4c85ab40cR760

Hello, anything new about this problem? :)

commented

Hello, anything new about this? :)

I have an open PR, #3946, which fixes this. It's currently waiting on approval but feel free to test the jars found here

commented

Hello, anything new about this? :)

I have an open PR, #3946, which fixes this. It's currently waiting on approval but feel free to test the jars found here

Thanks, im glad to hear that :)

commented

@Henriiig #3946 has been merged which should allow this once again. You can download the latest version from the EssentialsX website.