From 8b8709d1d8446938eb070129bfc8d6d81af27f94 Mon Sep 17 00:00:00 2001
From: Scott Kiener <scott.kiener@safelite.com>
Date: Mon, 19 Dec 2022 11:49:32 -0500
Subject: [PATCH 1/2] CSR-731 | Fixing package creation logic + styling defect

Styling defect is for CSR-986
---
 .../base-input-button/base-input-button.vue   |  5 ++-
 .../button-functionality-props.js             |  1 +
 .../button-question/button-question.vue       |  1 +
 .../service-package-question.vue              | 31 ++++++++-----------
 src/mixins/input-button-wrapper-mixin.js      |  1 +
 src/ux-components/list-card/list-card.vue     |  3 +-
 6 files changed, 22 insertions(+), 20 deletions(-)

diff --git a/src/common-components/base-input-button/base-input-button.vue b/src/common-components/base-input-button/base-input-button.vue
index bbe92c1d5..91b36c348 100644
--- a/src/common-components/base-input-button/base-input-button.vue
+++ b/src/common-components/base-input-button/base-input-button.vue
@@ -1,6 +1,9 @@
 <template>
     <label
-        :class="[buttonWrapperClasses, { 'has-error': errors.length > 0, selected: isChecked }]"
+        :class="[
+            buttonWrapperClasses,
+            { 'has-error': errors.length > 0 && !suppressError, selected: isChecked },
+        ]"
         :for="buttonId"
         @focusin="handleFocus"
         @focusout="handleBlur"
diff --git a/src/common-components/base-input-button/button-functionality-props.js b/src/common-components/base-input-button/button-functionality-props.js
index 1faa369e2..9a4aef5c9 100644
--- a/src/common-components/base-input-button/button-functionality-props.js
+++ b/src/common-components/base-input-button/button-functionality-props.js
@@ -27,4 +27,5 @@ export const inputButtonProps = {
         type: Boolean,
         default: false,
     },
+    suppressError: Boolean,
 };
diff --git a/src/common-components/button-question/button-question.vue b/src/common-components/button-question/button-question.vue
index b417248a2..646b244be 100644
--- a/src/common-components/button-question/button-question.vue
+++ b/src/common-components/button-question/button-question.vue
@@ -51,6 +51,7 @@
                             :additionalButtonStyling="additionalButtonStyling"
                             :lastValuePushedToGa="lastValuePushedToGa"
                             :setLastValuePushedToGa="setLastValuePushedToGa"
+                            :suppressError="suppressError"
                             v-model="selectedValues" />
                         <!-- For nested questions -->
                         <transition name="fade" mode="out-in">
diff --git a/src/layouts/quote/service-package-question/service-package-question.vue b/src/layouts/quote/service-package-question/service-package-question.vue
index 51cd16a3e..35bb104ef 100644
--- a/src/layouts/quote/service-package-question/service-package-question.vue
+++ b/src/layouts/quote/service-package-question/service-package-question.vue
@@ -118,29 +118,24 @@ export default {
         },
         rearWiperApplicableForTierThree() {
             const rearWiperIsAvailable = this.lineItemsContainsPartType(partTypeStrings.REAR_WIPER);
-            return (
-                this.glassToReplaceContainsGlassLocation(glassLocations.REAR) &&
-                rearWiperIsAvailable
-            );
-        },
-        rainDefenseApplicableForTierThree() {
             const frontWipersAreAvailable = this.lineItemsContainsPartType(
                 partTypeStrings.FRONT_WIPER
             );
-            const glassToReplaceContainsWindshield = this.glassToReplaceContainsGlassLocation(
-                glassLocations.WINDSHIELD
+            return (
+                rearWiperIsAvailable &&
+                (this.glassToReplaceContainsGlassLocation(glassLocations.REAR) ||
+                    !frontWipersAreAvailable)
             );
-            const glassToReplaceContainsRearGlass = this.glassToReplaceContainsGlassLocation(
-                glassLocations.REAR
-            );
-            if (this.frontWipersApplicableForTierTwo) {
-                return true;
-            } else if (!frontWipersAreAvailable) {
-                return true;
-            } else if (!glassToReplaceContainsWindshield && !glassToReplaceContainsRearGlass) {
-                return true;
-            } else {
+        },
+        rainDefenseApplicableForTierThree() {
+            if (
+                this.rearWiperApplicableForTierTwo &&
+                !this.frontWipersApplicableForTierTwo &&
+                this.frontWipersApplicableForTierThree
+            ) {
                 return false;
+            } else {
+                return true;
             }
         },
         shouldDisplayTierTwoPackage() {
diff --git a/src/mixins/input-button-wrapper-mixin.js b/src/mixins/input-button-wrapper-mixin.js
index 1d34d6520..41dd278fe 100644
--- a/src/mixins/input-button-wrapper-mixin.js
+++ b/src/mixins/input-button-wrapper-mixin.js
@@ -22,6 +22,7 @@ export default {
         screenReaderOnlyText: String,
         isWide: Boolean,
         additionalButtonStyling: String,
+        suppressError: Boolean,
     },
     computed: {
         selectedValue: {
diff --git a/src/ux-components/list-card/list-card.vue b/src/ux-components/list-card/list-card.vue
index ed70705c9..fd037361b 100644
--- a/src/ux-components/list-card/list-card.vue
+++ b/src/ux-components/list-card/list-card.vue
@@ -5,7 +5,8 @@
             'list-card w-100 rounded-3 d-flex align-items-center h-100 base-input-button',
             { horizontal: isWide },
         ]"
-        v-model="selectedValue">
+        v-model="selectedValue"
+        :suppressError="suppressError">
         <div
             class="d-flex w-100 align-items-center px-2 h-100 list-card-content button-content rounded-3"
             :class="labelClasses">

From e61591010bf5ae0caa871b6ae5be3994a09989ff Mon Sep 17 00:00:00 2001
From: Scott Kiener <scott.kiener@safelite.com>
Date: Mon, 19 Dec 2022 11:55:49 -0500
Subject: [PATCH 2/2] CSR-986 | Refactor button question changes

---
 src/mixins/input-button-wrapper-mixin.js  | 1 -
 src/ux-components/list-card/list-card.vue | 3 +--
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/mixins/input-button-wrapper-mixin.js b/src/mixins/input-button-wrapper-mixin.js
index 41dd278fe..1d34d6520 100644
--- a/src/mixins/input-button-wrapper-mixin.js
+++ b/src/mixins/input-button-wrapper-mixin.js
@@ -22,7 +22,6 @@ export default {
         screenReaderOnlyText: String,
         isWide: Boolean,
         additionalButtonStyling: String,
-        suppressError: Boolean,
     },
     computed: {
         selectedValue: {
diff --git a/src/ux-components/list-card/list-card.vue b/src/ux-components/list-card/list-card.vue
index fd037361b..ed70705c9 100644
--- a/src/ux-components/list-card/list-card.vue
+++ b/src/ux-components/list-card/list-card.vue
@@ -5,8 +5,7 @@
             'list-card w-100 rounded-3 d-flex align-items-center h-100 base-input-button',
             { horizontal: isWide },
         ]"
-        v-model="selectedValue"
-        :suppressError="suppressError">
+        v-model="selectedValue">
         <div
             class="d-flex w-100 align-items-center px-2 h-100 list-card-content button-content rounded-3"
             :class="labelClasses">