Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Discrepancy between Target Damage and System Damage Buttons #65

Closed
XyrilP opened this issue Oct 19, 2023 · 6 comments
Closed

Discrepancy between Target Damage and System Damage Buttons #65

XyrilP opened this issue Oct 19, 2023 · 6 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@XyrilP
Copy link

XyrilP commented Oct 19, 2023

As title says, may be incompatible with the healing-received selector overall.
Screenshot 2023-10-20 003831
(Healing with Target Damage vs. without it (untargeted).)

@Aeristoka
Copy link

Have this exact same issue.

Original Application buttons have class="heal-damage vs. Target Damage buttons with class="pf2e-td heal-damage applied"

It's definitely something with how pf2e is interpreting the Target Damage healing application button.

@MrVauxs
Copy link
Owner

MrVauxs commented Oct 26, 2023

It's not about the classes. It's about having an outdated applyDamageFromMessage function.
Will have to copy paste the system code again so its up to date.

Made an issue in the pf2e system to not have to copy paste code and keep it from going stale.
foundryvtt/pf2e#10927

@Aeristoka
Copy link

It's not about the classes. It's about having an outdated applyDamageFromMessage function. Will have to copy paste the system code again so its up to date.

Made an issue in the pf2e system to not have to copy paste code and keep it from going stale. foundryvtt/pf2e#10927

Oh interesting, and a good ask of the pf2e devs!

@MrVauxs
Copy link
Owner

MrVauxs commented Oct 26, 2023

Looks like updating the damage function will need more involvement than a mere copy paste, as that doesn't seem to be doing much in terms of fixing these issues, if not outright breaking stuff.

If anyone wants to look into this, I will gladly merge the PR.

@MrVauxs MrVauxs added the help wanted Extra attention is needed label Oct 26, 2023
@MrVauxs MrVauxs pinned this issue Oct 27, 2023
@MrVauxs MrVauxs changed the title Target Damage doesn't work with Healer's Blessing effect Discrepancy between Target Damage and System Damage Buttons Oct 27, 2023
@SMJL88
Copy link

SMJL88 commented Oct 30, 2023

I'm not sure but would this be what is causing the life oracle curse to not apply the penalty properly properly? The system damage button works properly however the target one does not.

@MrVauxs
Copy link
Owner

MrVauxs commented Oct 30, 2023

I'm not sure but would this be what is causing the life oracle curse to not apply the penalty properly properly? The system damage button works properly however the target one does not.

Yes

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants