Jump to content

  •  

Bug Tracker Migration

June 3rd
Good news everyone! The staff has decided that it is time to slowly kill off this Bug Tracker. We will begin the process of slowly migrating from this Bug Tracker over to our Github Issues which can be found here: https://github.com/HerculesWS/Hercules/issues

Over the next couple of days, I will be closing off any opportunity to create new reports. However, I still will keep the opportunity to reply to existing Bug Reports. Doing this will allow us to slowly fix any bug reports we have listed here so that we can easily migrate over to our Issue Tracker.

Update - June 7th 2015: Creating new bug posts has been disabled. Please use our https://github.com/HerculesWS/Hercules/issues tracker to post bugs. Users are still able to reply to existing bug posts.

- Administration

Issue Information

  • #006800

  • 0 - None Assigned

  • Fixed

Issue Confirmations

  • Yes (2)No (0)
Photo

script_commands.txt

Posted by Hercules Bot on 18 October 2012 - 03:18 PM

Originally posted by AnnieRuru
seems like I always post this every year lol
anyways 4 of them


1. boss_monster missing explanation


2.

*waitingroom "<chatroom name>",<limit>{,<event label>,<trigger>,<required zeny>,<min lvl>,<max lvl>};
....
...
// The NPC will have a box above its head, it will say "Disco - Waiting Room"
// and will have 8 waiting slots. Clicking this will enter the chat room, where
// the player will be able to wait until 7 players accumulate. Once this happens,
// it will cause the NPC "Bouncer" run the label "OnStart".

    waitingroom "Disco - Waiting Room",8,"Bouncer::OnStart",8;

please try this script, tested with double client
prontera,157,186,5    script    djfsdkfjd    100,{
    end;
OnInit:
    waitingroom "test", 3, strnpcinfo(0)+"::Onaa", 2;
    end;
Onaa:
    announce getwaitingroomstate( 0, strnpcinfo(0) )+" players and trigger this", 0;
    end;
}
this waitingroom can hold 2 players( + npc ), and once 2 players inside, will trigger Onaa label
if set both <trigger> and <limit> is the same, Onaa label will never trigger
so the <trigger> should be 1 number less than <limit>
that highlighted red should be 7, not 8


3.
http://rathena.org/b...hl__soundeffect
missing documentation - soundeffect only support 23 characters
and while at it, can you guys fix *soundeffect in script.c and @sound in atcommand.c to limit at 23 char ?


4.

*sprintf(<format>[,param[,param[,...]]]) [Mirei]

    C style sprintf. The resulting string is returned same as in PHP. All C format
    specifiers are supported except %n. More info: sprintf @ www.cplusplus.com.
    The number of params is only limited by eA's script engine.

    See thread: http://www.eathena.w...howtopic=190410

    Example:
    .@format$ = 'The %s contains %d monkeys';
    dispbottom(sprintf(.@format$, "zoo", 5));        //prints "The zoo contains 5 monkeys"
    dispbottom(sprintf(.@format$, "barrel", 82));    //prints "The barrel contains 82 monkeys"

please click on the topic link

Originally posted by Akkarin
Updating report status while i test these out.

*prods Annie to make them as separate reports next time*

The sprintf link you posted is something i don't have access to on the old boards. I also don't see why it would be required - that example using dispbottom can be output using the normal methods just as easily.

I'll fix up the others.

Originally posted by AnnieRuru
4.
yes that's the point, because the link is already broken xD


*ouch*
I always did like this every year in eathena forum
http://www.eathena.w...ker&showbug=373
ok I'll separate them next time

Originally posted by Akkarin
Partial fix in [rev=16864] for points 3 and 4.

Originally posted by frenzmu06
@off from issues 1-4 but since saw the sound effect documentation maybe just some tip :D >add a documentation about the max chars a map_name can hold (forgot exactly how much it can but i think it reads only 11 and wont read the map when map name exceeds) affects map_cache, maps_athena.conf idk if it affects scripts
can u take a look bout this mr. Akkarin :)

Originally posted by Euphy
Fixed the rest in [rev=16963], thanks for reporting this!