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

Support Active Effects for Items, Spells, and Features which apply to the Item Owner (self) #799

Open
aaclayton opened this issue Nov 11, 2020 · 15 comments

Comments

@aaclayton
Copy link
Contributor

Progress Tracking: https://docs.google.com/spreadsheets/d/1-n6clYWgbBJu_jLlvgQAUcSgWZNZPbejeztjmaFB3Uk/edit (you must request access)

@aaclayton
Copy link
Contributor Author

marked this issue as related to #797

@aaclayton
Copy link
Contributor Author

marked this issue as related to #796

@aaclayton
Copy link
Contributor Author

marked this issue as related to #795

@aaclayton
Copy link
Contributor Author

marked this issue as related to #798

@aaclayton
Copy link
Contributor Author

Originally in GitLab by @siliconsaint

Defaults

If a Source is linked to an AE, then use that source info for the following:

[Details Tab]
Effect Label
Effect Icon

[Duration Tab]
Effect Duration

Example, for creating a AE for bless you would gain:
Effect Label: Bless
Effect Icon: systems/dnd5e/icons/spells/haste-sky-1.jpg
Duration: 1 min

@aaclayton
Copy link
Contributor Author

Originally in GitLab by @siliconsaint

Duration
Simplify duration increment to a dropdown that offers the following (ordered by increasing size):/
seconds, rounds, turns, minutes, hours, days

  • seconds, rounds, turns, minutes are useful for combat
  • minutes, hours are useful for effects that reset after a short or long rest
  • days are useful for items that may reset at dawn/dusk

Ideally this would eliminate 2-4 areas of duplication on the duration tab and maybe even allow it to be combined with the [Details] tab?

@aaclayton
Copy link
Contributor Author

Originally in GitLab by @siliconsaint

Priority
DAE introduces the concept of priority of an effect. If AE supports priority inherently by top to bottom. Top being highest priority and bottom being least, then it should allow us to move a row up or down to re-order on the Effects tab.

@aaclayton
Copy link
Contributor Author

Originally in GitLab by @siliconsaint

Attribute Reference within Effect Value Field

Consideration for Attribute reference within the Effects Value field.

Examples:

  • Unarmored Defense (Monk), Effect Value: 10 + @abilities.wis.mod + @abilities.dex.mod
  • Unarmored Defense (Barbarian), Effect Value: 10 + @abilities.dex.mod + @abilities.con.mod
  • Second-Story Work, Effect Value: @attributes.movement.walk
  • Dragon Wings, Effect Value: @attributes.movement.walk
  • Mage Armor, Effect Value: 12+@abilities.dex.mod

@aaclayton
Copy link
Contributor Author

It is an intentional limitation - at least for now - that the values of active effect changes be static. It sounds like you're wanting a way to handle armor class scaling which will be handled through a separate system.

@aaclayton
Copy link
Contributor Author

Originally in GitLab by @siliconsaint

Indeed, I have offered this under the associated https://gitlab.com/foundrynet/dnd5e/-/issues/424 but I feel attribute references could be useful outside of strictly AC related items. Just trying to list all the current limitations of AE, if we choose to expand it in a certain direction.

@aaclayton
Copy link
Contributor Author

Originally in GitLab by @siliconsaint

Regeneration effects

There are a variety of features, spells, and items that may apply hit points or temporary hit points "every round". Currently it seems AE will expand the total HP or Temp HP, but does not Heal or Top off Temp HPs on a round by round basis.

(some examples are beyond SRD, but the concepts are still valid)

Examples:

  • Regenerate: For the Duration of the spell, the target regains 1 hit point at the start of each of its turns (10 h⁠it points each minute)
  • Twilight Cleric's Twilight Sanctuary: 1d6+Cleric level Temp HPs every round
  • Oath of Redemption Paladin: 1d6+half paladin level healing every turn (if less than half HPs)
  • Survivor: 5 + con mod every turn (if less than half HPs)

...and it would not sadden me to remove the need for 80 line macros that are required to create effects like regenerate.

@aaclayton
Copy link
Contributor Author

Originally in GitLab by @siliconsaint

Vision updated for Actor, but not Token

This is a reminder that we can modify vision on the Actor via AE but that is not reflected on the token itself.

@aaclayton
Copy link
Contributor Author

Originally in GitLab by @siliconsaint

Special Traits -> Global Bonuses do not reflect AE adjustments

Modifications to fields by AE that can also be adjusted by Global Bonuses will not show up on the Global Bonuses field list within the Character Sheet.

example: Cloak of Protection AE for +1 to all saving throws will not appear under the Global Saving Throw Bonus of Global Bonuses. Functionally it will work, however.

@aaclayton
Copy link
Contributor Author

Originally in GitLab by @DanielAyoung

Priority between active effects is essential to someday enabling derived data like ability modifiers to be used within AE effect value fields. This is because you need to ensure that any AE that changes the ability value be processed before any AE that uses the ability modifier.
This would benefit many game elements beyond AC. As an example, (non-srd, but illustrative) the war mage "tactical wit" give an initiative bonus equal to the int modifier.

@tdockx
Copy link

tdockx commented Feb 9, 2024

Hi @aaclayton,

Apologies, for maybe bringing back an old topic. (Using the latest version : DND 3.0.1)
It is more related to 797

It seems the Bracers of Archery should also have :

  • Additionally, while wearing these bracers, the character gains a +2 bonus to damage rolls made with longbows and shortbows.

This still does not seem to be the case, when equipped and attuned the damage bonus is not calculated with that in mind.

Thanks a lot.
Thibault.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants