DeadlySilence

Replace line 10 with this: mes "Please remove your"+((checkfalcon())?"falcon":"")+((checkcart())?"cart":"")+((checkriding())?"Peco":""))+"before proceeding.";

Autotrade requires the autotrade mapflag. So, you can only use it on maps where the mapflag is set //edit: Meh, too slow :<

You should show your script so we can look at possible error sources. The error itself seems to be very obvious: you use a non-existing variable for somehing.

Oh, and I noticed you use a temporary character variable. This won't work as there seems to be no player attached to the script. You should use .donateitems rather than @donateitems.

You could try to put this part setarray @donateitems[0],21849,21850,21851,21852,21267,21268,21269,21270,21271,21272,21273,21274,21275,21276,21277,21278,1316,1588,1932,1987,1833,1700,13435,1500,1600,13080,13079; under a OnInit: label like this. OnInit: setarray @donateitems[0],21849,21850,21851,21852,21267,21268,21269,21270,21271,21272,21273,21274,21275,21276,21277,21278,1316,1588,1932,1987,1833,1700,13435,1500,1600,13080,13079; end; I haven't tested it, but it might work.

You cannot use number formats other than integer, unfortunately =/ But if you use Emistry's way, you select the value as a string, which is not as limited as an integer.

Did the map server show any error? //edit: You had a quotation mark at the end of the I_Pink label, and a comma was missing there, too. Replace line 29 with this. menu "Green Bloomy Wing",I_Green,"Yellow Bloomy Wings",I_Yellow,"White Bloomy Wings",I_White,"Dark Bloomy Wings",I_Dark,"Saikos Bloomy Wings",I_Saikos,"Metallish Bloomy Wings",I_Metallish,"Blue Bloomy Wings",I_Blue,"Purple Bloomy Wings",I_Purple,"Pink Bloomy Wings",I_Pink,"Pure White Bloomy Wings",I_Pure,"Lemon Bloomy Wings",I_Lemon,"Solar Bloomy Wings",I_Solar,"Return to Menu",H_top;

Update: I have fixed the problem with spaces in names and added shortcuts for the longer commands. The new commands are @bn, @info, @deposit, @withdrawl, @transaction, @password. Usage: @bn: same as @bank @info : will display information about the bank and your account @deposit <amount> : deposits given amount @withdraw <amount> : withdraws given amount @transaction <username/id> <amount> : transfers given amount to give username's/id's account @password [<new password>] : changes current bank password to new password, new password is optional and the password will be reset if no new password was given End of update log. @Stolao: That's a good idea. I'll have a look at this this weekend, maybe. I think it will require a source modification, unless every merchant NPC gets updated manually to support the bank, though.

Thank you for your feedback. I'll have a look at the problem with the spaces and add shorter commands, too.

Yes, the direct download links have been added about half an hour ago. But the link to the download section (including version 2.0) have been there for 2 days now. I gues they're inaccessible because they're not approved yet.

Meh, that's probably because the new version is yet to be approved. I'll attach both versions and the other files to the first post. //edit: Sorry for the inconvenience, the scripts are now attached to the first post. Please let me know if there are still any problems.

The link works for me

The problem is the name. The string 'Lope's Clue' is interrupted by the apostrophe after Lope. You have to write the string like this: 'Lope\'s Clue'

mes "You still need to kill " + quincykill - quincy2 + " Whispers!"; This should work. If you put the mathematic symbol between quotes, it is interpreted as a string rather than a mathematic function.

You have to add it to every getarg in every compare. For example the lines 13 and 14: // line 13 for(set .e,0; !compare(getarg(.e+1),"Zeny"); set .e,.e+2) {} => for(set .e,0; !compare(getarg(.e+1) + "","Zeny"); set .e,.e+2) {} // line 14 npcshopadditem "EV_VSET",getarg(.e+2),getarg(.e)*compare(getarg(.e+1),"SZeny"); => npcshopadditem "EV_VSET",getarg(.e+2),getarg(.e)*compare(getarg(.e+1) + "","SZeny");

As the debug messages say: the compare function expects a string, but you gave it a number. To convert a number to string, just use this: .@yourNumberVar + "" By adding an empty string to it (""), it will be intrepreted as a string.

Currently I'm working on some optimization and a @bank command with which every function can be used without talking to the NPC. (As of last night, this feature is almost done, only logs are still in progress. It's likely that I will finish this tonight.) A loan system sounds good, but how could the payback be forced to prevent a player from loaning 1,000,000,000 Zeny without paying them back? I finally found some time to finish version 2.0. It now contains a fully functional @bank command with which every bank action can be done from afar. Thanks to Anakid for this suggestion.

status->cri += 10 + (status->luk*10/3); //(every 1 luk = +0.3 critical) Means you gain 10/3 crit for every point of luck. Let's assume you have 9 luck, sou you'd gain 3 crit. If you derease the crit per luck: status->cri += 10 + (status->luk*10/5); Means you get 1 crit every 5 luck, which equals in 1.8 additional crit with our assumed 9 luck. You have to use basic math to change it to a value you'd find more satifying. //edit: Basically, the last number in this calculation determines how many points of luck you need for 1 crit.

Change the 30 to 32, as 32 is 80% of 40.

Since this is a database question it should be written in the appropriate category. To add a bonus to an item you can look at this file for every possible bonus. http://svn.code.sf.net/p/rathena/svn/trunk/doc/item_bonus.txt

How about using my script then? It should do exactly what you want.

Go ingame, type "@version", check if it is greater than "r16114". If no, update your server with "svn up" (via putty) and restart.

prontera,155,150,0 script Test 123,{ // end the script in case someone talks to the NPC end; OnInit: // set the chat's name's characters setarray .chars$, "V", "E", "N", "D", "I", "N", "G"; // infinity loop, yay while (1) { // loop through all the characters for (.@i = 0; .@i < getarraysize(.chars$); .@i += 1) { // reset name on first run or after a complete run if (0 == .@i) { .@chatName$ = ""; } .@chatName$ = .@chatName$ + .chars$[.@i]; waitingroom(.@chatName$, 0); sleep(1000); delwaitingroom; } } }

Search for status->cri += 10 + (status->luk*10/3); //(every 1 luk = +0.3 critical) in src/map/status.c and modify it to your needs.

As far as I know you either have to put your main menu into a function or into a label. A function would be better as labels are not very good...