Only include `skill_` fields when card has at least 1 of that symbol
This commit is contained in:
parent
01a0ff0aae
commit
7351eade40
|
@ -88,18 +88,6 @@ function build_card(component) {
|
|||
|
||||
var code = pack_code + component.settings.get('CollectionNumber');
|
||||
|
||||
var skills = {
|
||||
Agility: 0,
|
||||
Intellect: 0,
|
||||
Combat: 0,
|
||||
Willpower: 0,
|
||||
Wild: 0,
|
||||
None: 0,
|
||||
};
|
||||
for (var i = 1; i<=6; i++) {
|
||||
skills[component.settings.get('Skill' + i)] += 1;
|
||||
}
|
||||
|
||||
var card_data = {
|
||||
code: String(code),
|
||||
deck_limit: 2, // TODO: could be derived?
|
||||
|
@ -114,11 +102,6 @@ function build_card(component) {
|
|||
quantity: 2, // TODO
|
||||
//restrictions: null, // TODO
|
||||
sanity: int_or_null(component.settings.get('Sanity')),
|
||||
skill_agility: skills["Agility"],
|
||||
skill_combat: skills["Combat"],
|
||||
skill_intellect: skills["Intellect"],
|
||||
skill_wild: skills["Wild"],
|
||||
skill_willpower: skills["Willpower"],
|
||||
// TODO: should also handle "Victory" field
|
||||
text: substitute_tags(String(
|
||||
component.settings.get('Keywords') + '\n' + component.settings.get('Rules'))),
|
||||
|
@ -127,6 +110,25 @@ function build_card(component) {
|
|||
xp: int_or_null(component.settings.get('Level')),
|
||||
};
|
||||
|
||||
var skills = {
|
||||
Agility: 0,
|
||||
Intellect: 0,
|
||||
Combat: 0,
|
||||
Willpower: 0,
|
||||
Wild: 0,
|
||||
};
|
||||
for (var i = 1; i<=6; i++) {
|
||||
var skill_icon = component.settings.get('Skill' + i);
|
||||
if (skill_icon in skills) {
|
||||
skills[skill_icon] += 1;
|
||||
}
|
||||
}
|
||||
for (var skill in skills) {
|
||||
if (skills[skill] > 0) {
|
||||
card_data["skill_" + skill.toLowerCase()] = skills[skill];
|
||||
}
|
||||
}
|
||||
|
||||
var raw_cost = component.settings.get('ResourceCost');
|
||||
if (raw_cost) {
|
||||
card_data.cost = int_or_null(raw_cost);
|
||||
|
|
Loading…
Reference in New Issue