Jump to content
Sign in to follow this  
TRiLeZ

TRiBot Beta 7.43_0

Recommended Posts

Hey Trilez, thanks for fixing the Banking API so fast.

 

Banking.DepositAll() could just click the [Deposit All] button now, instead of looping through the inventory items array, right clicking each item, and clicking deposit all. (Not only more efficient, but if everyone in rs is using the Deposit All button, and the bots still deposit all the old one, might look suspicious).

 

I also recommend adding Banking.DepositAllEquipped() // clicks the [Deposit All Equipped Items] on the bank screen.

 

Also, 3 bank rows (24 bank slots) were added to the bank. Which means the scroll bar height has changed, meaning bank scrolling (in the Banking API) could be off. So if an item is not on the top screen of the bank, the withdraw function may fail, since the scrollbars have changed (ratios, height, etc). My suggestion is this:

 

Inside of withdraw(), if the item is not on the first bank page, it clicks [search] button, it types in the item name, then this makes the item show at the top of the bank, then withdraw can run as normal. This is an easy-fix, but also a much more accurate method for withdrawing items that are not on the first bank page (even with the old bank, it often failed, which is why script writers would always say that you must put items at the top of your bank)

 

Also if someone does Banking.DepositAllExcept(x) and let's say X is a "Knife" (fletching script), then it might be better for it to click the [Deposit All] button, then to left click to withdraw the Knife back into the inventory. Much quicker, and more human-like. (This behaviour on DepositAllExcept would only occur if 1 item is passed as the argument, and there is only 1 of this item in the inventory).

Share this post


Link to post
Share on other sites

Banking.DepositAll() could just click the [Deposit All] button now, instead of looping through the inventory items array, right clicking each item, and clicking deposit all. (Not only more efficient, but if everyone in rs is using the Deposit All button, and the bots still deposit all the old one, might look suspicious).

Banking.depositAll() was changed with this update to use the button. :)

 

I also recommend adding Banking.DepositAllEquipped() // clicks the [Deposit All Equipped Items] on the bank screen.

Banking.depositEquipment() was also added, just not in the java docs yet.

Also, 3 bank rows (24 bank slots) were added to the bank. Which means the scroll bar height has changed, meaning bank scrolling (in the Banking API) could be off. So if an item is not on the top screen of the bank, the withdraw function may fail, since the scrollbars have changed (ratios, height, etc). My suggestion is this:

 

Inside of withdraw(), if the item is not on the first bank page, it clicks [search] button, it types in the item name, then this makes the item show at the top of the bank, then withdraw can run as normal. This is an easy-fix, but also a much more accurate method for withdrawing items that are not on the first bank page (even with the old bank, it often failed, which is why script writers would always say that you must put items at the top of your bank)

Typing takes too long. I'll just change the scroll position calculation.

Also if someone does Banking.DepositAllExcept(x) and let's say X is a "Knife" (fletching script), then it might be better for it to click the [Deposit All] button, then to left click to withdraw the Knife back into the inventory. Much quicker, and more human-like. (This behaviour on DepositAllExcept would only occur if 1 item is passed as the argument, and there is only 1 of this item in the inventory).

I don't really want the API having too much logic for a method like this. It should be up to the script to have that logic. The API is just a framework that should do exactly what it is told.

Share this post


Link to post
Share on other sites
Guest
This topic is now closed to further replies.
Sign in to follow this  

  • Recently Browsing   0 members

    No registered users viewing this page.

×