Refactor variable declarations to use let instead

of var
This commit is contained in:
Andrew Malsbury
2023-11-17 18:30:43 +00:00
parent feea83d380
commit 955b7cc53b
2 changed files with 6 additions and 6 deletions

View File

@@ -58,7 +58,7 @@
} }
function loadOpenTickets() { function loadOpenTickets() {
var openTicketsContainer = document.getElementById('openTicketsContainer'); let openTicketsContainer = document.getElementById('openTicketsContainer');
openTicketsContainer.innerHTML = ''; // Clear existing content openTicketsContainer.innerHTML = ''; // Clear existing content
for (let i = 0; i < localStorage.length; i++) { for (let i = 0; i < localStorage.length; i++) {
@@ -68,7 +68,7 @@
let ticketID = key.replace("ticket-timer-running-", ""); let ticketID = key.replace("ticket-timer-running-", "");
let isRunning = JSON.parse(localStorage.getItem(key)); let isRunning = JSON.parse(localStorage.getItem(key));
var ticketDiv = document.createElement('div'); let ticketDiv = document.createElement('div');
ticketDiv.classList.add('card', 'card-outline', 'mb-3'); ticketDiv.classList.add('card', 'card-outline', 'mb-3');
// Add class based on ticket status // Add class based on ticket status
ticketDiv.classList.add(isRunning ? 'card-info' : 'card-warning'); ticketDiv.classList.add(isRunning ? 'card-info' : 'card-warning');
@@ -95,7 +95,7 @@
} }
function updateRunningTickets() { function updateRunningTickets() {
var runningTickets = document.querySelectorAll('[id^="ticket-"]'); let runningTickets = document.querySelectorAll('[id^="ticket-"]');
runningTickets.forEach(ticket => { runningTickets.forEach(ticket => {
let ticketID = ticket.id.replace("ticket-", ""); let ticketID = ticket.id.replace("ticket-", "");
let isRunning = JSON.parse(localStorage.getItem("ticket-timer-running-" + ticketID)); let isRunning = JSON.parse(localStorage.getItem("ticket-timer-running-" + ticketID));
@@ -135,7 +135,7 @@
// Add event listener to modal // Add event listener to modal
document.addEventListener('DOMContentLoaded', function() { document.addEventListener('DOMContentLoaded', function() {
var modal = document.getElementById('openTicketsModal'); let modal = document.getElementById('openTicketsModal');
if (modal) { if (modal) {
$('#openTicketsModal').on('show.bs.modal', loadOpenTickets); $('#openTicketsModal').on('show.bs.modal', loadOpenTickets);
} }

View File

@@ -128,7 +128,7 @@
function updateRunningTicketsCount() { function updateRunningTicketsCount() {
var runningTickets = parseInt(document.getElementById('runningTicketsCount').innerText, 10); let runningTickets = parseInt(document.getElementById('runningTicketsCount').innerText, 10);
if (!isPaused && timerInterval) { if (!isPaused && timerInterval) {
runningTickets += 1; runningTickets += 1;
@@ -182,7 +182,7 @@
// Wait for other synchronous actions (if any) to complete before resetting the timer. // Wait for other synchronous actions (if any) to complete before resetting the timer.
setTimeout(clearTimeStorage, 100); // 100ms delay should suffice, but you can adjust as needed. setTimeout(clearTimeStorage, 100); // 100ms delay should suffice, but you can adjust as needed.
}); });
try { try {
displayTime(); displayTime();
if (!localStorage.getItem(getLocalStorageKey("startTime")) && !localStorage.getItem(getLocalStorageKey("pausedTime"))) { if (!localStorage.getItem(getLocalStorageKey("startTime")) && !localStorage.getItem(getLocalStorageKey("pausedTime"))) {