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

Incompatibility with module "Magic Items" #355

Open
SirZeel opened this issue Dec 30, 2021 · 2 comments
Open

Incompatibility with module "Magic Items" #355

SirZeel opened this issue Dec 30, 2021 · 2 comments

Comments

@SirZeel
Copy link

SirZeel commented Dec 30, 2021

The "Better Rolls" module prevents the "Magic Items" module from correctly applying the upcasting on spells inside the magic item. For example, a level 4 Cure Wounds will still only heal 1d8+mod.

https://i.imgur.com/G3oPunp.png

As you can see, spell is casted at level 4 but still only heals 1d8+mod.
Once I deactivate Better Rolls, everything works fine.

https://gitlab.com/riccisi/foundryvtt-magic-items/-/issues/94

@CarlosFdez
Copy link
Collaborator

CarlosFdez commented Dec 31, 2021

I am no longer maintaining Better Rolls (having switched entirely to pf2e), and we've tried to pass it on, but there have been no takers yet. But I feel for anyone down the line, I should add the context.

The core problem is that 5e doesn't allow you to specify a cast level. 5e casting > damage is split up, which means that they work by editing the spell itself. Magic Items Module for example make a copy of the spell, with the spell level overriden when it is cast. This item does not preserve the original level, making it impossible for BR to detect that its upcast.

The two solutions that come to mind are:

  1. Update core to allow a cast level when the card is made, rather than editing the item. Have magic items make use of that param and not edit the item.
  2. Have magic items not override the level in the copy's source data. Instead create an exact copy and change the copy's prepared level data. That way BR can try to compare against source data.

@CarlosFdez CarlosFdez pinned this issue Dec 31, 2021
@SirZeel
Copy link
Author

SirZeel commented Dec 31, 2021

I am no longer maintaining Better Rolls (having switched entirely to pf2e), and we've tried to pass it on, but there have been no takers yet. But I feel for anyone down the line, I should add the context.

The core problem is that 5e doesn't allow you to specify a cast level. 5e casting > damage is split up, which means that they work by editing the spell itself. Magic Items Module for example make a copy of the spell, with the spell level overriden when it is cast. This item does not preserve the original level, making it impossible for BR to detect that its upcast.

The two solutions that come to mind are:

  1. Update core to allow a cast level when the card is made, rather than editing the item. Have magic items make use of that param and not edit the item.
  2. Have magic items not override the level in the copy's source data. Instead create an exact copy and change the copy's prepared level data. That way BR can try to compare against source data.

Well, thank you for taking the time to answer that. :)

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

No branches or pull requests

2 participants