From bf9e59d64cd298e5220fa2a6d351316264ba0efe Mon Sep 17 00:00:00 2001
From: smilerz <smilerz@gmail.com>
Date: Mon, 20 Dec 2021 14:59:56 -0600
Subject: [PATCH] fix Shopping Modal to filter onhand from initial list

---
 vue/src/components/Modals/ShoppingModal.vue | 297 +++++++++++---------
 1 file changed, 158 insertions(+), 139 deletions(-)

diff --git a/vue/src/components/Modals/ShoppingModal.vue b/vue/src/components/Modals/ShoppingModal.vue
index be0089208..5219d8513 100644
--- a/vue/src/components/Modals/ShoppingModal.vue
+++ b/vue/src/components/Modals/ShoppingModal.vue
@@ -1,158 +1,177 @@
 <template>
     <div>
-      <b-modal :id="`shopping_${this.modal_id}`" hide-footer @show="loadRecipe"> 
-        <template v-slot:modal-title><h4>{{$t('Add_Servings_to_Shopping',{'servings': servings})}}</h4></template>
-        <loading-spinner v-if="loading"></loading-spinner>
-        <div class="accordion" role="tablist" v-if="!loading">
-            <b-card no-body class="mb-1">
-                <b-card-header header-tag="header" class="p-1" role="tab">
-                    <b-button block v-b-toggle.accordion-0 class="text-left" variant="outline-info">{{recipe.name}}</b-button>
-                </b-card-header>
-                <b-collapse id="accordion-0" visible accordion="my-accordion" role="tabpanel">
-                    <ingredients-card 
-                        :steps="steps"
-                        :recipe="recipe.id"
-                        :ingredient_factor="ingredient_factor"
-                        :servings="servings"
-                        :show_shopping="true"
-                        :add_shopping_mode="true"
-                        :header="false"
-                        @add-to-shopping="addShopping($event)"
-                    />
-                </b-collapse>
-                <!-- eslint-disable vue/no-v-for-template-key-on-child -->
-                <template v-for="r in related_recipes">
-                  <b-card no-body class="mb-1" :key="r.recipe.id">
+        <b-modal :id="`shopping_${this.modal_id}`" hide-footer @show="loadRecipe">
+            <template v-slot:modal-title
+                ><h4>{{ $t("Add_Servings_to_Shopping", { servings: servings }) }}</h4></template
+            >
+            <loading-spinner v-if="loading"></loading-spinner>
+            <div class="accordion" role="tablist" v-if="!loading">
+                <b-card no-body class="mb-1">
                     <b-card-header header-tag="header" class="p-1" role="tab">
-                        <b-button btn-sm block v-b-toggle="'accordion-' +r.recipe.id" class="text-left" variant="outline-primary">{{r.recipe.name}}</b-button>
+                        <b-button block v-b-toggle.accordion-0 class="text-left" variant="outline-info">{{ recipe.name }}</b-button>
                     </b-card-header>
-                    <b-collapse  :id="'accordion-'+r.recipe.id" accordion="my-accordion" role="tabpanel">
+                    <b-collapse id="accordion-0" visible accordion="my-accordion" role="tabpanel">
                         <ingredients-card
-                          :steps="r.steps"
-                          :recipe="r.recipe.id"
-                          :ingredient_factor="ingredient_factor"
-                          :servings="servings"
-                          :show_shopping="true"
-                          :add_shopping_mode="true"
-                          :header="false"
-                          @add-to-shopping="addShopping($event)"
-                      />
+                            :steps="steps"
+                            :recipe="recipe.id"
+                            :ingredient_factor="ingredient_factor"
+                            :servings="servings"
+                            :show_shopping="true"
+                            :add_shopping_mode="true"
+                            :header="false"
+                            @add-to-shopping="addShopping($event)"
+                        />
                     </b-collapse>
-                  </b-card>
-                </template>
-                <!-- eslint-disable vue/no-v-for-template-key-on-child -->
-            </b-card>
-            
-        </div>
-        <div class="row mt-3 mb-3">
-            <div class="col-12 text-right">
-                <b-button class="mx-2" variant="secondary" @click="$bvModal.hide(`shopping_${modal_id}`)">{{ $t('Cancel') }}
-                </b-button>
-                <b-button class="mx-2" variant="success" @click="saveShopping">{{ $t('Save') }}
-                </b-button>
+                    <!-- eslint-disable vue/no-v-for-template-key-on-child -->
+                    <template v-for="r in related_recipes">
+                        <b-card no-body class="mb-1" :key="r.recipe.id">
+                            <b-card-header header-tag="header" class="p-1" role="tab">
+                                <b-button btn-sm block v-b-toggle="'accordion-' + r.recipe.id" class="text-left" variant="outline-primary">{{ r.recipe.name }}</b-button>
+                            </b-card-header>
+                            <b-collapse :id="'accordion-' + r.recipe.id" accordion="my-accordion" role="tabpanel">
+                                <ingredients-card
+                                    :steps="r.steps"
+                                    :recipe="r.recipe.id"
+                                    :ingredient_factor="ingredient_factor"
+                                    :servings="servings"
+                                    :show_shopping="true"
+                                    :add_shopping_mode="true"
+                                    :header="false"
+                                    @add-to-shopping="addShopping($event)"
+                                />
+                            </b-collapse>
+                        </b-card>
+                    </template>
+                    <!-- eslint-disable vue/no-v-for-template-key-on-child -->
+                </b-card>
             </div>
-        </div>
-      </b-modal>
+            <div class="row mt-3 mb-3">
+                <div class="col-12 text-right">
+                    <b-button class="mx-2" variant="secondary" @click="$bvModal.hide(`shopping_${modal_id}`)">{{ $t("Cancel") }} </b-button>
+                    <b-button class="mx-2" variant="success" @click="saveShopping">{{ $t("Save") }} </b-button>
+                </div>
+            </div>
+        </b-modal>
     </div>
 </template>
 
 <script>
-import Vue from 'vue'
-import {BootstrapVue} from 'bootstrap-vue'
+import Vue from "vue"
+import { BootstrapVue } from "bootstrap-vue"
 Vue.use(BootstrapVue)
 
-const {ApiApiFactory} = require("@/utils/openapi/api");
-import {StandardToasts} from "@/utils/utils";
-import IngredientsCard from "@/components/IngredientsCard";
-import LoadingSpinner from "@/components/LoadingSpinner";
-
+const { ApiApiFactory } = require("@/utils/openapi/api")
+import { StandardToasts } from "@/utils/utils"
+import IngredientsCard from "@/components/IngredientsCard"
+import LoadingSpinner from "@/components/LoadingSpinner"
 
 export default {
-  name: 'ShoppingModal',
-  components: {IngredientsCard, LoadingSpinner},
-  mixins: [],
-  props: {
-    recipe: {required: true, type: Object},
-    servings: {type: Number},
-    modal_id: {required: true, type: Number},
-
-  },
-  data() {
-    return {
-      loading: true,
-      steps: [],
-      recipe_servings: 0,
-      add_shopping: [],
-      related_recipes: []
-    }
-  },
-  mounted() {
-  },
-  computed: {
-    ingredient_factor: function () {
-      return this.servings / this.recipe.servings || this.recipe_servings
+    name: "ShoppingModal",
+    components: { IngredientsCard, LoadingSpinner },
+    mixins: [],
+    props: {
+        recipe: { required: true, type: Object },
+        servings: { type: Number },
+        modal_id: { required: true, type: Number },
     },
-  },
-  watch: {
-
-  },
-  methods: {
-    loadRecipe: function() {
-      this.add_shopping = []
-      this.related_recipes = []
-      let apiClient = new ApiApiFactory()
-      apiClient.retrieveRecipe(this.recipe.id).then((result) => {
-        this.steps = result.data.steps
-        // ALERT: this will all break if ingredients are re-used between recipes
-        // ALERT: this also doesn't quite work right if the same recipe appears multiple time in the related recipes
-        this.add_shopping = [...this.add_shopping, ...this.steps.map(x => x.ingredients).flat().map(x => x.id)]
-        this.recipe_servings = result.data?.servings
-        this.loading = false
-      })
-
-      // get a list of related recipes
-      apiClient.relatedRecipe(this.recipe.id).then((result) => {
-        return result.data
-      }).then((related_recipes) => {
-        let promises = []
-        related_recipes.forEach(x => {
-            promises.push(apiClient.listSteps(x.id).then((recipe_steps) => {
-              this.related_recipes.push({
-                'recipe': x,
-                'steps': recipe_steps.data.results.filter(x => x.ingredients.length > 0)
-              })
-            }))
-        })
-        return Promise.all(promises)
-      }).then(() => {
-        this.add_shopping = [
-          ...this.add_shopping,
-          ...this.related_recipes.map(x => x.steps).flat().map(x => x.ingredients).flat().map(x => x.id)
-        ]
-      })
+    data() {
+        return {
+            loading: true,
+            steps: [],
+            recipe_servings: 0,
+            add_shopping: [],
+            related_recipes: [],
+        }
     },
-    addShopping: function(e) {
-      if (e.add) {
-        this.add_shopping.push(e.item.id)
-      } else {
-        this.add_shopping = this.add_shopping.filter(x => x !== e.item.id)
-      }
+    mounted() {},
+    computed: {
+        ingredient_factor: function () {
+            return this.servings / this.recipe.servings || this.recipe_servings
+        },
+    },
+    watch: {},
+    methods: {
+        loadRecipe: function () {
+            this.add_shopping = []
+            this.related_recipes = []
+            let apiClient = new ApiApiFactory()
+            apiClient
+                .retrieveRecipe(this.recipe.id)
+                .then((result) => {
+                    this.steps = result.data.steps
+                    // ALERT: this will all break if ingredients are re-used between recipes
+                    // ALERT: this also doesn't quite work right if the same recipe appears multiple time in the related recipes
+                    this.add_shopping = [
+                        ...this.add_shopping,
+                        ...this.steps
+                            .map((x) => x.ingredients)
+                            .flat()
+                            .filter((x) => !x.food.on_hand && !x.food.ignore_shopping)
+                            .map((x) => x.id),
+                    ]
+                    this.recipe_servings = result.data?.servings
+                    this.loading = false
+                })
+                .then(() => {
+                    // get a list of related recipes
+                    apiClient
+                        .relatedRecipe(this.recipe.id)
+                        .then((result) => {
+                            return result.data
+                        })
+                        .then((related_recipes) => {
+                            let promises = []
+                            related_recipes.forEach((x) => {
+                                promises.push(
+                                    apiClient.listSteps(x.id).then((recipe_steps) => {
+                                        this.related_recipes.push({
+                                            recipe: x,
+                                            steps: recipe_steps.data.results.filter((x) => x.ingredients.length > 0),
+                                        })
+                                    })
+                                )
+                            })
+                            return Promise.all(promises)
+                        })
+                        .then(() => {
+                            this.add_shopping = [
+                                ...this.add_shopping,
+                                ...this.related_recipes
+                                    .map((x) => x.steps)
+                                    .flat()
+                                    .map((x) => x.ingredients)
+                                    .flat()
+                                    .filter((x) => !x.food.on_hand && !x.food.ignore_shopping)
+                                    .map((x) => x.id),
+                            ]
+                        })
+                })
+        },
+        addShopping: function (e) {
+            if (e.add) {
+                this.add_shopping.push(e.item.id)
+            } else {
+                this.add_shopping = this.add_shopping.filter((x) => x !== e.item.id)
+            }
+        },
+        saveShopping: function () {
+            // another choice would be to create ShoppingListRecipes for each recipe - this bundles all related recipe under the parent recipe
+            let shopping_recipe = {
+                id: this.recipe.id,
+                ingredients: this.add_shopping,
+                servings: this.servings,
+            }
+            let apiClient = new ApiApiFactory()
+            apiClient
+                .shoppingRecipe(this.recipe.id, shopping_recipe)
+                .then((result) => {
+                    StandardToasts.makeStandardToast(StandardToasts.SUCCESS_CREATE)
+                })
+                .catch((err) => {
+                    StandardToasts.makeStandardToast(StandardToasts.FAIL_CREATE)
+                })
+            this.$bvModal.hide(`shopping_${this.modal_id}`)
+        },
     },
-    saveShopping: function() {
-      // another choice would be to create ShoppingListRecipes for each recipe - this bundles all related recipe under the parent recipe
-      let shopping_recipe = {
-        'id': this.recipe.id,
-        'ingredients': this.add_shopping,
-        'servings': this.servings
-      }
-      let apiClient = new ApiApiFactory()
-      apiClient.shoppingRecipe(this.recipe.id, shopping_recipe).then((result) => {
-        StandardToasts.makeStandardToast(StandardToasts.SUCCESS_CREATE)
-      }).catch((err) => {
-        StandardToasts.makeStandardToast(StandardToasts.FAIL_CREATE)
-      })
-      this.$bvModal.hide(`shopping_${this.modal_id}`)
-    }
-  }
 }
-</script>
\ No newline at end of file
+</script>