buy / sell signs invalid charge problem with money over 1000
Twi5TeD opened this issue · 22 comments
not sure if this is a bug or not but when i have a buy shop with a price of 1000 + i get the error invalid charge if i do below 1000 it works fine version 2.0.1 - b82
Are you sure you have more than $1k when you're trying to make that picture?
And can you reproduce this with ess debug mode enabled please.
and this is when i charge only 1$ to test
[buy]
1
1
$1
http://pastebin.com/mvqwLwb1
https://gyazo.com/590faa96b6b9290831fc1ddc966f9e35 as you can see from the top sign it automaticly addes a comma and my balance was 60k
@Twi5TeD Could you try using that version of essentials by itself without any other plugin on the server and see if the issue still occurs?
comma-separated-money: true add this to your config and try create the sign
ive tried with just that plugin and vault still the same
it works fine when comma-separated-money: true is set to false
but then the money doesnt have comma
That is correct. I'll dig deeper and see if I can fix this. Thanks for the help. We'd appreciate it if you could share your findings as soon as possible in order to squash the bugs and make other people have a better experience. :)
Upon further review. This was actually caused by something else entirely. The only case where you get the error message "Invalid Charge" is when you don't insert a currency symbol before the sign (e.g $, £, €, etc.) before the price.
So this works:
[Buy]
1
1
$1000
However, this doesn't work because it lacks the money symbol:
[Buy]
1
1
$3000
https://youtu.be/OiaNgGFfMJw there you go
This is weird. I can't recall exactly where I saw comma-separated-money
but the project has never had that property in the configuration based on my search queries... Are you using our EssentialsX or a fork?
Ahh I remember. I believe this is it Terrobility@daec100
I'll go add some configuration towards my pretty currency commit as I assume that is the reason you're using that fork.
I've created a PR #665 which should get accepted sometime soon.