[Update] XenoBot Apophis v14.7.6.656 [10.50 & Apophis Global Release]

Quote:

---

Originally Posted by Nakuu

Changed it as you told - to call it only once - and it didn't deposit my rare items at all, dunno if it's any better sadly :(

Deposited (stackable) items to first backpack in depo just fine (it had to go deeper into cascaded backpacks) then re-opened browse field (to open second backpack in depot) and got stuck here (as you can see on the screenshot) trying to do some crazy shit, eventually stopped trying and ran away.
Here is the screenshot: http://i.imgur.com/Y1fdQl3.png

---

Okay that's a different issue then, it's still recommended to use one function. Nick said he'll look into it when he gets home.
I'll take a look after work if he hasn't already found the solution.

Quote:

---

Originally Posted by Syntax

Okay that's a different issue then, it's still recommended to use one function. Nick said he'll look into it when he gets home.
I'll take a look after work if he hasn't already found the solution.

---

I think it's supposed to close the depot backpack (once it's done depositing to that backpack) but it closes BrowseField with Locker instead (or maybe it fails to detect that its already open and calls OpenLocker funcion for no reason) - unless it's intended but seems pointless to close BrowseField and then reopen it and leave backpack that won't be needed anymore still open ;d

its easier to make the depot function based on one index per item set , like DepositItems({index=0, items = {3154, 3455, 6555}}, {index =2, items = {4456, 6777, 1123}})

atleast that's how i did in the deposit function i made

Quote:

---

Originally Posted by Nakuu

I think it's supposed to close the depot backpack (once it's done depositing to that backpack) but it closes BrowseField with Locker instead - unless it's intended but seems pointless to close BrowseField and then reopen it and leave backpack that won't be needed anymore still open ;d

---

Okay thanks, most likely the BrowseField change in the recent update then. I assumed it opened the Locker in the same window as BrowseField, which was incorrect.
Nick will be home soon to test and patch the issue.

Quote:

---

Originally Posted by ppgab

its easier to make the depot function based on one index per item set , like DepositItems({index=0, items = {3154, 3455, 6555}}, {index =2, items = {4456, 6777, 1123}})

atleast that's how i did in the deposit function i made

---

That looks worse tbh.
You're putting the focus on the slots, and not the items.

yeah it does looks ugly lol, but it goes smoother

Quote:

---

Originally Posted by ppgab

yeah it does looks ugly lol, but it goes smoother

---

Probably, the current depositor is very old, one of the first things I made, and even though I was pr0 back then ;) it's pretty dated.
After I get done moving into my new place, I'll give you guys some new depositor/withdrawing functions.
I have a few improvements I need to make to them, and make them backwards compatible with the current input.

Quote:

---

Originally Posted by Syntax

Probably, the current depositor is very old, one of the first things I made, and even though I was pr0 back then ;) it's pretty dated.
After I get done moving into my new place, I'll give you guys some new depositor/withdrawing functions.
I have a few improvements I need to make to them, and make them backwards compatible with the current input.

---

yeah, those two functions in the xenolualib could be cleaner imo, but the only thing i would add to self.deposit (i also added to mine) is keep opening equiped bps that are full, so it doesn't miss loot (probably shoud ignore this if Container(i):ID() == Self.Backpack().id).

this is why i separate by index in mine :
lua code:

Self.OpenDepot()

local dp = Container('Depot Chest')

for i = 0, #arg do

    dp:UseItem(arg[i].Index, true) -- opening in new window

    -- loop trough bps and move found items

    for j = dp:Index()+1, #Container.GetAll() do -- closing what was opened

        Container(j):Close()

    end

end

i know yu pro but i tought my function was clever :D

THE DEPOSITOR BUGS ARE FIXED, PLEASE CLOSE TIBIA AND RESTART XENOSUITE TO DOWNLOAD THE CHANGES

Detail:
The deposit function has been completely re-written to use Browse Field, minimize backpacks, and close them after use. It is now also smarter and faster than before.

Quote:

---

Originally Posted by DarkstaR

THE DEPOSITOR BUGS ARE FIXED, PLEASE CLOSE TIBIA AND RESTART XENOSUITE TO DOWNLOAD THE CHANGES

Detail:
The deposit function has been completely re-written to use Browse Field, minimize backpacks, and close them after use. It is now also smarter and faster than before.

---

You're doing an amazing work. Thanks.

Quote:

---

Originally Posted by Tripkip

LOL Waste of programming time.

---

Considering I haven't had this issue with any other program, I would think the fix would be time well spent.

Regards,

Tibian