Results 1 to 10 of 10

Thread: Invalid number of parameters for smart, does anyone have a smart file I can have?

  1. #1
    Join Date
    Sep 2010
    Posts
    5,762
    Mentioned
    136 Post(s)
    Quoted
    2739 Post(s)

    Default Invalid number of parameters for smart, does anyone have a smart file I can have?

    Hey, I wanted to change my SMART from 07 to EOC so i just updated SRL so it would overide the smart params for 07. Now when I try to run, I get invalid number of parameters at line 281 :s

    The params look right for EOC?

    Simba Code:
    function GetSmartParams: TStringArray;
    var
      Params: TStringArray;
      Page: String;
    begin
      Page := Between('<iframe id="game" src="', '"', GetPage('http://www.runescape.com/game.ws?j=1'));
      Params := Explode(',', Page);
      Result := Params;
    end;


    Can someone else upload their smart include file, I think when i used the 07 auto installer it messed with some other smart params CBA to figure out

  2. #2
    Join Date
    Apr 2013
    Posts
    395
    Mentioned
    1 Post(s)
    Quoted
    177 Post(s)

    Default

    Remove the -1 at line 294?

    Result := SmartSpawnClient(ReplaceWrap(PluginPath, '\', '/', [rfReplaceAll]), Params[0], ',' + Params[1], 765, 553, 's', '', '', -1);

  3. #3
    Join Date
    Mar 2006
    Location
    Belgium
    Posts
    3,564
    Mentioned
    111 Post(s)
    Quoted
    1475 Post(s)

    Default

    Just re-install?

    Creds to DannyRS for this wonderful sig!

  4. #4
    Join Date
    Sep 2010
    Posts
    5,762
    Mentioned
    136 Post(s)
    Quoted
    2739 Post(s)

    Default

    Quote Originally Posted by Haxz View Post
    Remove the -1 at line 294?

    Result := SmartSpawnClient(ReplaceWrap(PluginPath, '\', '/', [rfReplaceAll]), Params[0], ',' + Params[1], 765, 553, 's', '', '', -1);
    that worked, you sir get += 7 rep

  5. #5
    Join Date
    Apr 2013
    Posts
    395
    Mentioned
    1 Post(s)
    Quoted
    177 Post(s)

    Default

    Not sure why but devs should update the SmartParams, like they did to OSR-SRL include. Since nobody using Smart 7.2 anymore

  6. #6
    Join Date
    Nov 2011
    Location
    United States
    Posts
    815
    Mentioned
    6 Post(s)
    Quoted
    284 Post(s)

    Default

    Quote Originally Posted by Haxz View Post
    Not sure why but devs should update the SmartParams, like they did to OSR-SRL include. Since nobody using Smart 7.2 anymore
    Im 99% sure it is updated for Smart8 Already, only need to remove the -1 if you running an older version. Unless they downgraded for OSRS? Idk , but last i checked that problem was only for 7.2 and older

  7. #7
    Join Date
    Apr 2013
    Posts
    395
    Mentioned
    1 Post(s)
    Quoted
    177 Post(s)

    Default

    Quote Originally Posted by Itankbots View Post
    Im 99% sure it is updated for Smart8 Already, only need to remove the -1 if you running an older version. Unless they downgraded for OSRS? Idk , but last i checked that problem was only for 7.2 and older
    You need to remove the -1 if you're using Smart 8 as well. Try force update your SRL, I believe you will encounter the error again.

    In OSR-SRL smartparams, its already excluded, looks like this
    Simba Code:
    //Removal of Smart 7.2?
      //{$IFDEF SMART8}
      Result := SmartSpawnClient(ReplaceWrap(PluginPath, '\', '/', [rfReplaceAll]), 'http://oldschool' + World + '.runescape.com/', 'j1', 765, 503, '', '', '');
      //{$ELSE}
      //Result := SmartSpawnClient(ReplaceWrap(PluginPath, '\', '/', [rfReplaceAll]), Params[0], ',' + Params[1], 765, 553, 's', '', '');
      //{$ENDIF}

    In SRL5...
    Simba Code:
    {$IFDEF SMART8}
      Result := SmartSpawnClient(ReplaceWrap(PluginPath, '\', '/', [rfReplaceAll]), Params[0], ',' + Params[1], 765, 553, 's', '', '');
      {$ELSE}
      Result := SmartSpawnClient(ReplaceWrap(PluginPath, '\', '/', [rfReplaceAll]), Params[0], ',' + Params[1], 765, 553, 's', '', '', -1);
      {$ENDIF}

  8. #8
    Join Date
    Nov 2011
    Location
    United States
    Posts
    815
    Mentioned
    6 Post(s)
    Quoted
    284 Post(s)

    Default

    Smart8 Needs the -1 . Ive forced updated everything to check before i even posted, And it auto updates, last time i got that error was when i ran Smart7.2

    The -1 has to do with the the NavBar at the top of runescape i believe. Which OSRS does NOT have, which is why its commented out.

    Either way doesn't matter, you guys got it working for you and thats all that matters.

  9. #9
    Join Date
    Apr 2013
    Posts
    395
    Mentioned
    1 Post(s)
    Quoted
    177 Post(s)

    Default

    Please note that I'm not trying to start an argument or fight, just trying to get my doubts cleared.

    Simba Code:
    {$IFDEF SMART8}
      Result := SmartSpawnClient(ReplaceWrap(PluginPath, '\', '/', [rfReplaceAll]), Params[0], ',' + Params[1], 765, 553, 's', '', '');
      {$ELSE}
      Result := SmartSpawnClient(ReplaceWrap(PluginPath, '\', '/', [rfReplaceAll]), Params[0], ',' + Params[1], 765, 553, 's', '', '', -1);
      {$ENDIF}
    If define Smart8, then no -1. If not define Smart8, then -1. But you say Smart8 needs -1. You commented out the -1 on OSRS because of no NavBar, but you commented it out on EOC too to make it able to compile.

  10. #10
    Join Date
    Sep 2010
    Posts
    5,762
    Mentioned
    136 Post(s)
    Quoted
    2739 Post(s)

    Default

    Quote Originally Posted by Haxz View Post
    Please note that I'm not trying to start an argument or fight, just trying to get my doubts cleared.

    Simba Code:
    {$IFDEF SMART8}
      Result := SmartSpawnClient(ReplaceWrap(PluginPath, '\', '/', [rfReplaceAll]), Params[0], ',' + Params[1], 765, 553, 's', '', '');
      {$ELSE}
      Result := SmartSpawnClient(ReplaceWrap(PluginPath, '\', '/', [rfReplaceAll]), Params[0], ',' + Params[1], 765, 553, 's', '', '', -1);
      {$ENDIF}
    If define Smart8, then no -1. If not define Smart8, then -1. But you say Smart8 needs -1. You commented out the -1 on OSRS because of no NavBar, but you commented it out on EOC too to make it able to compile.
    hmm IDK I can always make my scripts with a 50 y offset

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
  •