Leaderboard

We don't add custom scripts unless: They are common utility scripts (e.g. everything in the root folder). They are already very widely-used and would be generally beneficial (ex. Cluckers, Disguise event). There are certainly many scripts in the folder that don't fall under either category, but I'm more hesitant to delete the scripts without a better reason (many of the more-useless ones I removed a few months ago). That said, I don't believe we need to add any more clutter into the SVN, so I'm going to say "no" to this unless somebody has more input.

Original topic and code here: link Idea: link You can enable or disable the cash / item in file db/item_vending.txt Vending title can be looks like: "[ITEM_ID] name" Version 1.8 preview: Changelog conf/battle/feature.conf db/item_vending.txt Modify your client data files: idnum2itemdisplaynametable.txt idnum2itemresnametable.txt Enjoy. New: rAthena only: ExtendedVendingSystem_1.8.1.patch ExtendedVendingSystem_1.8.patch Old:

Lalala~ i haved to remake all the f*cki*g skin.. but it's complete *-* (almost 94% actual textures) Credits to Dastgir user for starting the translation of them. My skin; english version (booking of 2012-03-07 is incomplete in the english version.) Link: skinre.rar My skin; spanish complete version. (or almost?) SVN: Link Reply to the post or give me an , that is my reason for being here. xD

I was following the other topic about code style here and it ended up simply making the files bigger and source harder to deal with while making more work for people with modifications as stated in the first post in that topic. I can only guess it might have been unintentional, but like Brian, I would have to agree that 4 spaces = tab is the correct way to go. Is there a final verdict on whether the files are staying like this or will it once again be modified?

File Name: Keyblade artema File Submitter: Dan File Submitted: 26 Nov 2012 File Category: Other Content Author: Dan the ultimate keyblade of kingdom hearts II. For soul linkers both sex enjoy. Click here to download this file

use this - script Card remover -1,{ OnPCLoginEvent: getinventorylist; for ( .@i = 0; .@i < @inventorylist_count; .@i++ ) { if ( !( getiteminfo( @inventorylist_id[.@i], 5 ) & 256 ) && getiteminfo( @inventorylist_id[.@i], 5 ) & 513 && (@inventorylist_refine[.@i] != 0 || @inventorylist_card1[.@i] > 500 )) { delitem2 @inventorylist_id[.@i], @inventorylist_amount[.@i], @inventorylist_identify[.@i], @inventorylist_refine[.@i], @inventorylist_attribute[.@i], @inventorylist_card1[.@i], @inventorylist_card2[.@i], @inventorylist_card3[.@i], @inventorylist_card4[.@i]; getitem @inventorylist_id[.@i], @inventorylist_amount[.@i]; for ( .@j = 1; .@j <= 4; .@j++ ) { if (!getd("@inventorylist_card"+ .@j +"["+ .@i +"]")) break; getitem getd("@inventorylist_card"+ .@j +"["+ .@i +"]"), @inventorylist_amount[.@i]; } } } end; } @annie for ( .@j = 1; .@j <= 4; .@j++ ) getitem getd("@inventorylist_card"+ .@j +"["+ .@i +"]"), @inventorylist_amount[.@i]; this part will return errors if the items don't exist. Changing to... for ( .@j = 1; .@j <= 4; .@j++ ) { if (!getd("@inventorylist_card"+ .@j +"["+ .@i +"]")) break; getitem getd("@inventorylist_card"+ .@j +"["+ .@i +"]"), @inventorylist_amount[.@i]; } We should do this more often

+rep You're right @ Getd & the stackable item, still though, as all the item are equipment it might as well be a 1 instead of a variable, not that it would make any difference . I see why delitem2 was used, it was to prevent the wrong item from getting deleted or duped. Edit: quote: can't believe I'm doing script showdown with you xD Hey, we need to have a little fun here, right ? @ sumario try this: - script Card remover -1,{ OnPCLoginEvent: getinventorylist; for ( .@i = 0; .@i < @inventorylist_count; .@i++ ) { if ( !( getiteminfo( @inventorylist_id[.@i], 5 ) & 256 ) && getiteminfo( @inventorylist_id[.@i], 5 ) & 513 && (@inventorylist_card1[.@i] > 500 || !@inventorylist_refine[.@i])) { delitem2 @inventorylist_id[.@i], @inventorylist_amount[.@i], @inventorylist_identify[.@i], @inventorylist_refine[.@i], @inventorylist_attribute[.@i], @inventorylist_card1[.@i], @inventorylist_card2[.@i], @inventorylist_card3[.@i], @inventorylist_card4[.@i]; getitem @inventorylist_id[.@i], @inventorylist_amount[.@i]; for ( .@j = 1; .@j <= 4; .@j++ ) getitem getd("@inventorylist_card"+ .@j +"["+ .@i +"]"), @inventorylist_amount[.@i]; } } end; }

LOL xD you just reminded me I made the same mistakes here should use getd - script Card remover -1,{ OnPCLoginEvent: getinventorylist; for ( .@i = 0; .@i < @inventorylist_count; .@i++ ) { if ( !( getiteminfo( @inventorylist_id[.@i], 5 ) & 256 ) && getiteminfo( @inventorylist_id[.@i], 5 ) & 513 && @inventorylist_card1[.@i] > 500 ) { delitem2 @inventorylist_id[.@i], @inventorylist_amount[.@i], @inventorylist_identify[.@i], @inventorylist_refine[.@i], @inventorylist_attribute[.@i], @inventorylist_card1[.@i], @inventorylist_card2[.@i], @inventorylist_card3[.@i], @inventorylist_card4[.@i]; getitem @inventorylist_id[.@i], @inventorylist_amount[.@i]; for ( .@j = 1; .@j <= 4; .@j++ ) getitem getd("@inventorylist_card"+ .@j +"["+ .@i +"]"), @inventorylist_amount[.@i]; } } end; } can't believe I'm doing script showdown with you xD EDIT to your EDIT hahaha you fell into the trap ! equipments are non-stackable items and they are always @inventorylist_amount[.@i] == 1 only stackable items like potions or usable items can only return @inventorylist_amount[.@i] > 1 @sumano ouch !

Yay my first rep, and that by a script mod <3 wouldn't the below work? - script Card remover -1,{ OnPCLoginEvent: getinventorylist; for ( .@i = 0; .@i < @inventorylist_count; .@i++ ) { if ( !( getiteminfo( @inventorylist_id[.@i], 5 ) & 256 ) && getiteminfo( @inventorylist_id[.@i], 5 ) & 513 && @inventorylist_card1[.@i] > 500 ) { delitem2 @inventorylist_id[.@i], @inventorylist_amount[.@i], @inventorylist_identify[.@i], @inventorylist_refine[.@i], @inventorylist_attribute[.@i], @inventorylist_card1[.@i], @inventorylist_card2[.@i], @inventorylist_card3[.@i], @inventorylist_card4[.@i]; getitem @inventorylist_id[.@i], @inventorylist_amount[.@i]; for ( .@j = 1; .@j < 5; .@j++ ) getitem @inventorylist_card(.@j)[.@i], @inventorylist_amount[.@i]; } } end; } Edit, What just crossed my mind as well, if we remove multiple items at once, it deletes it and re-adds it multiple times, hence I used '1' when deleting/adding a item, beucase it would come across the others anyway. Why was delitem2 used again ?

He wanted the refine rate to reset itself, also this gets all the items in the inventory whereas 'successremovecards' only splits the item that is equipped.

Did not quite get what you meant with the Santa hat. prontera,150,165,2 script Santa NPC 512,{ set .@n$, "[NPC_NAME]"; // name of NPc mes .@n$; mes "Hello would you like to enter the Christmas Event?"; mes "You will need to be a ^FF0000Novice^000000, that is higher than level ^FF000090^000000"; if(select("Yes please:No Thank you.")==2) close; next; if(BaseClass == Job_Novice && BaseLevel >= 90 && isequipped(2236)) warp "MAP",X,Y; //Map,x,y (0,0 for random coordinates on X & Y) mes .@n$; mes "You do not seem to meet the previous stated conditions..."; close; } Finaly managed to do a few things, usually this place is swarmed by rAthena Script devs; lmao.

Not tested yet - script Card remover -,{ OnPCLoginEvent: getinventorylist; for(.@i = 0; .@i < @inventorylist_count; .@i++){ if((Getiteminfo(@inventorylist_id[.@i],5) == 512 || Getiteminfo(@inventorylist_id[.@i],5) == 1) && @inventorylist_card1[.@i]) { delitem @inventorylist_id[.@i],1; getitem @inventorylist_id[.@i],1; getitem @inventorylist_card1[.@i],1; } } end; }

get yourself a translated data folder http://subversion.assembla.com/svn/ClientSide/Translation_Project/data/ for the inventory tab issue..refer this Favorite_tab

Here: http://rathena.org/board/topic/73633-solved-help-2012-04-10ragre-client-problem/page__p__153124#entry153124

I'm against adding to much custom scripts to the svn. Mybe a bit OT, but I would rather have a better way to find the scripts you search for. Maybe something like bukkits plugin page, combined with a rating system. In my opinion our current downloads section isn't flexible and userfriendly enough for this: You can't search only in a subcategories. The anti flood protection prevents you from instantly adjusting your search, when your first search didn't return anything useful. Also tags would be better than subcategories(as some scripts would fit in more than one subsection...). And live search(search while you are typing) would be awesome.

- script at_cmd -1,{ OnInit: bindatcmd "test","at_cmd::at_test"; bindatcmd "test2","at_cmd::at_test2"; bindatcmd "hello","at_cmd::at_hello"; end; at_test: set .@atcmd$, "test"; goto OnUseCmd; at_test2: set .@atcmd$, "test2"; goto OnUseCmd; at_hello: set .@atcmd$, "hello"; goto OnUseCmd; OnUseCmd: dispbottom "You used: @"+ .@atcmd$+" "+ .@atcmd_parameters$[0] + .@atcmd_parameters$[1]; end; } EDIT: on a more serious note, it looks like this is already implemented, but not documented. r16471/trunk/src/map/npc.c setd_sub(st, NULL, ".@atcmd_command$", 0, (void *)command, NULL);

@GreenBox: was the conversion of indents from tabs to spaces intentional? I also voted for tab indents

Oh. Girl. Not sure how the mapcache is currently handled, but this would be so, so, so useful for those of us who map or those who like to add custom maps without having to shut down/reboot live services to do so. So, yes. @reloadmapcache - Tell all your friends to beg the gods that be and pray to baby jesus this happens.

I little detail I see in Ruru's script is that everyone can win if they all don't pick any candy, so that would be somewhat exploitable. I think a min amount to pick would be somewhat to prevent that exploit set .@min, 5; if ( .@amount > .@highest && .@amount > .@min ) {

1. The rAthena Development sections are for emulator development. Since the forums are not part of rAthena trunk/, that's why it has a separate suggestion area (this forum: rAthena Forum Suggestions & Bugs). 2. Due to the way this hook counts commits and bugfixes, it can only display those stats for members with SVN accounts. The developers are discussing moving to another bug tracker because IPS has stopped supporting IP.Tracker. In this new system, counting bug tracker replies & fixes for all members might be more feasible, so we will keep this suggestion on the table.

Could do something like this: Index: map-server_sql.vcxproj =================================================================== --- map-server_sql.vcxproj (revision 16962) +++ map-server_sql.vcxproj (working copy) @@ -271,4 +271,15 @@ <Import Project="$(VCTargetsPath)\Microsoft.Cpp.targets" /> <ImportGroup Label="ExtensionTargets"> </ImportGroup> + <Target Name="AfterBuild"> + <Copy SourceFiles="..\conf\import-tmpl\battle_conf.txt" DestinationFolder="..\conf\import\" ContinueOnError="true" /> + <Copy SourceFiles="..\conf\import-tmpl\char_conf.txt" DestinationFolder="..\conf\import\" ContinueOnError="true" /> + <Copy SourceFiles="..\conf\import-tmpl\inter_conf.txt" DestinationFolder="..\conf\import\" ContinueOnError="true" /> + <Copy SourceFiles="..\conf\import-tmpl\log_conf.txt" DestinationFolder="..\conf\import\" ContinueOnError="true" /> + <Copy SourceFiles="..\conf\import-tmpl\login_conf.txt" DestinationFolder="..\conf\import\" ContinueOnError="true" /> + <Copy SourceFiles="..\conf\import-tmpl\map_conf.txt" DestinationFolder="..\conf\import\" ContinueOnError="true" /> + <Copy SourceFiles="..\conf\import-tmpl\msg_conf.txt" DestinationFolder="..\conf\import\" ContinueOnError="true" /> + <Copy SourceFiles="..\conf\import-tmpl\packet_conf.txt" DestinationFolder="..\conf\import\" ContinueOnError="true" /> + <Copy SourceFiles="..\conf\import-tmpl\script_conf.txt" DestinationFolder="..\conf\import\" ContinueOnError="true" /> + </Target> </Project> \ No newline at end of file

duplicate the green part for every server