Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix getCurrentMarketItemNameId error #186

Closed
disanyixing opened this issue Sep 30, 2023 · 3 comments
Closed

Fix getCurrentMarketItemNameId error #186

disanyixing opened this issue Sep 30, 2023 · 3 comments

Comments

@disanyixing
Copy link

disanyixing commented Sep 30, 2023

image

image

var matches = /Market_LoadOrderSpread\( (.+) \);/.exec(page);
var item_nameid = matches[1];

This will result in two values:
The first value comes from Market_LoadOrderSpread( 77489411 ); // The matches[1] is 77489411
The second value comes from Market_LoadOrderSpread( 77489411 ); }, 2 * 60 * 1000 ); // The matches[1] is 77489411 ); }, 2 * 60 * 1000
image
The second value will cause the value of the red box to become 77489411 ); }, 2 * 60 * 1000

image
After fixing the regular expression:
Both values come from Market_LoadOrderSpread( 77489411 ); // Both matches[1] are 77489411

@disanyixing disanyixing changed the title fix getCurrentMarketItemNameId error Fix getCurrentMarketItemNameId error Sep 30, 2023
@xPaw
Copy link
Contributor

xPaw commented Jul 14, 2024

This was fixed by 1fcd98a i think

@disanyixing
Copy link
Author

This was fixed by 1fcd98a i think

Yes. But this issue opened on Sep 30, 2023. Before that time this issue had not fixed.

@xPaw
Copy link
Contributor

xPaw commented Jul 14, 2024

Well you can close the issue then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants