Skip to content

Commit

Permalink
Fix regression editing NPC lore skills (foundryvtt#15290)
Browse files Browse the repository at this point in the history
  • Loading branch information
CarlosFdez committed Jun 29, 2024
1 parent eb157c8 commit c4ea68a
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion src/module/actor/character/data.ts
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,7 @@ interface CharacterSkillData extends SkillData {
/** Is this skill a Lore skill? */
lore?: boolean;
/** If this is a lore skill, what item it came from */
itemID: string | null;
itemId: string | null;
}

interface CharacterAbilityData extends AbilityData {
Expand Down
2 changes: 1 addition & 1 deletion src/module/actor/character/document.ts
Original file line number Diff line number Diff line change
Expand Up @@ -967,7 +967,7 @@ class CharacterPF2e<TParent extends TokenDocumentPF2e | null = TokenDocumentPF2e
...statistic.getTraceData(),
rank: statistic.rank,
armor: baseData.armor ?? false,
itemID: loreItem?.id ?? null,
itemId: loreItem?.id ?? null,
lore: !!statistic.lore,
});
return [key, data];
Expand Down
2 changes: 1 addition & 1 deletion src/module/actor/npc/data.ts
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,7 @@ interface NPCSkillData extends NPCSkillSource, AttributeBasedTraceData {
/** Is this skill a Lore skill? */
lore?: boolean;
/** If this is a lore skill, what item it came from */
itemID?: string;
itemId?: string;
special: NPCSpecialSkill[];
}

Expand Down
2 changes: 1 addition & 1 deletion src/module/actor/npc/document.ts
Original file line number Diff line number Diff line change
Expand Up @@ -386,7 +386,7 @@ class NPCPF2e<TParent extends TokenDocumentPF2e | null = TokenDocumentPF2e | nul
const data = fu.mergeObject(baseData, {
...statistic.getTraceData(),
mod: statistic.check.mod,
itemID: loreItem?.id ?? null,
itemId: loreItem?.id ?? null,
lore: !!statistic.lore,
visible: statistic.proficient,
});
Expand Down
4 changes: 2 additions & 2 deletions static/templates/actors/character/tabs/proficiencies.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -42,12 +42,12 @@
{{!-- Lore Skills --}}
{{#each data.skills as |skill idx|}}
{{#if skill.lore}}
<li class="item" data-item-id="{{skill.itemID}}" data-item-type="lore" data-statistic="{{skill.slug}}">
<li class="item" data-item-id="{{skill.itemId}}" data-item-type="lore" data-statistic="{{skill.slug}}">
<a class="d20" data-action="roll-check" data-tooltip="{{skill.breakdown}}">
{{> "systems/pf2e/templates/actors/character/icons/d20.hbs"}}
</a>
<a class="modifier" data-action="roll-check">{{numberFormat skill.value decimals=0 sign=true}}</a>
<input class="name" data-item-id="{{skill.itemID}}" data-item-property="name" type="text" value="{{skill.label}}" />
<input class="name" data-item-id="{{skill.itemId}}" data-item-property="name" type="text" value="{{skill.label}}" />
<select class="skill-proficiency pf-rank adjust-item-stat-select" data-item-property="system.proficient.value" data-tooltip="{{skill.breakdown}}" data-rank="{{skill.rank}}">
{{> "systems/pf2e/templates/actors/character/partials/proficiencylevels-dropdown.hbs" proflevel=skill.rank}}
</select>
Expand Down

0 comments on commit c4ea68a

Please sign in to comment.