HerculesWS / Hercules

Hercules is a collaborative software development project revolving around the creation of a robust massively multiplayer online role playing game (MMORPG) server package. Written in C, the program is very versatile and provides NPCs, warps and modifications. The project is jointly managed by a group of volunteers located around the world as well as a tremendous community providing QA and support. Hercules is a continuation of the original Athena project.
http://herc.ws
GNU General Public License v3.0
896 stars 753 forks source link

[RE] Wrong and missing item script list #980

Open zackdreaver opened 8 years ago

zackdreaver commented 8 years ago

I've found many wrong and missing renewal item script in herc, according kRO desc and 'Aegis' script :

There are still many wrong and missing item script that I haven't check. if you need english desc for related item, you can check translation I made : iteminfo.lua

Jedzkie commented 8 years ago

@zackdreaver some of the Item effects in renewal are same in pre-re, i'll include pre-renewal also.

zackdreaver commented 8 years ago
Jedzkie commented 8 years ago

@zackdreaver, please do give me info regarding ID#12221, i have 3 aegis special.sc's in my pc but i dont know where to choose the right one.

zackdreaver commented 8 years ago

wow 3 Aegis... I only have two, one is below 2013 (±770 KB) and newer 2014 (878 KB) here is aegis script for 12221 :

event OnConsume:
    var name = PcName
    dialog "Note: Talk into a megaphone to let Midgard hear your voice. ^FF0000Offensive content will not be tolerated.^000000. "
    dlgwritestr
    var str = inputstr
    broadcastserver "FFAF1B" FW_NORMAL 12 0 0 name+" talks into a megaphone : "+ str +""
    close
return
Jedzkie commented 8 years ago

is this also the megaphone's behavior in kRO?

zackdreaver commented 8 years ago

dunno, not kRO player, and iRO has removed this item long time ago, can't test

Jedzkie commented 8 years ago

Maybe we should disregard to change the Megaphone until we get kRO's info.

hemagx commented 8 years ago

How much official the NPC dialog is not matter, neither the megaphone syntax. this is rare case, anyway just use PlayerName: MegaPhoneTalk please define a color and don't let the color undefined, this lead to abuse-able bug in client.

zackdreaver commented 8 years ago

I've made translantion fix for these:

https://github.com/zackdreaver/ROenglishRE/commit/618e63b8c0bcab51e5e322badf4c323641efe124 I'll include all items in forum too as I promise

zackdreaver commented 8 years ago
Jedzkie commented 8 years ago

@zack, please include if PRE-RE or RE in every item thanks!

Jedzkie commented 8 years ago

@zackdreaver PRE-RE Item description for Rainbow Scarf, do you have one? I Prefered kRO, or Other servers but same as kRO's Description.

zackdreaver commented 8 years ago

Rainbow scarf is always 1% matk bonus, not matk +30 because of renewal (remember how staff matk bonus applied after renewal?)

Jedzkie commented 8 years ago

So, the pre-renewal effect of Rainbow Scarf in our repo is correct?

zackdreaver commented 8 years ago

Yes, based on item desc of pre-renewal twRO (i will follow prere desc from twro instead iRO classic from now)

zackdreaver commented 8 years ago
zackdreaver commented 8 years ago

kRO has changed cards 4684 to 4696 's script on Main server. Each of them has different % and skill bonus now.

What you add right now are based on sakray server