Address Issue #1381

geoffhumphrey · Dec 12, 2022 · 3434b56 · 3434b56
1 parent 02bc3dd
commit 3434b56
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/admin/judging_locations.admin.php b/admin/judging_locations.admin.php
@@ -751,12 +751,12 @@
         <div class="radio">
             <!-- Input Here -->
             <label>
-                <input type="radio" name="judgingLocType" value="0" id="judgingLocType_0"  <?php if (($section == "step5") || ($action == "add")) echo "CHECKED"; if ((isset($row_judging['judgingLocType'])) && ($section != "step5") && ($row_judging['judgingLocType'] == "0")) echo "CHECKED";  ?> /> Traditional <small>(typically a single day in a central location)</small>
+                <input type="radio" name="judgingLocType" value="0" id="judgingLocType_0"  <?php if (($section == "step5") || ($action == "add")) echo "CHECKED"; if ((isset($row_judging['judgingLocType'])) && ($section != "step5") && ($row_judging['judgingLocType'] == "0")) echo "CHECKED";  ?> required /> Traditional <small>(typically a single day in a central location)</small>
             </label>
         </div>
         <div class="radio">
             <label>
-                <input type="radio" name="judgingLocType" value="1" id="judgingLocType_1" <?php if ((isset($row_judging['judgingLocType'])) && ($section != "step5") && ($row_judging['judgingLocType'] == "1")) echo "CHECKED"; ?>/> Distributed <small>(multi-day and/or multi-location)</small>
+                <input type="radio" name="judgingLocType" value="1" id="judgingLocType_1" <?php if ((isset($row_judging['judgingLocType'])) && ($section != "step5") && ($row_judging['judgingLocType'] == "1")) echo "CHECKED"; ?> required /> Distributed <small>(multi-day and/or multi-location)</small>
             </label>
         </div>
         <span class="help-block">Indicate whether judge teams in this session will be evaluating entries at a single, designated location, typically collectively, or over a series of days in various locations. For example, choose <em>Distributed</em> if judges will be evaluating entries virtually - synchronously or asynchronously - or if locations will be ad-hoc, such as in a judge team member home.</span>

diff --git a/includes/process/process_archive_hosted.inc.php b/includes/process/process_archive_hosted.inc.php
@@ -37,7 +37,8 @@
 		'brewerJudge' => 'N',
 		'brewerSteward' => 'N',
 		'brewerJudgeLocation' => NULL,
-		'brewerStewardLocation' => NULL
+		'brewerStewardLocation' => NULL,
+		'brewerDropOff' => '999'
 	);
 	$result = $db_conn->update ($update_table, $data);
 	if (!$result) {

diff --git a/includes/process/process_judging_locations.inc.php b/includes/process/process_judging_locations.inc.php
@@ -22,6 +22,8 @@
 	$judgingLocation = sterilize($judgingLocation);
 	$judgingLocType = sterilize($_POST['judgingLocType']);
 
+	if (empty($judgingLocType)) $judgingLocType = 0;
+
 	$judgingRounds = "";
 	if (isset($_POST['judgingRounds'])) $judgingRounds = sterilize($_POST['judgingRounds']);
 
@@ -30,7 +32,7 @@
 
 	$update_table = $prefix."judging_locations";
 	$data = array(
-		'judgingLocType' => blank_to_null($judgingLocType),
+		'judgingLocType' => $judgingLocType,
 		'judgingDate' => blank_to_null($judgingDate),
 		'judgingDateEnd' => blank_to_null($judgingDateEnd),
 		'judgingLocation' => blank_to_null($judgingLocation),