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 ID

  #008471

• Severity

  0 - None Assigned

• Status

  Duplicate

Issue Confirmations

• Yes (0)No (0)

0

Locked 

Genetic Hell Plant damage calculation

Posted by Emistry on 20 December 2014 - 01:30 AM

src/map/battle.c#L3925

	case GN_HELLS_PLANT_ATK:
		md.damage = skill_lv * status->get_lv(target) * 10 + sstatus->int_ * 7 / 2 * (18 + (sd ? sd->status.job_level : 0) / 4) * (5 / (10 - (sd ? pc->checkskill(sd, AM_CANNIBALIZE) : 0)));
		md.damage = md.damage*(1000 + tstatus->mdef) / (1000 + tstatus->mdef * 10) - tstatus->mdef2;
		break;

I notice the value for md.damage is calculated 2 times, I think the second like of md.damage calculation is not needed ?
should be the 1st line for damage calculation.

• Back to top

malufett 

changed status to: Duplicate, 

locked issue

1. Bug Tracker
2. → Hercules Bug Archives
3. → Core
4. → Status: Duplicate
5. Privacy Policy
6. Hercules Forum Rules ·