From 86b61dba30cad54d02994ee3ee18e0a787d949ad Mon Sep 17 00:00:00 2001
From: "Christopher C. Wells" <git@chris-wells.net>
Date: Sat, 15 May 2021 21:25:01 -0700
Subject: [PATCH] Handle day updates during Goal edit

---
 app/Http/Controllers/GoalController.php | 5 ++++-
 resources/views/goals/edit.blade.php    | 8 ++++----
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/app/Http/Controllers/GoalController.php b/app/Http/Controllers/GoalController.php
index a714a4e..407f361 100644
--- a/app/Http/Controllers/GoalController.php
+++ b/app/Http/Controllers/GoalController.php
@@ -62,10 +62,13 @@ class GoalController extends Controller
         if (isset($attributes['days'])) {
             $attributes['days'] = array_sum($attributes['days']);
         }
+        else if (!empty($goal->days)) {
+            $attributes['days'] = null;
+        }
         $goal->fill($attributes)->user()->associate(Auth::user());
         $goal->save();
         session()->flash('message', "Goal updated!");
-        return redirect()->route('goals.show', $goal);
+        return redirect()->route('goals.index');
     }
 
     /**
diff --git a/resources/views/goals/edit.blade.php b/resources/views/goals/edit.blade.php
index b9a1f16..52f5832 100644
--- a/resources/views/goals/edit.blade.php
+++ b/resources/views/goals/edit.blade.php
@@ -22,12 +22,12 @@
             </div>
             <div class="flex flex-col space-y-4 md:flex-row md:space-x-4 md:space-y-0">
                 <!-- Days of the week -->
-                <fieldset class="flex-auto">
+                <fieldset>
                     <legend class="block font-medium text-sm text-gray-700">Days of the week</legend>
-                    <div class="inline-flex justify-between divide-x border rounded-md shadow-sm border-gray-300">
+                    <div class="flex flex-col justify-content divide-x border rounded-md shadow-sm border-gray-300 md:inline-flex md:flex-row">
                         @foreach(\App\Models\Goal::days() as $day)
-                            <x-inputs.label class="inline-flex items-center p-2">
-                                <x-inputs.input type="checkbox" name="days[]" :value="$day['value']" />
+                            <x-inputs.label class="inline-flex justify-center p-2">
+                                <x-inputs.input type="checkbox" name="days[]" :value="$day['value']" :checked="($goal->days & $day['value']) != 0" />
                                 <span class="ml-2">{{ \Illuminate\Support\Str::ucfirst($day['label']) }}</span>
                             </x-inputs.label>
                         @endforeach