Foundry/.dungeon
0
0
Fork 0
Code Issues 57 Pull requests Projects Releases 7 Packages Wiki Activity Actions

Get the item embedding working in a cleaner way where most of the item embedding logic is handled in the item's class instead of the actor's class

Browse source
This commit is contained in:
Oliver-Akins 2024-03-22 20:23:28 -06:00
parent 0605542d62
commit 18cdc2addc
4 changed files with 20 additions and 48 deletions
Download patch file Download diff file Expand all files Collapse all files

33
module/documents/Actor/Player.mjs
View file

@ -27,39 +27,6 @@ export class Player extends DotDungeonActor {
return this.itemTypes.aspect.length >= limit;
};
async preAspectEmbed(item) {
if (this.atAspectLimit) {
ui.notifications.error(
game.i18n.format(
`dotdungeon.notification.error.aspect-limit-reached`,
{ limit: game.settings.get(`dotdungeon`, `aspectLimit`) }
),
{ console: false }
);
return false;
};
};
/**
* TODO: Make this quantity increase work for all item types
* TODO: Find item based of the source's ID, not name
* @param {DotDungeonItem} item
*/
// async preUntypedEmbed(item) {
// let inventoryItem = this.itemTypes.untyped.find(i => i.name === item.name);
// if (inventoryItem) {
// inventoryItem.update({"system.quantity": inventoryItem.system.quantity + 1});
// ui.notifications.info(
// game.i18n.format(
// `dotdungeon.notification.info.increased-item-quantity`,
// { name: inventoryItem.name }
// ),
// { console: false }
// );
// return false;
// };
// };
getRollData() {
const data = {
initiative: this.system.stats.hands ?? 0,