From 89a50d132b8bed194540c3b3e843b117b09636f4 Mon Sep 17 00:00:00 2001
From: gaazzopa <gabriella.azzopardi@cern.ch>
Date: Wed, 11 Nov 2020 20:11:12 +0100
Subject: [PATCH] Added default lunch times on select

---
 cara/apps/calculator/static/js/form.js                | 11 ++++++++---
 .../apps/calculator/templates/calculator.form.html.j2 |  8 ++++----
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/cara/apps/calculator/static/js/form.js b/cara/apps/calculator/static/js/form.js
index 9fc382ec..8d63b7b1 100644
--- a/cara/apps/calculator/static/js/form.js
+++ b/cara/apps/calculator/static/js/form.js
@@ -133,10 +133,15 @@ function require_recurrent_event(option) {
 
 function require_lunch(option) {
   $("#lunch_start").prop('required', option);
-}
-
-function require_lunch(option) {
   $("#lunch_finish").prop('required', option);
+  if (option) {
+    document.getElementById("lunch_start").value = "12:30";
+    document.getElementById("lunch_finish").value = "13:30";
+  } 
+  else {
+    document.getElementById("lunch_start").value = "";
+    document.getElementById("lunch_finish").value = "";
+  }
 }
 
 function require_mask(option) {
diff --git a/cara/apps/calculator/templates/calculator.form.html.j2 b/cara/apps/calculator/templates/calculator.form.html.j2
index 6f05e286..8e5d068b 100644
--- a/cara/apps/calculator/templates/calculator.form.html.j2
+++ b/cara/apps/calculator/templates/calculator.form.html.j2
@@ -137,14 +137,14 @@
 
     <!-- Lunch Options -->
     Lunch break:
-    <input type="radio" id="lunch_option_no" name="lunch_option" value=0 checked="checked" onclick="require_fields(this)">
+    <input type="radio" id="lunch_option_no" name="lunch_option" value=0 onclick="require_fields(this)">
       <label for="lunch_option">No</label>
-    <input type="radio" id="lunch_option_yes" name="lunch_option" value=1 onclick="require_fields(this)"></input>
+    <input type="radio" id="lunch_option_yes" name="lunch_option" value=1 checked="checked" onclick="require_fields(this)"></input>
       <label for="lunch_option">Yes</label><br>
     
     <div id="DIVlunch_break">
-        Start: <input type="time" id="lunch_start" name="lunch_start" unrequired> &nbsp;&nbsp;
-        Finish: <input type="time" id="lunch_finish" class="finish_time" name="lunch_finish" unrequired>
+        Start: <input type="time" id="lunch_start" name="lunch_start" value="12:30" required> &nbsp;&nbsp;
+        Finish: <input type="time" id="lunch_finish" class="finish_time" name="lunch_finish" value="13:30" required>
         <span id="lunch_time_error" class="red_text" hidden>Finish time must be after start</span><br>
     </div>