From 491c34cc9318a9e09e7d27862be8b255aecfdb2d Mon Sep 17 00:00:00 2001
From: Taylor Courage <taylorjamescourage@gmail.com>
Date: Fri, 7 Mar 2025 22:20:28 -0500
Subject: [PATCH] Added "all" to years display to show ALL results

---
 display/division_results.php | 12 +++++++++---
 display/general_results.php  |  1 +
 scripts/results.js           | 14 +++++++++++---
 3 files changed, 21 insertions(+), 6 deletions(-)

diff --git a/display/division_results.php b/display/division_results.php
index abdd2d0..4fc5395 100644
--- a/display/division_results.php
+++ b/display/division_results.php
@@ -18,12 +18,18 @@ try {  // Try opening the SQL database connection
     } else {
         $getMonth = "MONTH(tournamentDate)=\"" . $_GET["month"] . "\" AND ";
     }
-    // Grab year and division
-    $year = $_GET["year"];
+    // Grab year similar to above
+    if ($_GET["year"] == "all") {
+        $getYear = "";
+    } else {
+        $getYear = "YEAR(tournamentDate)=\"" . $_GET["year"] . "\" AND ";
+    }
+
+    // Grab division
     $division = $_GET["division"];
 
     // Select all the winners from the table where the month, year, and division all match
-    $sqlGetTopWinnersList = $conn->prepare("SELECT winner1,winner2,winner3,winner4 FROM " . $tournamentDataTableName . " WHERE $getMonth YEAR(tournamentDate)=\"" . $year . "\" AND tournamentDivision=\"" . $division . "\"");
+    $sqlGetTopWinnersList = $conn->prepare("SELECT winner1,winner2,winner3,winner4 FROM " . $tournamentDataTableName . " WHERE $getMonth $getYear tournamentDivision=\"" . $division . "\"");
     $sqlGetTopWinnersList->execute();
 
     // Fetch the results
diff --git a/display/general_results.php b/display/general_results.php
index 9775d50..078f1ed 100644
--- a/display/general_results.php
+++ b/display/general_results.php
@@ -211,6 +211,7 @@ session_start();
                     ?>
                 </select>
                 <select size="1" name="year" id="year" onchange="refreshDisplay()">
+                <option value="all">All</option>
                     <?php
                     // This uses the years we grabbed earlier and ensures we're only showing
                     // the years we have entries for
diff --git a/scripts/results.js b/scripts/results.js
index a13898c..ebd989e 100644
--- a/scripts/results.js
+++ b/scripts/results.js
@@ -1,19 +1,27 @@
 function refreshDisplay() {
+    // Grab the division buttons by their name
     var divisionButtons = document.getElementsByName("division");
     var currentDivision = "";
 
+    // Loop through the division buttons and see which one is checked
+    // Set the current division to that option
     for (var i = 0; i < divisionButtons.length; i++) {
         if (divisionButtons[i].checked) {
             currentDivision = divisionButtons[i].value;
         }
     }
 
-    var displayDiv = document.getElementById("divisionDisplay");
+    // If we set 'years' to all, then we should show all months too. 
+    // This is effectively a "show all" setting
+    if (document.getElementById("year").value == "all") {
+        document.getElementById("month").value = "all";
+    }
 
+    // Grab the current division from the page
     document.getElementById("divisionDisplay").innerHTML = currentDivision;
-    console.log(currentDivision);
+    
 
-    var html = "";
+    var html = ""; // placeholder for webpage
     var image = ""; //get trophy image
     html += "<div class=\"divisionPanel\">";