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

  • #008513

  • 0 - None Assigned

  • Fixed

Issue Confirmations

  • Yes (0)No (0)
Photo

Lightning Spear damage

Posted by Nihad on 29 January 2015 - 04:50 PM

As reported by one of our players and then confirmed by staff.


The skill's damage appears to increase according to the skill level used. The skill is supposed to do 70% of my matk per hit, my matk being 199+64, so it should be doing around 165 damage to a chonchon (Which has 0 hard and soft mdef and takes 90% from water.) per hit, when in reality it's doing a range of 283 to 303 at level 1 per hit, nearly double that value.

At level 2, the damage per hit moves up to 443, and it continues to increase per level, where at level 10 it does 1506 per hit to a chonchon.


can you show us the complete test setup?

:meow:

#ifdef RENEWAL
				case NJ_HYOUSENSOU:
					skillratio -= 30;
					if (sd && sd->charm_type == CHARM_TYPE_WATER && sd->charm_count > 0)
						skillratio += 5 * sd->charm_count;
#endif
					/* Fall through */
				case NJ_HYOUSYOURAKU:
					skillratio += 50 * skill_lv;
					if (sd && sd->charm_type == CHARM_TYPE_WATER && sd->charm_count > 0)
						skillratio += 25 * sd->charm_count;
					break;
@malu in battle.c I think there is a break missing NJ_HYOUSENSOU got the bonus of NJ_HYOUSYOURAKU

#ifdef RENEWAL
				case NJ_HYOUSENSOU:
					skillratio -= 30;
					if (sd && sd->charm_type == CHARM_TYPE_WATER && sd->charm_count > 0)
						skillratio += 5 * sd->charm_count;
#endif
					/* Fall through */
				case NJ_HYOUSYOURAKU:
					skillratio += 50 * skill_lv;
					if (sd && sd->charm_type == CHARM_TYPE_WATER && sd->charm_count > 0)
						skillratio += 25 * sd->charm_count;
					break;
@malu in battle.c I think there is a break missing NJ_HYOUSENSOU got the bonus of NJ_HYOUSYOURAKU


Confirmed, made a pull request. Adding break fixes it.

You should probably try to figure out where the "Fall through" comment comes from, because that usually means that the break; was intentionally left away.

changed status to: Fixed