dungeonslayers/ds4
dungeonslayers/ds4
3
0
Fork 0
Code Issues 9 Pull requests 7 Projects Releases 10 Packages 1 Activity

fix: fix issue where active effects would be applied multiple times

Browse source
This commit is contained in:
Johannes Loher 2022-12-02 00:00:39 +01:00
parent 53216d30cc
commit db6ee373ff
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/documents/active-effect.js
View file

@ -152,8 +152,9 @@ export class DS4ActiveEffect extends ActiveEffect {
// Apply all changes // Apply all changes
for (const changeWithEffect of changesWithEffect) { for (const changeWithEffect of changesWithEffect) {
const result = changeWithEffect.effect.apply(document, changeWithEffect.change); if (!changeWithEffect.change.key) continue;
if (result !== null) overrides[changeWithEffect.change.key] = result; const changes = changeWithEffect.effect.apply(document, changeWithEffect.change);
Object.assign(overrides, changes);
} }
// Expand the set of final overrides // Expand the set of final overrides