Results 1 to 4 of 4

Thread: TReflectBankItem.Withdraw(-1)

  1. #1
    Join Date
    May 2013
    Posts
    75
    Mentioned
    3 Post(s)
    Quoted
    48 Post(s)

    Default TReflectBankItem.Withdraw(-1)

    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:
    Simba Code:
    Result := Reflect.Text.ChooseOption('All', 200 + Random(100));
    to:
    Simba Code:
    Result := Reflect.Text.ChooseOption('All ');

    Thanks.

  2. #2
    Join Date
    Feb 2013
    Location
    Rimmington
    Posts
    319
    Mentioned
    33 Post(s)
    Quoted
    183 Post(s)

    Default

    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?





  3. #3
    Join Date
    Nov 2011
    Location
    England
    Posts
    3,072
    Mentioned
    296 Post(s)
    Quoted
    1094 Post(s)

    Default

    '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.

  4. #4
    Join Date
    Feb 2013
    Location
    Rimmington
    Posts
    319
    Mentioned
    33 Post(s)
    Quoted
    183 Post(s)

    Default

    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-...a2729cccdd91ff





Thread Information

Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •