Prevent negative time entry

karmenlu · Mar 1, 2019 · c6bd2a4 · c6bd2a4
1 parent 09d9661
commit c6bd2a4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/tasks1/lib/tasks1_web/controllers/task_controller.ex b/tasks1/lib/tasks1_web/controllers/task_controller.ex
@@ -56,7 +56,10 @@ defmodule Tasks1Web.TaskController do
     task = Tasks.get_task!(id)
 
     task_params = Map.put(task_params, "doer_id", _parse_doer(Map.get(task_params, "doer_id")))
-    task_params = Map.put(task_params, "timeSpent", closestFifteen(elem(Integer.parse(Map.get(task_params, "timeSpent")), 0)))
+    if Map.get(task_params, "timeSpent", nil) != nil do
+      task_params = Map.put(task_params, "timeSpent", closestFifteen(elem(Integer.parse(Map.get(task_params, "timeSpent")), 0)))
+    else
+    end
     case Tasks.update_task(task, task_params) do
       {:ok, task} ->
         conn

diff --git a/tasks1/lib/tasks1_web/templates/task/form.html.eex b/tasks1/lib/tasks1_web/templates/task/form.html.eex
@@ -26,7 +26,7 @@
 
     <div class="form-group">
       <%= label f, :timeSpent %>
-      <%= number_input f, :timeSpent, step: 15 %>
+      <%= number_input f, :timeSpent, step: 15, min: 0 %>
       <%= error_tag f, :timeSpent %>
     </div>
   <% end %>