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

17.17-beta4 Move and Attack, ranged penalty issues #2109

Closed
devakm opened this issue Jan 6, 2025 · 2 comments
Closed

17.17-beta4 Move and Attack, ranged penalty issues #2109

devakm opened this issue Jan 6, 2025 · 2 comments
Labels
bug Something isn't working

Comments

@devakm
Copy link

devakm commented Jan 6, 2025

Multiple issues here related to range penalty and/or Move and Attack.

First, Move and Attack has a ranged penalty of -2, not -4 as seen here:

image

Second, the #ranged tag is not working to apply the -2 penalty for missile spells if I invoke it within a larger OtF. I can create a simple OtF like [R:Fireball+1 Acc] and the tagged modifier works, but not if I do it in an OtF that casts the spell first like:

["Fireball"/r [/if [Sp:Fireball] cs:{/if [R:Fireball+1 Modifier Bucket Applied] {/:SpellDamage title=Fireball dice=1 adds=+2 type=burn adjust=-8 crit=true }} cf:{/rolltable CritMiss} s:{/if [R:Fireball+1 Modifier Bucket Applied] cs:{/:SpellDamage title=Fireball dice=1 adds=+2 type=burn adjust=-8 crit=true } s:{/:SpellDamage title=Fireball dice=1 adds=0 type=burn adjust=-2 crit=false} cf:{/rolltable CritMiss} f:{/fp -1}} f:{/fp -1} ]]

Third, the range penalty for distance to target is auto-tagged as Ranged and Spell, which makes sense for Regular spells (they get the #spell tag), but this is wrong for Missile spells. Unlike Regular spells, you cast a missile spell into your hand and then can hold onto it or attack with it using Innate Attack skill to make the Ranged [R:Fireball] hit roll, where the range penalty and Aim Accuracy +1 counts.

The whole idea of Ranged Accuracy is intended to offset the range penalty.

We may need to come up with a new way of distinguishing Missile spells from Regular spells to avoid this confusion.

@devakm devakm changed the title 17.17-beta4 Move and Attack ranged penalty issues 17.17-beta4 Move and Attack, ranged penalty issues Jan 6, 2025
@mjeffw mjeffw added the bug Something isn't working label Jan 6, 2025
@devakm
Copy link
Author

devakm commented Jan 12, 2025

Most of this ticket was resolved in another ticket except for the Move and Attack #ranged penalty, which is still applying -4 instead of the correct -2.

@mjeffw
Copy link
Collaborator

mjeffw commented Jan 12, 2025

Give it a few hours...

image

@mjeffw mjeffw closed this as completed Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants