SRL Archive - Bugs

TReflectBankItem.Withdraw(-1)


MariusK

2015-06-15 12:29:25

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

2015-06-15 13:17:20

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

2015-06-15 13:19:21

'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

2015-06-15 13:24:45

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