From 64735b5ee5d7cb0a99579d0a46e8750123beded8 Mon Sep 17 00:00:00 2001
From: Dan Elbert <dan.elbert@gmail.com>
Date: Sun, 9 May 2021 23:31:44 -0500
Subject: [PATCH] Convert recipe to task list

---
 app/controllers/task_lists_controller.rb | 12 ++++++++-
 app/javascript/components/RecipeShow.vue | 34 ++++++++++++++++++++++++
 app/javascript/lib/Api.js                |  4 +++
 app/javascript/store/index.js            |  8 ++++++
 app/models/task_list.rb                  | 22 +++++++++++++++
 app/views/home/index.html.erb            |  2 +-
 config/routes.rb                         |  3 +++
 7 files changed, 83 insertions(+), 2 deletions(-)

diff --git a/app/controllers/task_lists_controller.rb b/app/controllers/task_lists_controller.rb
index b321401..a48e1e8 100644
--- a/app/controllers/task_lists_controller.rb
+++ b/app/controllers/task_lists_controller.rb
@@ -1,7 +1,7 @@
 class TaskListsController < ApplicationController
 
   before_action :ensure_valid_user
-  before_action :set_task_list, only: [:show, :update, :destroy]
+  before_action :set_task_list, only: [:show, :update, :destroy, :add_recipe]
 
   def index
     @task_lists = TaskList.for_user(current_user).includes(:task_items).order(created_at: :desc)
@@ -40,6 +40,16 @@ class TaskListsController < ApplicationController
       head :no_content
     end
   end
+  
+  def add_recipe
+    ensure_owner(@task_list) do
+      recipe = Recipe.find(params[:recipe_id])
+      
+      @task_list.add_recipe_ingredients(recipe)
+      
+      head :no_content
+    end
+  end
 
   private
 
diff --git a/app/javascript/components/RecipeShow.vue b/app/javascript/components/RecipeShow.vue
index c249bf9..5065b47 100644
--- a/app/javascript/components/RecipeShow.vue
+++ b/app/javascript/components/RecipeShow.vue
@@ -37,6 +37,11 @@
             <div class="message-header">
               Ingredients
               <button class="button is-small is-primary" type="button" @click="showConvertDialog = true">Convert</button>
+              <app-dropdown :open="addToTasksMenuOpen" label="Add to list" button-class="is-small is-primary" @open="addToTasksMenuOpen = true" @close="addToTasksMenuOpen = false">
+                <button class="button primary" v-for="tl in taskLists" :key="tl.id" @click="addRecipeToList(tl)">
+                  {{tl.name}}
+                </button>
+              </app-dropdown>
             </div>
             <div class="message-body content">
               <ul v-if="recipe.ingredients.length > 0" v-click-strike>
@@ -143,6 +148,9 @@
 
 <script>
 
+  import api from "../lib/Api";
+  import { mapActions, mapMutations, mapState } from "vuex";
+
   export default {
     props: {
       recipe: {
@@ -155,6 +163,7 @@
       return {
         showNutrition: false,
         showConvertDialog: false,
+        addToTasksMenuOpen: false,
 
         scaleValue: '1',
         systemConvertValue: "",
@@ -176,6 +185,10 @@
     },
 
     computed: {
+      ...mapState([
+          'taskLists'
+      ]),
+      
       timeDisplay() {
         let a = this.formatMinutes(this.recipe.active_time);
         const t = this.formatMinutes(this.recipe.total_time);
@@ -222,6 +235,23 @@
     },
 
     methods: {
+      ...mapActions([
+          'ensureTaskLists'
+      ]),
+      
+      ...mapMutations([
+          'setCurrentTaskList'
+      ]),
+
+      addRecipeToList(list) {
+        console.log(list);
+        api.addRecipeToTaskList(list.id, this.recipe.id)
+        .then(() => {
+          this.setCurrentTaskList(list);
+          this.$router.push({name: 'task_lists'})
+        });
+      },
+      
       convert() {
         this.showConvertDialog = false;
         this.$router.push({name: 'recipe', query: { scale: this.scaleValue, system: this.systemConvertValue, unit: this.unitConvertValue }});
@@ -256,6 +286,10 @@
           return "";
         }
       }
+    },
+    
+    mounted() {
+      this.ensureTaskLists();
     }
   }
 
diff --git a/app/javascript/lib/Api.js b/app/javascript/lib/Api.js
index 79f53b0..d61c978 100644
--- a/app/javascript/lib/Api.js
+++ b/app/javascript/lib/Api.js
@@ -423,6 +423,10 @@ class Api {
 
     return this.patch(`/task_lists/${listId}/task_items/complete`, params);
   }
+  
+  addRecipeToTaskList(listId, recipeId) {
+    return this.patch(`/task_lists/${listId}/add_recipe/${recipeId}`);
+  }
 
   getAdminUserList() {
     return this.get("/admin/users");
diff --git a/app/javascript/store/index.js b/app/javascript/store/index.js
index faac798..aaf71ad 100644
--- a/app/javascript/store/index.js
+++ b/app/javascript/store/index.js
@@ -186,6 +186,14 @@ export default new Vuex.Store({
 
       return api.getTaskLists(cb)
     },
+    
+    ensureTaskLists({dispatch, state}) {
+      if (state.user && state.taskLists.length === 0) {
+        return dispatch("refreshTaskLists");
+      } else {
+        return Promise.resolve();
+      }
+    },
 
     createTaskList({commit, dispatch}, newList) {
       return api.postTaskList(newList)
diff --git a/app/models/task_list.rb b/app/models/task_list.rb
index ab29706..e0ce25f 100644
--- a/app/models/task_list.rb
+++ b/app/models/task_list.rb
@@ -9,4 +9,26 @@ class TaskList < ApplicationRecord
 
   scope :for_user, -> (user) { where(user_id: user) }
 
+  def add_recipe_ingredients(recipe, recurse_depth = 0)
+    if recurse_depth > 10
+      raise "This shouldn't be. Did you make a recipe loop?"
+    end
+    
+    recipe.recipe_ingredients.each do |ri|
+      if ri.ingredient.is_a?(Recipe)
+        add_recipe_ingredients(ri.ingredient, recurse_depth + 1)
+      else
+        item = self.task_items.detect { |i| i.name.downcase == ri.name.downcase } || TaskItem.new(name: ri.name, task_list: self)
+        quantity_str = [ri.quantity, ri.units].delete_if { |i| i.blank? }.join(' ')
+        if item.quantity.blank?
+          item.quantity = quantity_str
+        else
+          item.quantity += (', ' + quantity_str)
+        end
+        
+        item.save
+      end
+    end
+  end
+  
 end
diff --git a/app/views/home/index.html.erb b/app/views/home/index.html.erb
index 116eb9c..a9c6756 100644
--- a/app/views/home/index.html.erb
+++ b/app/views/home/index.html.erb
@@ -41,7 +41,7 @@
     }
 
     #app {
-      transition: opacity 0.25s ease-in;
+      transition: opacity 0.1s ease-in;
     }
     
     body.loading #app {
diff --git a/config/routes.rb b/config/routes.rb
index 36d3000..b007b26 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -33,6 +33,9 @@ Rails.application.routes.draw do
   end
 
   resources :task_lists, only: [:index, :show, :create, :update, :destroy] do
+    member do
+      patch 'add_recipe/:recipe_id', action: :add_recipe
+    end
     resources :task_items, only: [:create, :update] do
       collection do
         delete '/', action: :destroy, as: :destroy