Results 1 to 5 of 5

Thread: GetChooseOptions

  1. #1
    Join Date
    Nov 2011
    Posts
    1,532
    Mentioned
    0 Post(s)
    Quoted
    0 Post(s)

    Default GetChooseOptions

    To put it simply, GetChooseOptions never uses the TextType argument that's passed to the function, so the function always searches for all text colors.

    This can be fixed by adding a case block after the colors array definition in the function. Below is my quickfix for the record. I'm not sure what's the orange item used for so I just put it in every case.

    Simba Code:
    //Object Blue, Player White, Item Orange, Action Tan, NPC Yellow, Level Green
      Colors := [14342147, 14079443, 3832268, 9677229, 840143{, 710915}];
      //Below is what seems to be missing
      case TextType of
      'action': Colors := [14079443, 3832268, 9677229];
      'player': Colors := [14079443, 3832268, 9677229];
      'npc': Colors := [840143, 3832268];
      'object': Colors := [14342147, 3832268];
      end;

  2. #2
    Join Date
    Sep 2007
    Location
    Michigan
    Posts
    3,862
    Mentioned
    2 Post(s)
    Quoted
    1 Post(s)

    Default

    Quote Originally Posted by nosaj.421 View Post
    To put it simply, GetChooseOptions never uses the TextType argument that's passed to the function, so the function always searches for all text colors.

    This can be fixed by adding a case block after the colors array definition in the function. Below is my quickfix for the record. I'm not sure what's the orange item used for so I just put it in every case.

    Simba Code:
    //Object Blue, Player White, Item Orange, Action Tan, NPC Yellow, Level Green
      Colors := [14342147, 14079443, 3832268, 9677229, 840143{, 710915}];
      //Below is what seems to be missing
      case TextType of
      'action': Colors := [14079443, 3832268, 9677229];
      'player': Colors := [14079443, 3832268, 9677229];
      'npc': Colors := [840143, 3832268];
      'object': Colors := [14342147, 3832268];
      end;
    This is known. The tolerance has to be set so high now that its almost pointless to use TextType. I left it in there for compatibility reasons.
    (Scripts outdated until I update for new SRL changes)
    AK Smelter & Crafter [SRL-Stats] - Fast Fighter [TUT] [SRL-Stats]
    If you PM me with a stupid question or one listed in FAQ I will NOT respond. -Narcle
    Summer = me busy, won't be around much.

  3. #3
    Join Date
    Nov 2011
    Posts
    1,532
    Mentioned
    0 Post(s)
    Quoted
    0 Post(s)

    Default

    Noted. Then I will modify it in my script to do the ChooseOption correctly. FYI I was having problem with this. ChooseOption seems to always choose the first option that matches the text so I had to use the color options to make it choose the 2nd option.


  4. #4
    Join Date
    Feb 2011
    Location
    The Future.
    Posts
    5,600
    Mentioned
    396 Post(s)
    Quoted
    1598 Post(s)

    Default

    Those colours are the same -__________- in that picture..
    I am Ggzz..
    Hackintosher

  5. #5
    Join Date
    Nov 2011
    Posts
    1,532
    Mentioned
    0 Post(s)
    Quoted
    0 Post(s)

    Default

    I thought I needed the non-default colors to use the 2nd option Nvm now I found what to do

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
  •