Ninja

Hi, I think the cash shop (through the cash shop button) doesn't add any expiry time for items being put inside the rental items. I've seen numerous posts about rental items using @rentitem but I think using the official cash shop doesn't work that way. can anyone help me?

I was assuming he doesn't have any root access

Alright. I think I can find my way around knowing that.

if that's the case then he can just find and replace all "gettime(3)" function calls and replace them with "(gettime(3)-3)" that's if the servertime is @ GMT+0.

I see. Hahaha then I think I'm at the wrong side of the forums. THanks for the heads up though.

Yep, did. But my problem is about just anything that's being bought via an npc shop (shop script and not via callshop) and also vend

no problem

Yeah but when I looked at the src for mapmove and didn't find anything juicy. >_<

I'm not really good at source too, just thought of using my knowledge on logical statements from what you presented

but this one only recognizes the ones that use npcshop* right?

Yes. Like OnPcLoginEvent and the likes.

case MO_EXTREMITYFIST: if(sstatus->sp <= 7000) skillratio += 100 * (7 + sstatus->sp / 10); else skillratio += 100 * (7 + 7000 / 10); skillratio = min(500000,skillratio); //We stop at roughly 50k SP for overflow protection break; You can also comment out the last skill ratio statement because you have already set the max sp in the computation

the header comment says it all: /*========================================== * Invoked when a player has received damage *------------------------------------------*/ anyway, your issue about salamander and necromancer still triggering it is that because these monsters are considered as bosses. if you don't want that to happen you can manually change their "mode" in your mob_db sql tables by checking https://rathena.org/wiki/Custom_Mobs#Database_Structure by the way src->type == BL_MOB && is_boss(src) try using only is_boss(src) since a boss types can only be related to mobs. not sure about the part where a player is the one attacking though.

if (.@i < 3) limits you to do 3 quests. so try changing if (.@i < 3) // hunting quest wait time mes "The waiting time for a "+.@type$[.@i]+" collection quest has expired. You can now take another request."; else // buff wait time mes "The waiting time for storing "+.@crystal$[.@i]+" Energy Crystals has expired. You can now store them again."; to this mes "Please store your "+.@crystal$[.@i]+" Energy Crystals.";

Yes, I am aware that it works if you take only the event script part, but if you put in the monster flags jupe_ele,0,0,0,0 monster Richard Frost 3300,90,5000,0,0 jupe_ele,0,0,0,0 monster Melody Windrush 3301,80,5000,0,0 jupe_ele,0,0,0,0 monster Lara Nightstone 3302,65,5000,0,0 jupe_ele,0,0,0,0 monster Sakura de Sylva 3303,65,5000,0,0 jupe_ele,0,0,0,0 monster Tundra Neyomi 3304,65,5000,0,0 jupe_ele,139,158,20,11 monster Clown Richard 3200,1,9200000,600000,1 jupe_ele,138,138,36,34 monster Gypsy Melody 3201,1,9200000,600000,1 jupe_ele,114,138,12,16 monster Paladin Lara 3202,1,9200000,600000,1 jupe_ele,139,117,20,11 monster Professor Sakura 3203,1,9200000,600000,1 jupe_ele,163,138,12,16 monster Stalker Tundra 3204,1,9200000,600000,1 It doesn't. I don't know if it works on you, but it doesn't on mine.

I think this is more of a source request I looked at the diff patch given from the link you gave. try this and please do not just copy and paste, understand what I did so you can do the changes better if you find something wrong. here's what I did: 1. changed the func type from void to int of autoattack_motion 2. restructured the if-loop to for-loop and added a counter //replace void with int so that it can return a value int autoattack_motion(struct map_session_data* sd) { int i, target_id; for(i=0;i<=9;i++) { target_id=0; map_foreachinarea(buildin_autoattack_sub, sd->bl.m, sd->bl.x-i, sd->bl.y-i, sd->bl.x+i, sd->bl.y+i, BL_MOB, &target_id); if(target_id) { unit_attack(&sd->bl,target_id,1); break; } target_id=0; //add a return value if it killed something return 1; } if(!target_id) { unit_walktoxy(&sd->bl,sd->bl.x+(rand()%2==0?-1:1)*(rand()%10),sd->bl.y+(rand()%2==0?-1:1)*(rand()%10),0); } // default: return 0 if it killed nothing return 0; } int autoattack_timer(int tid, unsigned int tick, int id, intptr_t data) { struct map_session_data *sd=NULL; sd=map_id2sd(id); if(sd==NULL) return 0; /* Replace this * if(sd->sc.option & OPTION_AUTOATTACK) * { * autoattack_motion(sd); * add_timer(gettick()+2000,autoattack_timer,sd->bl.id,0); * } * return 0; */ // With this and add a counter for kill count (i) for(int i=0;(sd->sc.option & OPTION_AUTOATTACK)&&(i<500){ i += autoattack_motion(sd); add_timer(gettick()+2000,autoattack_timer,sd->bl.id,0); } // End }

I don't really think you have to because if you browse through the code you'll see something like "gettime()" and the functions description says: gettime(<type>) This function will return specified information about the current system time. 1 - Seconds (of a minute) 2 - Minutes (of an hour) 3 - Hour (of a day) 4 - Week day (0 for Sunday, 6 is Saturday) 5 - Day of the month. 6 - Number of the month. 7 - Year. 8 - Day of the year.

thanks! how about detecting the event that somebody buys something? is there such a thing?

Have you tried recompiling your binaries?

It may seem a bit weird but I tried using your script and found this error in the mapserver What I did was to take off the "My" from the events OnMyMVPDead and OnMy99Dead, and your script worked fine. I'm not really sure what went wrong there. and just a tip try using '-' instead of "jupe_ele,0,0,0" if you're creating an event script final code: jupe_ele,0,0,0,0 monster Richard Frost 3300,90,5000,0,0 jupe_ele,0,0,0,0 monster Melody Windrush 3301,80,5000,0,0 jupe_ele,0,0,0,0 monster Lara Nightstone 3302,65,5000,0,0 jupe_ele,0,0,0,0 monster Sakura de Sylva 3303,65,5000,0,0 jupe_ele,0,0,0,0 monster Tundra Neyomi 3304,65,5000,0,0 jupe_ele,139,158,20,11 monster Clown Richard 3200,1,9200000,600000,1 jupe_ele,138,138,36,34 monster Gypsy Melody 3201,1,9200000,600000,1 jupe_ele,114,138,12,16 monster Paladin Lara 3202,1,9200000,600000,1 jupe_ele,139,117,20,11 monster Professor Sakura 3203,1,9200000,600000,1 jupe_ele,163,138,12,16 monster Stalker Tundra 3204,1,9200000,600000,1 jupe_ele,0,0,0 script mvp_jupe_ele -1,{ OnTimer2000: // delay1 = 6000000 ms (100 min) stopnpctimer; sleep rand(0,1)*60000; // 0 to 1 minutes OnInit: // Select Coordinates to summon a random MVP on switch(rand(1,7)) { case 1: set .@x,77; set .@y,251; break; case 2: set .@x,147; set .@y,224; break; case 3: set .@x,219; set .@y,219; break; case 4: set .@x,244; set .@y,120; break; case 5: set .@x,149; set .@y,41; break; case 6: set .@x,53; set .@y,109; break; case 7: set .@x,149; set .@y,151; break; } set .@mob,rand(3200,3204); monster "jupe_ele",.@x,.@y,strmobinfo(1,.@mob),.@mob,1,strnpcinfo(3)+"::OnMVPDead"; // Select Coordinates to summon a random 99 on switch(rand(1,7)) { case 1: set .@x2,77; set .@y2,251; break; case 2: set .@x2,147; set .@y2,224; break; case 3: set .@x2,219; set .@y2,219; break; case 4: set .@x2,244; set .@y2,120; break; case 5: set .@x2,149; set .@y2,41; break; case 6: set .@x2,53; set .@y2,109; break; case 7: set .@x2,149; set .@y2,151; break; } set .@mob2,rand(3305,3309); monster "jupe_ele",.@x2,.@y2,strmobinfo(1,.@mob2),.@mob2,1,strnpcinfo(3)+"::On99Dead"; end; OnMVPDead: killmonster "jupe_ele",strnpcinfo(3)+"::On99Dead"; initnpctimer; On99Dead: end; }

// Configurations set .@max_ranking, 10; // Max Rankings to display. And you should look what Emistry just said.

what does exactly mean? by the looks of it, the script gives +1 pt per kill. about Please try this. I've put comments on where my changes were done. //===== eAthena Script ======================================= //= eAthena PVP-Ladder //===== By: ================================================== //= Terces //===== Current Version: ===================================== //= 1.7.2 //===== Compatible With: ===================================== //= Any Athena Version that has "for" implemented //===== Credits: ============================================= //= The whole eAthena Board and the eAthena Chat //===== Notes: =============================================== //= Note 1: //= Experience gain works like this formula: //= ($@LadderLength-(<new position>-1))*$@LadderExp //= so if someone gets the first place he'll get //= $@LadderLength*$@LadderExp experience points. //============================================================ //= Fixxed by Dante //= V1.7.2 //= Hosted by Dant3 aka Dante // HAFE FUN! READ THE TOPIC! IT DONT WORKS MAYBE ON OLDER REV // // // LAST UPDATE: 12th,February.2008 // // Main Script - Terces (Main Thanks to Terces ) // V1.7.0 - Dant3 // v1.7.1 - AnnieRuRu // v1.7.2 - Dant3 // THIS SCRIPT WORKS ONLY ON THE NEW EATHENA REVISIONS! ENJOY! - script PVPLADDEROPTIONS -1,{ end; OnInit: // 0 = Disabled (NO) // 1 = Enable (YES) set $@languageselect, 0; //Enable/Disable language selection [English/German] (Default: 1) set $@LadderAnnounce, 1; //Enable/Disable the announcement when a char reaches a new position in the ladder (Default: 1) set $@LadderLength, 30; //Set the length of the Ladder [!Not higher than 128!] (Default: 30) set $@LadderSteps, 10; //Set the views per page (Default: 10) set $@LadderExpGain, 100; //Enable/Disable experience gain when reaching a new position in the ladder (*Note1) set $@LadderExp, 150; //Set Experience gain value (*Note1) // Update v1.7.0 set $@LadderZenyGain, 1; //Enable/Disable zeny gain when reaching a new position in the ladder set $@LadderZeny, 100; //Set Zeny gain value set $@LadderChatRoom, 1; //Enable/Disable an Chat Room over the NPC with the message "PvP Ladder" (Users can not enter the Chat room) // Update v1.7.1 (Thanks to AnnieRuRu) // Added GM PvP Ladder Reset Function // Update v1.7.2 // Thanks to for the Feedback set $@LadderAskLogin, 1; //Enable/Disable that the NPC asks about the Broadcast when a Player logins / or with the Npc Chat (0 = Login Ask , 1 = Npc Chat) Related: 0 } // Request by iSiune set pvppts, 0; setarray .PvPMap$[0],"All"; //put all the maps you want here inside quotation marks and separate them with commas OnMapCheck: if ( .PVPMap$[0] != "All") { for(set .@i, 0; .@i <= (getarraysize(.PVPMap$)-1); set .@i, .@i+1){ if(strcharinfo(3) == .PvPMap$[.@i]) { return 1; } } return 0; } return 1; // End - script PVPLADDERS -1,{ OnPCKillEvent: // Request by iSiune set .@map, callsub (OnMapCheck); if !(.@map) {end;} // End setarray .@Map$[0],"cell_game"; if (terces_PVP_resets != $terces_PVP_resets) { dispbottom "Please relog your character, in order to get the correct kills for you in the PVP ladder."; dispbottom "Because a GM just reset the ladder."; end; } if ( killedrid == getcharid(3) ) { set PVPDeaths, PVPDeaths +1; set #PVPDeathsAccount, #PVPDeathsAccount +1; set @PVPDeathstoday, @PVPDeathstoday +1; end; } set $@PVPcounter,$@PVPcounter+1; set getd("$@PVPKill"+$@PVPcounter),getcharid(3); //getd to avoid errors when more than 1 people kill someone attachrid(killedrid); set PVPDeaths,PVPDeaths+1; set @PVPDeathstoday,@PVPDeathstoday+1; set #PVPDeathsAccount,#PVPDeathsAccount+1; set getd("$@PVPkilledplayer"+$@PVPcounter+"$"), strcharinfo(0); //again, getd to avoid possible glitches detachrid; attachrid(getd("$@PVPKill"+$@PVPcounter)); CountKills: set PVPKills,PVPKills+1; set @PVPKillstoday,@PVPKillstoday+1; set #PVPKillsAccount,#PVPKillsAccount+1; setarray @playerstats[0],@PVPKillstoday,@PVPDeathstoday,PVPKills,PVPDeaths,#PVPKillsAccount,#PVPDeathsAccount; l_ladder: set @considerdeath,0; for (set @PosinLadder, 0; @PosinLadder < $@LadderLength; set @PosinLadder, @PosinLadder + 1){ if (PVPKills >= $terces_PVP_kills[@PosinLadder]){ //Player deserves to be in the ladder //Check if Death plays a role on the position if ((PVPKills == $terces_PVP_kills[@PosinLadder]) && (PVPDeaths > $terces_PVP_deaths[@PosinLadder])) set @considerdeath,1; //Consider Deaths //Check if the player only topped his own scores if ($terces_PVP_names$[@PosinLadder] == strcharinfo(0)){ set $terces_PVP_kills[@PosinLadder],PVPKills; set $terces_PVP_deaths[@PosinLadder],PVPDeaths; end; } //Moves all characters in the Ladder for (set @beginmoving, $@LadderLength; @beginmoving >= (@PosinLadder+@considerdeath); set @beginmoving, @beginmoving - 1){ if ($terces_PVP_names$[@beginmoving] == strcharinfo(0)){ //If the player already is in the Ladder it only has to move players between characters new position and characters old position callsub L_LadderMove,0; end; } else if (@beginmoving == (@PosinLadder+@considerdeath)){ //Player is not in the Ladder and therefor it has to move all players from characters new position downwards callsub L_LadderMove,1; end; } } end; } } L_LadderMove: if (getarg(0) == 0) set @length,@beginmoving; if (getarg(0) == 1) set @length,$@LadderLength; for (set @movecycle, @length; @movecycle > (@PosinLadder+@considerdeath); set @movecycle, @movecycle - 1){ set $terces_PVP_names$[@movecycle],$terces_PVP_names$[(@movecycle-1)]; set $terces_PVP_kills[@movecycle],$terces_PVP_kills[(@movecycle-1)]; set $terces_PVP_deaths[@movecycle],$terces_PVP_deaths[(@movecycle-1)]; set $terces_PVP_times[@movecycle],$terces_PVP_times[(@movecycle-1)]; } //sets the character's stats in the new position set $terces_PVP_names$[(@PosinLadder+@considerdeath)],strcharinfo(0); set $terces_PVP_kills[(@PosinLadder+@considerdeath)],PVPKills; set $terces_PVP_deaths[(@PosinLadder+@considerdeath)],PVPDeaths; set $terces_PVP_times[(@PosinLadder+@considerdeath)],gettimetick(2); //Experience Gain if ($@LadderExpGain == 1){ set BaseExp,BaseExp+(($@LadderLength-((@PosinLadder+@considerdeath)-1))*$@LadderExp); dispbottom "You have just been rewarded with "+(($@LadderLength-((@PosinLadder+@considerdeath)-1))*$@LadderExp)+" base experience points"; } //Zeny Gain if ($@LadderZenyGain == 1){ set Zeny,Zeny+(($@LadderLength-((@PosinLadder+@considerdeath)-1))*$@LadderZeny); dispbottom "You have just been rewarded with "+(($@LadderLength-((@PosinLadder+@considerdeath)-1))*$@LadderZeny)+" Zeny"; } //Announcement (Setting for Char) if (pvpbroadcast == 2){ announce strcharinfo(0)+" just claimed the "+((@PosinLadder+@considerdeath)+1)+". Position in the Top "+$@LadderLength+" after killing "+getd("$@PVPkilledplayer"+$@PVPcounter+"$")+".",16; } //Announcement (Setting for Account) if (#pvpbroadcast == 2){ announce strcharinfo(0)+" just claimed the "+((@PosinLadder+@considerdeath)+1)+". Position in the Top "+$@LadderLength+" after killing "+getd("$@PVPkilledplayer"+$@PVPcounter+"$")+".",16; } end; OnPCLoginEvent: if (terces_PVP_resets != $terces_PVP_resets){ set PVPDeaths,0; set @PVPDeathstoday,0; set #PVPDeathsAccount,0; set PVPKills,0; set @PVPKillstoday,0; set #PVPKillsAccount,0; set terces_PVP_resets,$terces_PVP_resets; } if ($@LadderAskLogin == 0){ if (#pvpbroadcast == 2) goto L_ignore; if (pvpbroadcast == 2) goto L_ignore; if (#pvpbroadcast == 1) goto L_ignore; if (pvpbroadcast == 1) goto L_ignore; if (pvpbroadcast == 0) mes "Hi, "+strcharinfo(0)+""; mes "Me is the PvP Top Management!"; mes "Let me ask you one Question and i will let you play!"; menu "Later please",L_close,"Okay, fine",L_fine; L_fine: next; mes "Well, do you want read Broadcast Messages when a Player reach a new position on the PvP Ranglist?"; menu "Yeah, why not",L_accept,"No, thanks",L_accept_2,"Wtf is Broadcast?",L_broadcast; L_accept: next; mes "Yay, thank you! ^_^"; mes "Will this Setting work on all your Chars on this account or only this char?"; menu "Account",L_account,"Only this char",L_char; L_account: set #pvpbroadcast,2; next; mes "Okay, "+strcharinfo(0)+""; mes "Thank you for your time and have fun!"; close; L_char: set pvpbroadcast,2; next; mes "Okay, "+strcharinfo(0)+""; mes "Thank you for your time and have fun!"; goto access_eng; close; L_accept_2: next; mes "Yay, thank you! ^_^"; mes "Will this Setting work on all your Chars on this account or only this char?"; menu "Account",L_account_2,"Only this char",L_char_2; L_close: close; L_account_2: set #pvpbroadcast,1; next; mes "Okay, "+strcharinfo(0)+""; mes "Thank you for your time and have fun!"; close; L_char_2: set pvpbroadcast,1; next; mes "Okay, "+strcharinfo(0)+""; mes "Thank you for your time and have fun!"; goto access_eng; close; L_broadcast: mes "Broadcasts are this yellow messages from the GMs"; goto L_fine; } } rune,102,48,1 script PvP Ladder 57,{ if ($@LadderChatRoom == 1) { waitingroom "PvP Top "+$@LadderLength+"",0; // Look on the configuration! } set @name$,"[PvP-Ladder]"; //Colour of: Position Name Kills Deaths Time setarray @colour$[0], "^996600", "^006699", "^00AA00", "^FF0000", "^EE8800"; if ($@LadderAskLogin == 1){ if (#pvpbroadcast == 2) goto access_eng; if (pvpbroadcast == 2) goto access_eng; if (#pvpbroadcast == 1) goto access_eng; if (pvpbroadcast == 1) goto access_eng; if (pvpbroadcast == 0) mes "Hi, "+strcharinfo(0)+""; mes "Me is the PvP Top Management!"; mes "Let me ask you one Question and i will let you play!"; menu "Later please",L_close,"Okay, fine",L_fine; L_fine: next; mes "Well, do you want read Broadcast Messages when a Player reach a new position on the PvP Ranglist?"; menu "Yeah, why not",L_accept,"No, thanks",L_accept_2,"Wtf is Broadcast?",L_broadcast; L_accept: next; mes "Yay, thank you! ^_^"; mes "Will this Setting work on all your Chars on this account or only this char?"; menu "Account",L_account,"Only this char",L_char; L_account: set #pvpbroadcast,2; next; mes "Okay, "+strcharinfo(0)+""; mes "Thank you for your time and have fun!"; close; L_char: set pvpbroadcast,2; next; mes "Okay, "+strcharinfo(0)+""; mes "Thank you for your time and have fun!"; goto access_eng; close; L_accept_2: next; mes "Yay, thank you! ^_^"; mes "Will this Setting work on all your Chars on this account or only this char?"; menu "Account",L_account_2,"Only this char",L_char_2; L_close: close; L_account_2: set #pvpbroadcast,1; next; mes "Okay, "+strcharinfo(0)+""; mes "Thank you for your time and have fun!"; close; L_char_2: set pvpbroadcast,1; next; mes "Okay, "+strcharinfo(0)+""; mes "Thank you for your time and have fun!"; goto access_eng; close; L_broadcast: mes "Broadcasts are this yellow messages from the GMs"; goto L_fine; } access_eng: mes @name$; mes "Hello "+strcharinfo(0)+"..."; mes "If you want to I can show you your PVP stats."; next; M_selection_eng: if ( getgmlevel () < 99 ) menu "Show me the PVP Ladder",M_Ladder_eng,"PvP stats since my login",M_seitLogin_eng,"PvP stats of this Char",M_dieserChar_eng,"Stats of the whole account",M_vomAccount_eng,"Cancel",M_abbrechen_eng; else menu "Show me the PVP Ladder",M_Ladder_eng,"PvP stats since my login",M_seitLogin_eng,"PvP stats of this Char",M_dieserChar_eng,"Stats of the whole account",M_vomAccount_eng,"Reset Ladder",L_reset,"Cancel",M_abbrechen_eng; M_Ladder_eng: mes "Alright...I'll show you the Top "+ $@LadderLength +" with "+$@LadderSteps+" entries per page."; mes "It'll be viewed like this:"; mes @colour$[0]+"Place^000000: "+@colour$[1]+"<name>^000000 :"+@colour$[2]+"<kills>^000000:"+@colour$[3]+"<deaths>^000000 "+@colour$[4]+"<time>"; next; callsub L_Ladder; goto M_selection_eng; M_seitLogin_eng: mes @name$; mes "Your stats since your login:"; mes @PVPKillstoday+"/"+@PVPDeathstoday+"(Kills/Deaths)"; next; goto M_selection_eng; M_dieserChar_eng: mes @name$; mes "Your stats of this Char:"; mes PVPKills+"/"+PVPDeaths+"(Kills/Deaths)"; next; goto M_selection_eng; M_vomAccount_eng: mes @name$; mes "Your stats of the whole account:"; mes #PVPKillsAccount+"/"+#PVPDeathsAccount+"(Kills/Deaths)"; next; goto M_selection_eng; M_abbrechen_eng: mes @name$; mes "OK. You can come back to me and see your stats whenever you want."; close; L_reset: if (getgmlevel() < 99) end; mes "Do you want to reset the ladder?"; if (select ("Yes","No") == 2) goto L_end; mes "Are you really really sure you want to reset it?"; menu "Yes, ffs!!",-,"No",L_end; deletearray $terces_PVP_kills,128; deletearray $terces_PVP_deaths,128; deletearray $terces_PVP_names$,128; deletearray $terces_PVP_times,128; set $terces_PVP_resets,$terces_PVP_resets +1; L_end: mes "Okay...cya"; close; L_Ladder: for (set @y,0; @y < $@LadderLength; set @y,@y+$@LadderSteps){ for (set @x,@y; (@x < (@y+$@LadderSteps)) && (@x < ($@LadderLength)); set @x,@x+1){ if ($terces_PVP_names$[@x] != ""){ mes @colour$[0]+(@x+1)+"^000000: "+@colour$[1]+$terces_PVP_names$[@x]+"^000000 "+@colour$[2]+$terces_PVP_kills[@x]+"^000000:"+@colour$[3]+$terces_PVP_deaths[@x]+"^000000 ~ "+@colour$[4]+callfunc ("Gettime",$terces_PVP_times[@x])+"^000000"; } else { mes "^DD0000"+(@x+1)+": ^006699None^000000 "; } } next; } return; } function script Gettime { if (getarg(0)==0) return; set @difftimedays,(gettimetick(2) - getarg(0)); set @difftimehours,@difftimedays%86400; set @difftimeminutes,@difftimehours%3600; set @difftimeseconds,@difftimeminutes%60; set @days,@difftimedays/86400; set @hours,@difftimehours/3600; set @minutes,@difftimeminutes/60; set @seconds,@difftimeseconds; set @result$,""; if(@days != 0) set @result$,@result$+@days+"d "; if(@hours != 0) set @result$,@result$+@hours+"h "; if(@minutes != 0) set @result$,@result$+@minutes+"m "; if(@seconds != 0) set @result$,@result$+@seconds+"s"; return (@result$); }

Thanks @El Dragon. Solved.

Hi, as the title says, is there a way to do this?