EssentialsX

EssentialsX

2M Downloads

buy / sell signs invalid charge problem with money over 1000

Twi5TeD opened this issue · 22 comments

commented

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

commented

Which economy plugin are you using and which version of vault?

commented

using essentials and vault version 1.5.6-b49

commented

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.

commented

this happens when i do this on a sign

[buy]
1
1
$3000

http://pastebin.com/nx0cLtSA

commented

and this is when i charge only 1$ to test
[buy]
1
1
$1
http://pastebin.com/mvqwLwb1

commented

https://gyazo.com/590faa96b6b9290831fc1ddc966f9e35 as you can see from the top sign it automaticly addes a comma and my balance was 60k

commented

Can you use a paste service so the logs are legible please

commented

updated :)

commented

Hey, I just tested this on build 355. I am unable to reproduce this issue:
it works!

commented

@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?

commented

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

commented

it works fine when comma-separated-money: true is set to false
but then the money doesnt have comma

commented

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. :)

commented

ill make a video and show if you want?

commented

Yes please.

commented

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
commented

just rendering the video

commented
commented

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?

commented

its a fork from someone ill ask them thanks

commented

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.

commented

I've created a PR #665 which should get accepted sometime soon.