Xynvaroth

Please post a clear and complete question. I do not think people will understand what you actually mean with this.

That might be easier to understand, but the downside is that you will have as many scripts additionally running as the amount of players who are currently online is.

These questions do not fit the original subject, so you will have to open two new topics in the proper areas. None of these questions belong to the scripting support area.

You mean announcing "Go!" instead of "Start in 0 seconds..."? for ( .@i = 5; .@i >= 0; .@i-- ){// mapannounce the count 5 4 3 sleep 1000; mapannounce "guild_vs2", .@i == 0 ? "Go!" : "Start in "+ .@i +" seconds..",0; }

To answer your question directly, it will change the value of the .@i variable to .@i + 1. In plain language, it will increase the value of .@i by one. You should, however, make sure that you also understand the for( ) statement properly in general. I am saying this due to the fact that if you have learned how for( ) works, you normally also know what this statement does.

Like already discussed in this area, you should use the monster command and an additional label to be triggered when the monster dies if you only want to trigger the death of a few certain monsters. OnNPCKillEvent should only be used if you want a general label which is triggered whenever any monster on the server is killed. You could want to use this, for instance, if you want to count all the monsters a player kills, but not if you want to trigger the death of certain, few monsters. The resource difference might not be that noticable for all cases, since we are talking about a very short time duration, but it still exists and if you run many unoptimised scripts, you will eventually notice a different.

I am not sure whether I understood your demand properly. Nevertheless - if any guild master logs out, this will reset the cooldown for his or her guild: OnPCLogoutEvent: if( getcharid( 2 ) != 0 && getguildmasterid( getcharid( 2 ) ) == getcharid( 0 ) && getd( ".guild" + getcharid( 2 ) ) != 0 ) setd ".guild" + getcharid( 2 ), 0; end;

In general, the clone( ) command will return the monster unit id which can be used to later kill the clone using unitkill( ). That said, you will have to somehow store all the clones' ids which are spawned and as soon as a player enters a gvg map, kill the clones of that player. I would recommend you using my Xynvaroth_Array script, because you can create permanent character arrays with it. Everytime someone uses a clone pot, add the clone's id to the array. Then, when a player enters a gvg map - you can use the OnMapLoad event to know when a player enters such a map -, go through the array, kill the clones and reset the array using Xynvaroth_Array_Reset( ). You might face two problems with this solution: 1. The array will contain ids of clones which are already dead, because most clones will have a limited living duration. 2. The array will grow and grow when a player uses the potions and never goes on a gvg map. To solve these issues, you could, for example, always when a player uses a potion and before you save the clone's id, go through all the ids in the array and delete the ids of clones which are already dead using Xynvaroth_Delete( ).

Thank you!

Xynvaroth's Scripts Utility Xynvaroth_Array This script package contains functions which can be used to define a Xynvaroth_Array which allows you to also create character and account variables and it will significantly increase the index limit (which currently is 128). Increasing the index limit is very useful if you would like to store much data using server variables instead of MySQL for performance reasons. Besides, you can also create associative arrays (arrays with string indices) using Xynvaroth_Array. However, you will have to care about how to delete them yourself, because Xynvaroth_Array_Delete( ) can not be used to delete an associative Xynvaroth_Array. Important: Xynvaroth_Array will - due to limitations of the rAthena scripting engine - start with a number one index and not zero. Xynvaroth_CurrencyManager This function package provides several functions to manage (custom) currencies. Xynvaroth_CurrencyManager_Choose( ) lets a player choose currencies and their amounts. In addition, there are the functions Xynvaroth_CurrencyManager_CheckTake( ), Xynvaroth_CurrencyManager_Take( ), Xynvaroth_CurrencyManager_CheckGive( ) and Xynvaroth_CurrencyManager_Give( ) available. Xynvaroth_DynamicSelect This function is passed an array and it will then generate a string to be used as a parameter of select or prompt with the possibility of selecting each value of this array. Xynvaroth_ItemManager This function package provides several functions to manage items. Xynvaroth_ItemManager_Choose( ) lets a player choose items and their amounts. The function is very configurable! From selecting only one item securely to selecting a whole list of items - there are detailed options available to fit any need. In addition, there are the functions Xynvaroth_ItemManager_CheckTake( ), Xynvaroth_ItemManager_Take( ), Xynvaroth_ItemManager_CheckGive( ) and Xynvaroth_ItemManager_Give( ) available. Xynvaroth_TextManager TextManager is a script package used to separate texts from scripts with an optional multilangual support. You will never have to touch a script to change the texts. If you have staff members which should only translate or correct texts, they will only have to receive and change the data area of the Xynvaroth_TextManager. There is no need to spread all your scripts anymore.

This script package contains functions which can be used to define a Xynvaroth_Array. A Xynvaroth_Array allows you to also create character and account variables and compared to the normal rAthena arrays, the index limit (which currently is 128) is significantly increased. This is very useful if you would like to store much data using server variables instead of MySQL for performance reasons. Besides, you can also create associative arrays (arrays with string indices) using Xynvaroth_Array. However, you will have to care about how to delete them yourself, because Xynvaroth_Array_Delete( ) can not be used to delete an associative Xynvaroth_Array. Important: Xynvaroth_Array will - due to limitations of the rAthena scripting engine - start with a number one index and not zero. There is a detailed documentation inside the file. Xynvaroth_Array

You can not generalise that it will not work, because it will depend on how the other part is realised.If you use a variable and increase it by one everytime one of these events is run, you will be able to see how long they actually stayed online. However, that is just a rough plan; there will be needed more work to be done to ensure they do not log in every 15 minutes and so on. To prevent losing the RID, you could either use the OnMinuteXX labels I have suggested, or you use the sleep2 x; function which will pause the script for x seconds while keeping the RID attached.

As far as I can see, the script should work like this.That said, for further debugging, try this - script load_logout -1,{ OnPClogoutEvent: debugmes "reset$ = " + reset$; if(reset$ == "on") end; savepoint "prontera",156,179; atcommand "@return"; }If you implement this, you should be able to see the actual value of reset$ in a debug message from your map-server when you log out.In general, if you want to save a boolean state (on - off; true - false), it is better (only in rAthena, since rAthena's scripting engine does not feature boolean variables) to use an integer variable and set it to 0 or 1, because it is always good to keep the information saved as small as neccessary (to save strings like "on" and "off", more bytes are needed than to save integers with only one digit like 0 and 1).

Normally, you would not think about using OnNPCKillEvent in this context, because its clearly that only a single MVP is to be triggered: The problem is not that he is using killedrid; the problem is that he is using OnNPCKillEvent in the first place. What you have suggested has nothing to do with 2essy2killu's solution. You are using what I suggested - a single callback for the certain monster. Therefore, it can not be correct that , as your script is not even based on 2essy2killu's. Furthermore, the main problem with 2essy2killu's script is still that he is using the general OnNPCKillEvent which will be called whenever any monster out of all those monster who are in the whole server is killed. And thus, I think my objection is definitely reasonable, since using 2essy2killu's solution would be just a waste of performance that could easily be prevented by using another code. I hoper things are clearer now.

I do not think you two actually understod the explanations I made. rosfus obviously wanted to trigger the kill of a certain mvp. Now why should I suggest him to use OnNPCKillEvent: This special label triggers when a player kills a monster. The variable 'killedrid' is set to the Class of the monster killed.which will be called every time any player kills any monster while there is the possibility of only trigger the kill of the mvp?This has nothing to do with that I am going hard on anyone; it is just a waste of resources - uncountable calls of the event per minute on an active server - and not the common solution if you use OnNPCKillEvent, even though you only want to trigger the kill of one certain monster.

Evertime any player kills any mob, the code below the label is executed. Even if there is a if, it will always run this if whenever any monster is killed on the server. That said, it is a major resource eater. And you did correct my script wrongly. You can chose a - as the npc sprite; this is not incorrect.

2essy2killu's solution is not a good solution at all, because it will cost much resources due to the fact that the event is always called when any player kills any mob. I strictly recommend you to not use this code. This is not a command way to trigger the kill of a single mob. To make my script work, you will of course also have to fill in the place holders, for example, replacing <mapname> with prontera, the %TAB% place holders with actual tabulators, and so on. I just copied a generic code from script_commands.txt.

1. Delete the arrays you will use for query_sql( ). 2. Use query_sql( ) to get the datasets (order the result by `Points`). 3. Loop through the datasets using for( ). 4. For each dataset, select the character names from the `char` table and display both the names and the points. This will only work if you have less than 128 entries in the top list. See this suggestion if you want more than 128 entries.

<map name>,<x>,<y>,<xs>,<ys>%TAB%monster%TAB%<monster name>%TAB%<mob id>,<amount>,<delay1>,<delay2>,"MVPTrigger::OnKill"{,<mob size>,<mob ai>} - script MVPTrigger -,{ OnKill: // Give the reward to the player. end; }

The warnings are displayed, because integer values are put into string variables and the commands used in this script require integer values. Delete all the dollar symbols behind the variable names. For example, change set .@item$,501;toset .@item,501;

You should really post all those topics in which you ask someone to code a whole script for you into the Script Requests area. This area is just in case you already have a script and a certain problem with it.

I think you are making this easier when you use OnMinute00: OnMinute15: OnMinute30: OnMinute45:The code below these labels will run each 15 minutes.Try revising your script using these labels and if your script still does not work properly, please also post the error messages displayed by your map server the next time.

You're not using the getitem( ) function properly. For example, changegetitem 13758,1,969,20,1108,1,2104,1,4043,4;togetitem 13758,1; getitem 969,20; getitem 1108,1; getitem 2104,1; getitem 4043,4;

2.0 - Removed Option_EnableCurrencyCheck and added Xynvaroth_CurrencyChooser_Check( ) instead. - Removed Option_EnableCurrencyTake and added Xynvaroth_CurrencyChooser_Take( ) instead.

3.0 - Removed Option_EnableItemCheck and added Xynvaroth_ItemChooser_Check( ) instead. - Removed Option_EnableItemTake and added Xynvaroth_ItemChooser_Take( ) instead.