From 78efe639eda215e24c85c0c631250726461fa9b8 Mon Sep 17 00:00:00 2001
From: Stefan Zermatten <stefanzermatten@gmail.com>
Date: Thu, 28 Jan 2021 16:01:58 +0200
Subject: [PATCH] Fixed: Zero effect passive bonus now causes passive skill
 value to show.

---
 app/imports/api/creature/computation/EffectAggregator.js      | 3 ++-
 app/imports/ui/properties/components/skills/SkillListTile.vue | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/app/imports/api/creature/computation/EffectAggregator.js b/app/imports/api/creature/computation/EffectAggregator.js
index 4419ffd0..4ec98f4c 100644
--- a/app/imports/api/creature/computation/EffectAggregator.js
+++ b/app/imports/api/creature/computation/EffectAggregator.js
@@ -24,7 +24,7 @@ export default class EffectAggregator{
     this.max = Number.POSITIVE_INFINITY;
     this.advantage = 0;
     this.disadvantage = 0;
-    this.passiveAdd = 0;
+    this.passiveAdd = undefined;
     this.fail = 0;
     this.set = undefined;
     this.conditional = [];
@@ -74,6 +74,7 @@ export default class EffectAggregator{
         break;
       case 'passiveAdd':
         // Add all passive adds together
+        if (this.passiveAdd === undefined) this.passiveAdd = 0;
         this.passiveAdd += result;
         break;
       case 'fail':
diff --git a/app/imports/ui/properties/components/skills/SkillListTile.vue b/app/imports/ui/properties/components/skills/SkillListTile.vue
index bfa2d5eb..b4925012 100644
--- a/app/imports/ui/properties/components/skills/SkillListTile.vue
+++ b/app/imports/ui/properties/components/skills/SkillListTile.vue
@@ -31,7 +31,7 @@
         <template v-if="model.conditionalBenefits && model.conditionalBenefits.length">
           *
         </template>
-        <template v-if="model.passiveBonus">
+        <template v-if="'passiveBonus' in model">
           ({{ passiveScore }})
         </template>
       </v-list-tile-title>