search

dcoles / adventureland-code

Adventure Land character code
http://adventureland.dcoles.net/
MIT License
6 stars 2 forks source link

item.compound_all is not a function #1

Open MaZDuR opened 3 years ago

MaZDuR commented 3 years ago

I dont know if this exists as a missing include or if its an old deprecated function, but it appears that many of the merchant functions seems to be relying on this function which does not exist.

EDIT: Please note, this is for the 'console commands' the autobrain code seems to have a proper event loop (minus some missing items)

dcoles commented 3 years ago

Good catch. Thank you.

I ended up moving them into MerchantBrain but forgot about the /upgrade and /compound commands until a few days ago.

I've mostly been relying upon my merchant to automatically upgrade now, but if the separate commands were useful, it's not too hard to pull them out again.

MaZDuR commented 3 years ago

Good catch. Thank you.

I ended up moving them into MerchantBrain but forgot about the /upgrade and /compound commands until a few days ago.

I've mostly been relying upon my merchant to automatically upgrade now, but if the separate commands were useful, it's not too hard to pull them out again.

I am having issues with the merchantbrain not handling full inventory properly, so I am babysitting it while resetting the cycle to force it to upgrade items, it will handle items in the inventory fine, but it does not appear to account for items in the bank

dcoles commented 3 years ago

It's pretty dumb at the moment and mostly relies upon you having sufficient space in the account to juggle things around.

Making it try to compound items first will probably help out a little, but ultimately it might have to make some hard decisions such as discarding or selling items to free up space for better stuff.