search

Phr33d0m / NW-Profession-Bot

Automatically selects professions for empty slots
12 stars 31 forks source link

On Russian server does not consolidate AD #213

Closed metacorez closed 8 years ago

metacorez commented 8 years ago

Good day! On the Russian server did not consolidate the AD on the character of the banker. Perhaps the problem is that the names of the characters using Cyrillic letters. Orders without problems are set to 5 positions. Then no is a removal order and their consolidation on a character-the banker, only manually.

noonereally commented 8 years ago

I'm not sure. You tried to copy the name from one of the tabs ? Like "Refine Counters" it has all the character names, maybe copy pasting will solve it. I'll look into it if it's possible to make it a drop down the next time I'll change anything but it's probably loaded before the char list. Is there any indication in the log ? (ctrl+shift+j and debug option should be enabled in the script)

metacorez commented 8 years ago

Good Day! The names of the characters are short, and tried to copy it and enter it manually - no result. Errors in log no. In the presence of 5 buy orders AD. When switching to character banker in the log appears "Bank char not set or bank char, skipping posting." If it goes to the character who has, in the presence of AD - "Zen Max Listings Reached (5). Skipping ZAX Posting..". As if no function is triggered cancelZaxOffer, "Canceling ZAX orders" present and and then nothing happens.

metacorez commented 8 years ago

if (parseInt(item.price) == ZenRate && item.ordertype == "Buy") { -- not Work if (item.ordertype == "Buy") { -- Work

noonereally commented 8 years ago

You used "Override account settings" for your banking char ? If you did, try the last commit. It should be fixed.

Thanks for pointing where the bug is, it really helps.

metacorez commented 8 years ago

If you disable it "Override account settings" , everything works! Thank you very much for your help!

noonereally commented 8 years ago

Update to the latest version, it will work with the override. I fixed it.

https://raw.githubusercontent.com/Phr33d0m/NW-Profession-Bot/master/main.js