Changed so ancestry uses a single Features field

This commit is contained in:
WBHarry 2025-07-10 22:21:01 +02:00
parent 635dae7a2e
commit 0bda6b5dbe
3 changed files with 46 additions and 31 deletions

View file

@ -19,6 +19,19 @@ export default class AncestrySheet extends DHHeritageSheet {
features: { template: 'systems/daggerheart/templates/sheets/items/ancestry/features.hbs' }
};
async _preparePartContext(partId, context) {
await super._preparePartContext(partId, context);
switch (partId) {
case 'features':
context.primaryFeature = this.document.system.features.find(x => x.primary)?.value;
context.secondaryFeature = this.document.system.features.find(x => !x.primary)?.value;
break;
}
return context;
}
/* -------------------------------------------- */
/* Application Clicks Actions */
/* -------------------------------------------- */
@ -32,10 +45,13 @@ export default class AncestrySheet extends DHHeritageSheet {
type: 'feature',
name: game.i18n.format('DOCUMENT.New', { type: game.i18n.localize('TYPES.Item.feature') })
});
await this.document.update({
system: {
features: [...this.document.system.features.map(x => x.uuid), feature.uuid],
[`${button.dataset.type}Feature`]: feature.uuid
features: [
...this.document.system.features.map(x => ({ ...x, value: x.value.uuid })),
{ primary: button.dataset.type === 'primary', value: feature.uuid }
]
}
});
}
@ -46,7 +62,9 @@ export default class AncestrySheet extends DHHeritageSheet {
*/
static async #editFeature(_event, button) {
const target = button.closest('.feature-item');
const feature = this.document.system[`${target.dataset.type}Feature`];
const feature = this.document.system.features.find(x =>
target.dataset.type === 'primary' ? x.primary : !x.primary
)?.value;
if (!feature) {
ui.notifications.warn(game.i18n.localize('DAGGERHEART.UI.Notifications.featureIsMissing'));
return;
@ -62,7 +80,9 @@ export default class AncestrySheet extends DHHeritageSheet {
static async #removeFeature(event, button) {
event.stopPropagation();
const target = button.closest('.feature-item');
const feature = this.document.system[`${target.dataset.type}Feature`];
const feature = this.document.system.features.find(x =>
target.dataset.type === 'primary' ? x.primary : !x.primary
)?.value;
if (feature) {
const confirmed = await foundry.applications.api.DialogV2.confirm({
@ -78,10 +98,9 @@ export default class AncestrySheet extends DHHeritageSheet {
}
await this.document.update({
system: {
features: this.document.system.features.filter(x => x.uuid !== feature.uuid).map(x => x.uuid),
[`${target.dataset.type}Feature`]: null
}
'system.features': this.document.system.features
.filter(x => x.value.uuid !== feature.uuid)
.map(x => ({ ...x, value: x.value.uuid }))
});
}
@ -101,19 +120,12 @@ export default class AncestrySheet extends DHHeritageSheet {
const item = await fromUuid(data.uuid);
if (item?.type === 'feature') {
const update = {
system: {
features: [...this.document.system.features.map(x => x.uuid), item.uuid]
}
};
if (event.target.closest('.primary-feature')) {
update.system.primaryFeature = item.uuid;
} else if (event.target.closest('.secondary-feature')) {
update.system.secondaryFeature = item.uuid;
}
await this.document.update(update);
await this.document.update({
'system.features': [
...this.document.system.features.map(x => ({ ...x, value: x.value.uuid })),
{ primary: Boolean(event.target.closest('.primary-feature')), value: item.uuid }
]
});
}
}
}

View file

@ -1,4 +1,3 @@
import ForeignDocumentUUIDArrayField from '../fields/foreignDocumentUUIDArrayField.mjs';
import ForeignDocumentUUIDField from '../fields/foreignDocumentUUIDField.mjs';
import BaseDataItem from './base.mjs';
@ -14,11 +13,15 @@ export default class DHAncestry extends BaseDataItem {
/** @inheritDoc */
static defineSchema() {
const fields = foundry.data.fields;
return {
...super.defineSchema(),
features: new ForeignDocumentUUIDArrayField({ type: 'Item' }),
primaryFeature: new ForeignDocumentUUIDField({ type: 'Item' }),
secondaryFeature: new ForeignDocumentUUIDField({ type: 'Item' })
features: new fields.ArrayField(
new fields.SchemaField({
primary: new fields.BooleanField(),
value: new ForeignDocumentUUIDField({ type: 'Item' })
})
)
};
}
}

View file

@ -6,13 +6,13 @@
<fieldset class="one-column drop-section primary-feature">
<legend>{{localize "DAGGERHEART.ITEMS.Ancestry.primaryFeature"}} <a><i data-action="addFeature" data-type="primary" class="fa-solid fa-plus icon-button"></i></a></legend>
<div class="features-list">
{{#if document.system.primaryFeature}}
{{#if primaryFeature}}
<div class="feature-item"
data-action="editFeature"
data-type="primary"
>
<img class="image" src="{{document.system.primaryFeature.img}}" />
<span>{{document.system.primaryFeature.name}}</span>
<img class="image" src="{{primaryFeature.img}}" />
<span>{{primaryFeature.name}}</span>
<div class="controls">
<a data-action="removeFeature" data-type="primary"><i class="fa-solid fa-trash"></i></a>
</div>
@ -24,13 +24,13 @@
<fieldset class="one-column drop-section secondary-feature">
<legend>{{localize "DAGGERHEART.ITEMS.Ancestry.secondaryFeature"}} <a><i data-action="addFeature" data-type="secondary" class="fa-solid fa-plus icon-button"></i></a></legend>
<div class="features-list">
{{#if document.system.secondaryFeature}}
{{#if secondaryFeature}}
<div class="feature-item"
data-action="editFeature"
data-type="secondary"
>
<img class="image" src="{{document.system.secondaryFeature.img}}" />
<span>{{document.system.secondaryFeature.name}}</span>
<img class="image" src="{{secondaryFeature.img}}" />
<span>{{secondaryFeature.name}}</span>
<div class="controls">
<a data-action="removeFeature" data-type="secondary"><i class="fa-solid fa-trash"></i></a>
</div>