Jump to content

  •  

Bug Tracker Migration

June 3rd
Good news everyone! The staff has decided that it is time to slowly kill off this Bug Tracker. We will begin the process of slowly migrating from this Bug Tracker over to our Github Issues which can be found here: https://github.com/HerculesWS/Hercules/issues

Over the next couple of days, I will be closing off any opportunity to create new reports. However, I still will keep the opportunity to reply to existing Bug Reports. Doing this will allow us to slowly fix any bug reports we have listed here so that we can easily migrate over to our Issue Tracker.

Update - June 7th 2015: Creating new bug posts has been disabled. Please use our https://github.com/HerculesWS/Hercules/issues tracker to post bugs. Users are still able to reply to existing bug posts.

- Administration

Issue Information

Issue Confirmations

  • Yes (0)No (0)
Photo

Mvp Exp Reward Screwup

Posted by Hercules Bot on 23 March 2008 - 11:23 PM

Originally posted by theultramage
http://www.eathena.w...er&showbug=1259

A random test of MVP rewards on aegis showed that eathena has messed up mvp exp reward logic.

Aegis has only a single column for the exp reward, called 'exppercent'. Some wise guy interpreted that as 'chance to get exp reward', and also added another column to specify the value received.

Testing with various values  showed that this is not the case. The column specifies the percentage of the normal base exp kill reward. For all of the mvps, this is set to 50% in that table. Also, the exp reward is given every time, and regardless of the percentage setting.

One fix would be to make eathena ignore the value setting, and change the semantics of the percentage setting. And get rid of the mvp_exp config settings.
Another would be to rewrite all percentage values to 100% and keep the rest as it is.

Originally posted by Gepard
Fixed in [rev=15531].

Value of MVP exp reward has been updated in mob_db to 50% of base exp some time ago.

IMO no need to remove mvp exp rate config.