PDA

View Full Version : TReflectBankItem.Withdraw(-1)



MariusK
06-15-2015, 04:29 PM
Hi,

I noticed that the function does not work, when trying to "Withdraw-All" (amount = -1). In order to fix this we need to change Bank.simba line 181 from:Result := Reflect.Text.ChooseOption('All', 200 + Random(100)); to: Result := Reflect.Text.ChooseOption('All ');

Thanks.

Fitta
06-15-2015, 05:17 PM
Are you implying that the space after "All" is what's needed or that the timer needs to be shorten? Or is it both?

I'm using it quite heavily in my scripts and it seems to work very well? What problems are you seeing?

Olly
06-15-2015, 05:19 PM
'All' is found in example: Alloy so it would pick the first option being Withdraw Alloy.

Appending a space would make sure It wouldn't choose that.

Fitta
06-15-2015, 05:24 PM
and I am convinced :)

Thank you MariusK for your feedback! We need to wait for elfyyy to see my pull request, and then it'll auto-update on your side of the world!

Note: Link to request
https://github.com/FittaVillavu/OSR-Reflection/commit/e55f97ab1fad0fd1424a10d8c2a2729cccdd91ff