Fix for merchant price bugs

[trevius]

Sweet! I will definitely try to get this added to my unofficial updates for the 1110e release. Of course, I will make sure to run some tests on it before posting the new version. Looks like we might have a new coder out to help fix some issues I will test out your LoS agro changes too and see how it works. That one seems complex to fix since you don't want things agroing through walls in separate rooms. If it seems pretty good, I will try to get that one added as well.

[ChaosSlayer]

greggg, do you know if this will also affect the "salerate" field in "items" table of the DB, which suposely should control... SOMETHING price related?

if yes, then how?

[greggg230]

Originally Posted by ChaosSlayer

greggg, do you know if this will also affect the "salerate" field in "items" table of the DB, which suposely should control... SOMETHING price related?

if yes, then how?

From everything I can tell, it does nothing. It's probably an obsolete variable at this point. It's not clear what it might have done in the past. Unless I'm missing something faction, charisma, and the base price of an item should be the only variables needed to determine actual price.

Fine Steel Short Sword has a value of 5500, and I was able to accurately predict its sell and buy prices without even knowing about sellrate. Its sellrate is 9, apparently.

Someone please correct me if I'm wrong, though.

[ChaosSlayer]

i was under impression, that 'salerate' was some sort of an option to set a custom sell back for each specific item - could realy be usefull in order to create items which would sell back for 100% base price (i can think for dozen fo reason for having such items) - or at least its the only logical explation that coudl posibly fit the description of the field name.

of course the thing never worked, so... =)

but it would be nice to have it actualy working this way =)
(could also be interesting to have item which would sell back for ABOVE base price - again for various reasons) =)

[cavedude]

Moving to submission. To make it easier on KLS/Scorp, could you diff your changes?

[greggg230]

Originally Posted by cavedude

Moving to submission. To make it easier on KLS/Scorp, could you diff your changes?

Sure.

Line 3717 of zone/client_packet.cpp should be changed from:

mco->rate = 1.6;

to

mco->rate = 1/(.884*Client::CalcPriceMod(tmp,true));

line 3842 of zone/client_packet.cpp should be changed from:

mpo->price = (item->Price*127/100)*mp->quantity;

to

mpo->price = (item->Price*(1/.884)*Client::CalcPriceMod(tmp,false))*mp->quantity;

line 3947 of zone/client_packet.cpp should be changed from:

price=(int)((item->Price*mp->quantity)*.884);

to

price=(int)((item->Price*mp->quantity)*.884*Client::CalcPriceMod(vendor,true)+0.5); // need to round, because client does it automatically when displaying price

line 4001 of zone/client_packet.cpp should be changed from:

inst2->SetPrice(item->Price*127/100);

to

inst2->SetPrice(item->Price*(1/.884)*Client::CalcPriceMod(vendor,false));

in zone/client_process.cpp, the following should be moved from 878 to 804:

Mob* merch = entity_list.GetMobByNpcTypeID(npcid);

line 831 of zone/client_process.cpp should be changed from:

inst->SetPrice(item->Price*127/100);

to

inst->SetPrice((item->Price*(1/.884)*Client::CalcPriceMod(merch,false)));

line 862 of zone/client_process.cpp should be changed from:

inst->SetPrice(item->Price*127/100);

to

inst->SetPrice((item->Price*(1/.884)*Client::CalcPriceMod(merch,false)));

[Aramid]

Originally Posted by cavedude

Moving to submission. To make it easier on KLS/Scorp, could you diff your changes?

CD, for those of us who used to be WinDoze users, can you tell us what the correct Syntax is to create a diff file?

Thanks

[So_1337]

Another great fix. Keep up the great work, Greg! =)

[Angelox]

Originally Posted by Aramid

CD, for those of us who used to be WinDoze users, can you tell us what the correct Syntax is to create a diff file?

Thanks

In Linux, diff is integrated in the os. You can make diffs/patches via command line. There are also some very nice graphical interfaces you can add, that help you see even better (kompare, xxdiff).
When I first started here at EqEmu, I couldn't for the best of me, get the source to compile under my windows box. So I took it to my Linux box where I had little trouble there - it was better this happened to me, because Linux Distros usually come "developer ready" and have all these neat tools like diff, kompare, compilers, etc. They are all just as available as my "dir" or "copy" command in windows.
I think I'm gonna try windows compile again with that new compiler package that's posted here, just for the challenge.
Anyways, you usually can google what you need for windows - I just googled "diff program for windows xp", and came up with a whole swarm of stuff. The diff toolshould be self explanatory on how you present it here.