[Import DB is awesome...]

Why not just tell people to copy the file they want to change to said directory, and them have them change that file.

This way you can update all original files as much as you want, as long as the server would always  prioritize the import folder.

You can imagine that the vast majority of users doing this would never update their databases again. Definitely not what we want.

I was proposing making a way to nullfys an entry in the skill_tree.

...

An important check in all of the databases is the number of fields (columns) provided - with your suggestion, we'd have to remove that check.

I understand your intent, and agree that it'd be convenient, but I still can't agree with it. You're trading a handful couple of conflict edits for tens of thousands of extra checks/operations each time every database loads.

[matk and atk renewal]

Look for all the #ifdef RENEWAL in the src/map/ related to MATK and switch it with the Pre-Renewal code. It'll take a while, and there will almost definitely be a lot of balance issues afterwards...

[Import DB is awesome...]

To delete entries, you'd have to create a separate command for every database which would remove entries, and check for every entry whether it's being added or deleted. It's inefficient and not really worth the effort.

[Import DB is awesome...]

While being able to delete entries would be convenient, I feel that it's outside the scope of 'import' files.

[Falcon + Warg]

Marking as "approved", awaiting someone to code this.

[Emistry's rentitem2 be added to SVN!]

I'm not sure what a pull request is.

See the Pull Request Q&A.

[Emistry's rentitem2 be added to SVN!]

if someone make the pull request for it, I'll merge it.

What he said!

In general, if you want to add a feature that already exists, making a pull request will greatly increase the chances of us adding it (as well as speeding up the process).

[Class Specialization by Euphy]

hmm ... the idea of this script is nice ... especially the names of the job

I took the names from a (dead) server I used to play (and they're credited in my releases topic) XD.

And wow, that's an old script. One of the first ones I wrote, I think...

[Import DB is awesome...]

You can create your own files, if one isn't already there. Do you think it'd be beneficial to have empty files for all the databases, or could we simply do a better job documenting it?

[Reorganize scripts logic (Warps&Duplicates)]

Upon rereading this (and looking at the votes), I'm going to reject this proposal. I think the costs outweigh the benefits, and doing a large-scale reorganization makes updating official scripts far more difficult.

An easy way to enable/disable NPCs on maps would be to write a script command which returns the names of all NPCs on a map (similar to @mapinfo 2) and then looping through with 'enablenpc'/'disablenpc'.

[Replace hard coded statuses with scripted ones]

This was already implemented in the form of the 'bonus_script' command.

[New Cards 2013]

These items have already been added; any further comments should be posted in this topic.

[Lvl max]

This was completed by Cydh in 96443cd. If you have additional information (i.e. sources showing that our data is incorrect), please post a bug report.

[Euphy's Mvp Ladder Bug Npc Hides]

Eh... that was entirely a joke, I would never put such effort into a real script. ;D

 

In case you need one, here's the actual version of that script (as you can see, it's pretty bare):

prontera,148,170,6	script	MVP Ladder	891,{
	mes "[MVP Ladder]";
	mes "Hello!";
	mes "What are you doing here?";
	next;
	switch(select("Check rankings.:My stats.:Nothing...")) {
	case 1:
		mes "[MVP Ladder]";
		query_sql("SELECT char_id,CAST(`value` AS SIGNED) FROM `global_reg_value` WHERE `str` = 'MVP_Rank' ORDER BY CAST(`value` AS SIGNED) DESC LIMIT 20",.@cid,.@value);
		for(set .@i,0; .@i<getarraysize(.@cid); set .@i,.@i+1) {
			query_sql("SELECT `name` FROM `char` WHERE char_id = "+.@cid[.@i]+";",.@j$);
			set .@name$[.@i], .@j$;
		}
		if (!getarraysize(.@cid))
			mes "The rankings are empty.";
		else for(set .@i,0; .@i<getarraysize(.@cid); set .@i,.@i+1)
			mes "["+(.@i+1)+"] "+.@name$[.@i]+" ~ "+.@value[.@i]+" kills";
		close;
	case 2:
		mes "[MVP Ladder]";
		mes "You have killed "+((MVP_Rank)?"^0055FF"+MVP_Rank:"no")+"^000000 MVP"+((MVP_Rank == 1)?".":"s.");
		close;
	case 3:
		close;
	}

OnNPCKillEvent:
	if (getmonsterinfo(killedrid,22)) {
		set MVP_Rank, MVP_Rank+1;
		if (MVP_Rank == 1)
			query_sql("INSERT INTO `global_reg_value` (`char_id`,`str`,`value`,`type`,`account_id`) VALUES ("+getcharid(0)+",'MVP_Rank','1',3,0)");
		else
			query_sql("UPDATE `global_reg_value` SET `value` = '"+MVP_Rank+"' WHERE char_id = "+getcharid(0)+" AND `str` ='MVP_Rank'");
		dispbottom "~ You've killed "+MVP_Rank+" MVP"+((MVP_Rank == 1)?"":"s")+". ~";
		specialeffect2 313;
	}
	end;
}

[God Quests]

npc/quests/seals/

Read: http://irowiki.org/wiki/God_Items_Quest

[Euphy's Mvp Ladder Bug Npc Hides]

It's a joke, I suggest you don't use it. xD

[first classes quest disable]

There's no shortcut, you have to edit all the scripts manually.

[Private Message Help?]

Replace the ternary if with a plain 0.

[2 RO in 1 server]

Read: Multiple_Servers

More specifically, I'd guess you're referring to Multiple_Servers#Multiple_Map_Servers.

[Rare Drop]

That comment is actually outdated, which is why you couldn't find it. It's 541 in map_msg:

// Rare Items Drop/Steal announce
541: '%s' got %s's %s (chance: %0.02f%%)

[Revision 17694]

It was only there because the old formula was (way) incorrect. It works now, so there was no reason to leave the config there.

[Maybe its time to switch to this]

There are plenty of programs that already do this, for example: http://rathena.org/board/files/file/2252-rathena-database-editor/

GUIs will do a better job than any formatting of a plain-text file.

[Ticket Refiner]

It's already in the repository.

npc/re/merchants/ticket_refiner.txt

[Modifying base level...]

Moving to Server Support.

[Soul Breaker Bug?]

Please post in the Bug Tracker. Thanks!