Hijirikawa

- script asdfjkl -1,{ if(pvp_announce) .@menu$ = "Warp:[^00ff00Enabled^000000]Announce:Close"; else .@menu$ = "Warp:[^ff0000Disabled^000000]Announce:Close"; mes "[ PVP Warper ]"; mes "Warp?"; switch(select(.@menu$)){ case 1: close2; warp "prontera",0,0; .@enter$ = strcharinfo(0); addrid(0); if(pvp_announce){ announce "[ PvP Announcement ]: "+.@enter$+" has entered the PvP Room",bc_blue|bc_self; } end; case 2: if(pvp_announce){ pvp_announce = 0; mes "Announce disabled"; close; } else{ pvp_announce = 1; mes "Announce enabled"; close; } case 3: break; } end; OnInit: .announce = 1; // 0 = off end; } Untested.

- script asdfjkl -1,{ mes "[ PVP Warper ]"; mes "Warp?"; menu "Yeshhhh",-; close2; if(.announce){ announce strcharinf(0)+" has entered the PvP Room",0; } warp "prontera",0,0; end; OnInit: .announce = 1; // 0 = off end; } If you want players to be able to control it, that would be chaotic since they can just turn it off when someone turns it on, so I'm not gonna even go there since that just sounds silly.

Use OnPCLoadMapEvent and addtimer
- script asdfjkl -1,{ end; OnTick: getitem itm,itm_qty; dispbottom .sec+" second(s) has passed."; addtimer .time,"asdfjkl::OnTick"; end; OnPCLoadMapEvent: dispbottom "You have entered a special zone."; addtimer .time,"asdfjkl::OnTick"; end; OnInit: .time = 3000; .sec = .time/1000; .itm = 501; .itm_qty = 1; end; } prontera mapflag loadevent Untested.

- script asdfjkl -1,{ if(pvp_announce) .@menu$ = "Warp:[^00ff00Enabled^000000]Announce:Close"; else .@menu$ = "Warp:[^ff0000Disabled^000000]Announce:Close"; mes "[ PVP Warper ]"; mes "Warp?"; switch(select(.@menu$)){ case 1: close2; warp "prontera",0,0; .@enter$ = strcharinfo(0); addrid(0); if(pvp_announce){ announce "[ PvP Announcement ]: "+.@enter$+" has entered the PvP Room",bc_blue|bc_self; } end; case 2: if(pvp_announce){ pvp_announce = 0; mes "Announce disabled"; close; } else{ pvp_announce = 1; mes "Announce enabled"; close; } case 3: break; } end; OnInit: .announce = 1; // 0 = off end; } Untested.

Change all your close statements to end;
 
You should only use close; when there's a mes statement somewhere to avoid that kind of error. In this case, you aren't using any mes, so don't use close;

prontera,164,173,4 script Novice Enchantress 521,{ if(Weight > MaxWeight / 2) end; if( Class != 0){ end; } mes "[ ^FF0000Novice Enchantress^000000 ]"; mes "Welcome to ^FF0000RO NAME HERE^000000"; mes "I am here to assist you."; mes "Choose your class and join the adventure."; next; .@i = select( .Menu$ ) - 1; if( select( "Change to "+jobname( .JobList[.@i] )+"?:Cancel" ) == 1 ){ jobchange .JobList[.@i]; BaseLevel = .BaseLevel; JobLevel = .JobLevel; resetskill; resetstatus; set SkillPoint, 0; set SkillPoint, 118; //How many skill points do you want your players to have mes "[ ^FF0000Novice Enchantress^000000 ]"; mes "Welcome to RO NAME HERE."; close2; if(!#freebies){ switch(Class){ case 4008: //knight getitembound2 1108,2,1,10,0,0,0,0,0,1; //blade getitembound2 1408,2,1,10,0,0,0,0,0,1; //pike break; case 4009: //High Priest getitembound2 1602,2,1,10,0,0,0,0,0,1; //rod break; case 4010: //High Wizard getitembound2 1602,2,1,10,0,0,0,0,0,1; //rod break; case 4011: //Whitesmith getitembound2 1302,2,1,10,0,0,0,0,0,1; //axe break; case 4012: //Sniper getitembound2 1705,2,1,10,0,0,0,0,0,1; //composite bow break; case 4013: //Assassin Cross getitembound2 1208,2,1,10,0,0,0,0,0,1; //main gauche getitembound2 1264,2,1,10,0,0,0,0,0,1; //specialty jur break; case 4015: //Paladin getitembound2 1408,2,1,10,0,0,0,0,0,1; //pike break; case 4016: //Champion getitembound2 1505,2,1,10,0,0,0,0,0,1; //mace break; case 4017: //Professor getitembound2 1602,2,1,10,0,0,0,0,0,1; //rod break; case 4018: //Stalker getitembound2 1108,1,1,10,0,0,0,0,0,1; //blade getitembound2 1208,1,1,10,0,0,0,0,0,1; //main gauche getitembound2 1705,2,1,10,0,0,0,0,0,1; //composite bow break; case 4019: //Creator getitembound2 1302,2,1,10,0,0,0,0,0,1; //axe break; case 4020: //Bard getitembound2 1902,2,1,10,0,0,0,0,0,1; //violin getitembound2 1705,2,1,10,0,0,0,0,0,1; //composite bow break; case 4021: //Gypsy getitembound2 1951,2,1,10,0,0,0,0,0,1; //rope getitembound2 1705,2,1,10,0,0,0,0,0,1; //composite bow break; case 23: //Super Novice getitembound2 1602,2,1,10,0,0,0,0,0,1; //rod getitembound2 1208,2,1,10,0,0,0,0,0,1; //main gauche getitembound2 1108,2,1,10,0,0,0,0,0,1; //blade break; case 24: //Gunslinger getitembound2 13105,2,1,10,0,0,0,0,0,1; //garrison break; case 25: //Ninja getitembound2 1208,2,1,10,0,0,0,0,0,1; //main gauche getitembound2 13302,2,1,10,0,0,0,0,0,1; break; } set #freebies,1; } } end; OnInit: waitingroom "Jobchanger",0; setarray .JobList[0],4008,4009,4010,4011,4012,4013,4015,4016,4017,4018,4019,4020,4021,23,24,25; // class that player can choose from, refer to @jobchange in-game for the numbers .BaseLevel = 99; //max base level of your server .JobLevel = 70; //max job level of your server .@size = getarraysize( .JobList ); for( .@i = 0; .@i < .@size; .@i++ ) .Menu$ = .Menu$ + jobname( .JobList[.@i] )+":"; end; } Study how the script looks like, this one should be really easy to understand.

Why not use this one? http://pastebin.com/JZgsXCr7credit to owner.

*getwaitingroomusers "<NPC object name>"; This command get all the characters in the waiting room of the given NPC and stores their gids in the array .@waitingroom_users[]. Also, stores the the number of characters in the variable .@waitingroom_usercount Suppose you could use that.

morocc,161,90,5 script Poring#test 1002,{ if(.timer == 1) end; mes "Pay me?"; menu "Yes",-; if(zeny < .zeny) end; zeny -= .zeny; setnpcdisplay strnpcinfo(3), "Drops", .disguise; set .timer,1; initnpctimer; end; OnTimer5000: setnpcdisplay strnpcinfo(3), "Poring", .original; set .timer,0; stopnpctimer; end; OnInit: set .disguise, 1113; set .original, 1002; set .zeny, 1; end; } Working like a charm.

- script Healer -1,{ .@Price = 0; // Zeny required for heal .@Buffs = 1; // Also buff players? (1: yes / 0: no) .@Delay = 10; // Heal delay, in seconds if (@HD > gettimetick(2)) end; if (.@Price) { message strcharinfo(0), "Healing costs " + callfunc("F_InsertComma",.@Price) + " Zeny."; if (Zeny < .@Price) end; if (select("^0055FFHeal^000000:^777777Cancel^000000") == 2) end; Zeny -= .@Price; } specialeffect2 EF_HEAL2; percentheal 100,100; if (.@Buffs) { specialeffect2 EF_INCAGILITY; sc_start SC_INCREASEAGI,240000,10; specialeffect2 EF_BLESSING; sc_start SC_BLESSING,240000,10; } if (getbrokenid(1) >= 1) { repairall; } if(getgroupid() >= 5 ){ set(.@min, 5); // duration in minutes sc_start(SC_CP_HELM, (.@min * 1000 * 60), 1); sc_start(SC_CP_ARMOR, (.@min * 1000 * 60), 1); sc_start(SC_CP_SHIELD, (.@min * 1000 * 60), 1); sc_start(SC_CP_WEAPON, (.@min * 1000 * 60), 1); switch ( basejob ) { case Job_Alchemist: set .@spirit, 445; break; case Job_Monk: set .@spirit, 447; break; case Job_Star_Gladiator: set .@spirit, 448; break; case Job_Sage: set .@spirit, 449; break; case Job_Crusader: set .@spirit, 450; break; case Job_SuperNovice: set .@spirit, 451; break; case Job_Knight: set .@spirit, 452; break; case Job_Wizard: set .@spirit, 453; break; case Job_Priest: set .@spirit, 454; break; case Job_Bard: case Job_Dancer: set .@spirit, 455; break; case Job_Rogue: set .@spirit, 456; break; case Job_Assassin: set .@spirit, 457; break; case Job_Blacksmith: set .@spirit, 458; break; case Job_Hunter: set .@spirit, 460; break; case Job_Soul_Linker: set .@spirit, 461; break; default: if ( upper == 1 && baselevel < 70 ) set .@spirit, 494; } if ( .@spirit ) { sc_start4 sc_spirit, 360000, 5, .@spirit,0,0; skilleffect .@spirit, 5; } } if (.@Delay) @HD = gettimetick(2) + .@Delay; end; }  

If Statements
if(strcharinfo(3) == "payon"){ do payon stuff } else if(strcharinfo(3) == "prontera"){ do prontera stuff } else if(strcharinfo(3) == "alberta"){ do alberta stuff } IF + Switch Statements
if(strcharinfo(3) == "payon"){ set .@town,1; } if(strcharinfo(3) == "prontera"){ set .@town,2; } if(strcharinfo(3) == "alberta"){ set .@town,3; } switch(.@town){ case 1: payon here case 2: prontera here case 3: alberta here } Just do which ever will look cleaner and easier to follow, I'd pick the combo with switch if it was me though