Foundry/taf
1
0
Fork 0
Code Issues 13 Pull requests Releases 9 Wiki Activity Actions

Make it so preUpdate hooks can't violate my data model constraints

Browse source
This commit is contained in:
Oliver 2026-04-25 19:39:25 -06:00
parent 05dcb9afbd
commit d2599d0db3
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
module/data/Item/attribute.mjs
View file

@ -72,6 +72,9 @@ export class AttributeItemData extends foundry.abstract.TypeDataModel {
};
async _preUpdate(data, options, user) {
const allowed = await super._preUpdate(data, options, user);
if (allowed === false) { return false };
// Prevent invalid IDs
if (hasProperty(data, `system.key`) && !isValidID(data.system.key)) {
ui.notifications.error(_loc(
@ -91,8 +94,6 @@ export class AttributeItemData extends foundry.abstract.TypeDataModel {
setProperty(data, `system.value`, clamp(min, value, max));
};
return super._preUpdate(data, options, user);
};
// #endregion Lifecycle