From 594f5c6e4bd8a674636bf53fb735a61efa1a5ef3 Mon Sep 17 00:00:00 2001
From: "Christopher C. Wells" <git@chris-wells.net>
Date: Sat, 27 Mar 2021 12:33:36 -0700
Subject: [PATCH] Ensure ingredients list always has a weight entry

---
 resources/views/recipes/edit.blade.php | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/resources/views/recipes/edit.blade.php b/resources/views/recipes/edit.blade.php
index 6caa372..a2e5a01 100644
--- a/resources/views/recipes/edit.blade.php
+++ b/resources/views/recipes/edit.blade.php
@@ -133,7 +133,7 @@
                     @include('recipes.partials.separator-input', $item)
                 @endif
             @empty
-                @include('recipes.partials.ingredient-input')
+                @include('recipes.partials.ingredient-input', ['weight' => 0])
             @endforelse
             <div class="templates hidden">
                 <div class="ingredient-template">
@@ -255,10 +255,17 @@
                     const newNode = template.cloneNode(true).firstElementChild;
 
                     // Set weight based on previous sibling.
-                    const lastWeight = templates.previousElementSibling.querySelector('input[name$="[weight][]"]');
-                    if (lastWeight && lastWeight.value) {
-                        newNode.querySelector('input[name$="[weight][]"]').value = Number.parseInt(lastWeight.value) + 1;
+                    const weightField = newNode.querySelector('input[name$="[weight][]"]');
+                    if (templates.previousElementSibling) {
+                        const lastWeight = templates.previousElementSibling.querySelector('input[name$="[weight][]"]');
+                        if (lastWeight && lastWeight.value) {
+                            weightField.value = Number.parseInt(lastWeight.value) + 1;
+                        }
                     }
+                    else {
+                        weightField.value = 0;
+                    }
+
 
                     // Insert new node before templates.
                     $el.insertBefore(newNode, templates);