UniEpic Armor Merchant

[deathwish]

UniEpic Armor Merchant - By Deathwish

Epic Armor merchant that selects the clients Class and sends out the classes Epic Armor in a trade window.
UniEpicNPC.cs = GameNPC
EpicArmor MerchantItems.sql = Installs all the merchants items needed.
Items are already installed in dolpubdbV3.0.

[geshi]

Nice.. now we just need updated Epic Armor... since the current stuff in the public db is the really old stuff afaik..

Argain updated it all, dunno if the owner is still around and can contribute :/

[deathwish]

Hmm, i never checked to see if it was uptoday i must say, when i got some more free item, i will check it out and update them, if needed

[deathwish]

Do you want the same thing added for Champ weapons too?

[stephenxpimentel]

@Geshi - i already updated epic armor and posted it here

Edit: Didnt post them in user files, but should be in public DB.

Page 3.
http://www.dolserver.net/viewtopic.php? ... 8&start=30

[svartson]

I have a problem with this script. I created an UniEpicNPC and it gives the items to all classes except for those whose names change into suffix "ress", like enchant(e)ress, blademistress, sorceress.

Is there any fix for that?

I am not yet familiar with the objects/methods in DOL but i am searching for something like this (pseudocode):

Code: Select all

if player.gender is female { change dbquery string to female name }

or I need some explanation where a potential female class name can be reached.

[Tolakram]

Actual code will be very close.

if (player.Gender == eGender.Female)



^^ I think, without actually looking it up.

[svartson]

Yeah, but what other values than .Name can player.ClassName have? player.CharacterClass.Id or ClassId or what?

[svartson]

I changed the script so that players need to be level 50 to get it and it also fixed the female classes, although this is a very dirty workaround, I want to share this:

Code: Select all

public override bool Interact(GamePlayer player) { if (player.Level == 50) { if (player.CharacterClass.Name == "Enchantress") { TradeItems = new MerchantTradeItems("" + "Enchanter" + "Epic"); } else if (player.CharacterClass.Name == "Sorceress") { TradeItems = new MerchantTradeItems("" + "Sorcerer" + "Epic"); } else if (player.CharacterClass.Name == "Armswoman") { TradeItems = new MerchantTradeItems("" + "Armsman" + "Epic"); } else if (player.CharacterClass.Name == "Blademistress") { TradeItems = new MerchantTradeItems("" + "Blademaster" + "Epic"); } else if (player.CharacterClass.Name == "Heroine") { TradeItems = new MerchantTradeItems("" + "Hero" + "Epic"); } else if (player.CharacterClass.Name == "Huntress") { TradeItems = new MerchantTradeItems("" + "Hunter" + "Epic"); } else { TradeItems = new MerchantTradeItems("" + player.CharacterClass.Name + "Epic"); } player.Out.SendMerchantWindow(TradeItems, eMerchantWindowType.Bp); } else { SayTo(player, "You must be level 50 to be eligibile for your free Epic Armor!"); } return true;

[Tolakram]

That works, and thanks for sharing your changes!

[svartson]

As I said: If I have the time to read through the important parts of the DOL core, we can rewrite this part properly. The classnames like "Sorceress" must be saved somewhere in there (obviously they are displayed ingame) and if this is true we can just use the attributes and insert it into this patch (method will consist of max. 5 lines afterwards).

[Tolakram]

That part of the name is also local language (stored as part of language support), which is why you can't find it. Using classid plus gender is probably a better way.

[svartson]

Another question: RunemasterEpicVest is missing in the merchantwindow. The item exists in db and can be created with

Code: Select all

/item create RunemasterEpicVest

The item has the same stats in db like the other RunemasterEpic items. Where is the mistake? O.o