Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Venom Splasher should consider weapon element #8221

Closed
Playtester opened this issue Apr 4, 2024 · 2 comments · Fixed by #8248
Closed

Venom Splasher should consider weapon element #8221

Playtester opened this issue Apr 4, 2024 · 2 comments · Fixed by #8248
Assignees
Labels
component:skill A fault that deals specifically with a skill mode:prerenewal A fault that exists within the pre-renewal mode status:confirmed Issue that has been validated by a developer to affect rAthena type:bug Issue that is a bug within rAthena

Comments

@Playtester
Copy link
Member

rAthena Hash

8745391

Client Date

2021-11-03

Server Mode

Pre-Renewal

Result

Venom Splasher ignores the weapon element.

Relevant Log Output

No response

Expected Result

Venom Splasher does not ignore the weapon element.
It's also Poison element only when you used Enchant Poison on yourself.
It does not consider attack card modifiers however. Even racial cards are not considered.

How to Reproduce

Use Venom Splasher.
Cast Enchant Poison and use Venom Splasher again on an enemy that takes more or less damage from Poison.
Use Elemental Weapon (e.g. Katar of Quaking) and use Venom Splasher on an enemy that takes no damage from Earth.

Official Information

Assassin Cross 94/69, 99 DEX, All Masteries except Advanced Katar Mastery:

vs. Martin
Main Gauche - 46
Main Gauche + Splasher 1 - 464
VVVS + Splasher 1 - 504
VVVS + Enchant Poison 6 + Splasher 1 - 620
VVVS + Enchant Poison 6 + EDP 5 + Splasher 1 - 620

4x Underneath MG - 82
4x Underneath MG + Splasher 1 - 464
4x Underneath MG + Enchant Poison 6 + Splasher 1 - 580
4x Clamorous MG - 82
4x Clamorous MG + Splasher 1 - 464
4x Clamorous MG + Enchant Poison 6 + Splasher 1 - 580

Enchant Poison 6 + Splasher 1 vs. Verit - MISS
Katar of Quaking + Splasher 1 vs. Verit - 889
Katar of Quaking + Splasher 1 vs. Mandragora - MISS

Modifications that may affect results

No response

@Playtester
Copy link
Member Author

Playtester commented Apr 11, 2024

(All test above and below with Poison React 5.)

vs Golem
Splasher 1 - 185 damage after 11 seconds
Splasher 5 - 250 damage after 7 seconds

5x5 AoE
Damage is divided by amount of enemies in a 3x3 area around the infected.

@Playtester
Copy link
Member Author

Playtester commented Apr 11, 2024

Venom Splasher and Venom Knife both have 100% Poison Chance.

Reduced by Curse (250 -> 177).

Venom Knife Poison Duration ~23 seconds on Golem.

Venom Knife is also not affected by %ATK cards.

@Playtester Playtester self-assigned this Apr 11, 2024
@Playtester Playtester added status:confirmed Issue that has been validated by a developer to affect rAthena component:skill A fault that deals specifically with a skill mode:prerenewal A fault that exists within the pre-renewal mode labels Apr 11, 2024
Playtester added a commit that referenced this issue Apr 12, 2024
- Venom Splasher now ignores +% damage cards, but no longer ignores weapon element
- Venom Splasher now uses the correct durations and cooldowns in pre-re (was already correct in renewal)
- Both skills now correctly use default poison duration
- Venom Splasher and Venom Knife now have a poison chance of 100%
- Fixes #8221
Playtester added a commit that referenced this issue Apr 12, 2024
- Venom Splasher now ignores +% damage cards, but no longer ignores weapon element
- Venom Splasher now uses the correct durations and cooldowns in pre-re (was already correct in renewal)
- Venom Splasher and Venom Knife now correctly use default poison duration
- Venom Splasher and Venom Knife now have a poison chance of 100%
- Fixes #8221
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:skill A fault that deals specifically with a skill mode:prerenewal A fault that exists within the pre-renewal mode status:confirmed Issue that has been validated by a developer to affect rAthena type:bug Issue that is a bug within rAthena
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant