Market Problem

[OliverTrust]

Describe the bug

After this update c2a4d30 i have a problem with relist overpriced market listings. All my items on market marked as overpriced. My currency is RUB. Previous version work without any problems

version with problem

version without problem

Browser name and version

chrome 127.0.6533.100

[xPaw]

wtf valve, their russian "currency symbol" has a latin "p" instead of cyrillic one.

So it's failing to strip it from the amount, causing the bug I think.

GetPriceValueAsInt('0,47 pуб.') // latin
47

GetPriceValueAsInt('0,47 руб.') // cyrillic
4700

GetCurrencySymbol( 'RUB' )
'pуб.'

EDIT: Seems like the steam store itself also has this, so I'm not sure why it wouldn't parse on market, can you copy "руб." from the market page and paste it into https://apps.timwhitlock.info/unicode/inspect to check.

[OliverTrust]

Yes, i have the same result

[xPaw]

Then I'm not sure why it would cause issues, 0,47rub should parse to 47, the bug is some multiplier to 100 somewhere.

[SeRi0uS007]

It seems that using unsafeWindow.GetPriceValueAsInt with raw jQuery text() can lead to wrong script behavior.

I have exactly the same problem (my currency is ₴).
But I have stacked items for sale.

Because of the dot at the end, the GetPriceValueAsInt function doesn't work as it should.

In addition, priceSeller receives the price in parentheses (0.03₴) as an argument, which is why GetPriceValueAsInt simply returns 0...

[xPaw]

Because of the dot at the end, the GetPriceValueAsInt function doesn't work as it should.

That "/piece." is definitely problematic, didn't know this is even a thing. I don't know if just removing last dot will work for all languages here.

And for seller price, just stripping () should work. Wanna make a PR?

[SeRi0uS007]

It's even funny :)

@xPaw

Wanna make a PR?

I also want to fix the wrong math for the items in the stack. I want to return getPriceFromMarketListing (or create a new function with Regex) to clean the price strings.

[xPaw]

There shouldn't be strings in market with that many dots, so that's fine.

[xPaw]

their russian "currency symbol" has a latin "p" instead of cyrillic one.

well that was fixed after today's maintenance