LotusGuild/tinker_tickets
3
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

fix: Pass selectedOption parameter to performStatusChange function

Browse Source 
Fixed scope issue where selectedOption variable was not accessible in
performStatusChange(). Updated function signature to accept selectedOption
as a parameter and updated both call sites to pass it.

Resolves error: "selectedOption is not defined" when changing ticket status.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
Jared Vititoe
2026-01-09 17:08:11 -05:00
parent 63dc2d6314
commit 1b66663307
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
assets/js/ticket.js
View File

@@ -420,7 +420,7 @@ function updateTicketStatus() {
'warning', 'warning',
() => { () => {
// User confirmed, proceed with status change // User confirmed, proceed with status change
performStatusChange(statusSelect, newStatus); performStatusChange(statusSelect, selectedOption, newStatus);
}, },
() => { () => {
// User cancelled, reset to current status // User cancelled, reset to current status
@@ -430,11 +430,11 @@ function updateTicketStatus() {
return; return;
} }
performStatusChange(statusSelect, newStatus); performStatusChange(statusSelect, selectedOption, newStatus);
} }
// Extract status change logic into reusable function // Extract status change logic into reusable function
function performStatusChange(statusSelect, newStatus) { function performStatusChange(statusSelect, selectedOption, newStatus) {
// Extract ticket ID // Extract ticket ID
let ticketId; let ticketId;
if (window.location.href.includes('?id=')) { if (window.location.href.includes('?id=')) {