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

  • #008437

  • 0 - None Assigned

  • New

Issue Confirmations

  • Yes (0)No (0)
Photo

Burning status to MOB doesn't have damage

Posted by Hadeszeus on 12 November 2014 - 10:01 AM

When using banaspaty card to monster, it only gets burn status without burn damage. (Is this normal?)
But Dragons Breath (Fire) has burn damage.

Banaspaty Card isn't implemented at the moment.

Emulator structure is designed so that you are supposed to link the "default skill" to the status change so that we know the default duration which is taken from duration2. This wasn't done properly ever since rAthena started.

Two problems:
- Several devs added skills with status changes but put their duration in duration1 rather than duration2, that means duration2 is 0ms even if linking was done correctly
- The linking of common ailments doesn't include the new status changes:
//First we define the skill for common ailments. These are used in skill_additional_effect through sc cards. [Skotlex]
set_sc( NPC_PETRIFYATTACK , SC_STONE , SI_BLANK , SCB_DEF_ELE|SCB_DEF|SCB_MDEF );
set_sc( NPC_WIDEFREEZE , SC_FREEZE , SI_BLANK , SCB_DEF_ELE|SCB_DEF|SCB_MDEF );
set_sc( NPC_STUNATTACK , SC_STUN , SI_BLANK , SCB_NONE );
set_sc( NPC_SLEEPATTACK , SC_SLEEP , SI_BLANK , SCB_NONE );
set_sc( NPC_POISON , SC_POISON , SI_BLANK , SCB_DEF2|SCB_REGEN );
set_sc( NPC_CURSEATTACK , SC_CURSE , SI_BLANK , SCB_LUK|SCB_BATK|SCB_WATK|SCB_SPEED );
set_sc( NPC_SILENCEATTACK , SC_SILENCE , SI_BLANK , SCB_NONE );
set_sc( NPC_WIDECONFUSE , SC_CONFUSION , SI_BLANK , SCB_NONE );
set_sc( NPC_BLINDATTACK , SC_BLIND , SI_BLANK , SCB_HIT|SCB_FLEE );
set_sc( NPC_BLEEDING , SC_BLOODING , SI_BLOODING , SCB_REGEN );
set_sc( NPC_POISON , SC_DPOISON , SI_BLANK , SCB_DEF2|SCB_REGEN );
Only goes up to Deadly Poison.

To fix it, first you need to rewrite all the skills added since rAthena to use duration2 and also have duration2 defined in the skill_cast_db. Then look up which skill uses the default duration (for Burning we don't even have the default duration skill implemented -> should probably use NPC_WIDEBODYBURNNING, though) and link to it at the top (below the code I quoted).

Bump, the solution from Playtester's post doesn't work, status is also ignoring minimum specified duration.

Edited by Anisotropic Defixation, 25 August 2015 - 10:10 AM.