From 656fa6c255ffa72a72a9ea5772fde85d524e985e Mon Sep 17 00:00:00 2001
From: Haelnorr <lockedonohoe@gmail.com>
Date: Sun, 15 Mar 2026 17:08:26 +1100
Subject: [PATCH] fixed some button issues

---
 internal/view/seasonsview/fixture_detail.templ   |  6 +++---
 .../view/seasonsview/fixture_review_result.templ | 16 ++++++++--------
 internal/view/seasonsview/leagues_section.templ  |  2 +-
 .../view/seasonsview/series_review_result.templ  |  2 +-
 internal/view/seasonsview/series_schedule.templ  |  6 +++---
 5 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/internal/view/seasonsview/fixture_detail.templ b/internal/view/seasonsview/fixture_detail.templ
index ce73499..72e2603 100644
--- a/internal/view/seasonsview/fixture_detail.templ
+++ b/internal/view/seasonsview/fixture_detail.templ
@@ -1205,7 +1205,7 @@ templ fixtureScheduleStatus(
 						<div class="flex justify-center mt-4">
 							<button
 								type="button"
-								@click={ "window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Withdraw Proposal', message: 'Are you sure you want to withdraw your proposed time?', action: () => htmx.ajax('POST', '/fixtures/" + fmt.Sprint(fixture.ID) + "/schedule/" + fmt.Sprint(current.ID) + "/withdraw', { swap: 'none' }) } }))" }
+								onclick={ templ.JSUnsafeFuncCall("window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Withdraw Proposal', message: 'Are you sure you want to withdraw your proposed time?', action: () => htmx.ajax('POST', '/fixtures/" + fmt.Sprint(fixture.ID) + "/schedule/" + fmt.Sprint(current.ID) + "/withdraw', { swap: 'none' }) } }))") }
 								class="px-4 py-2 bg-surface1 hover:bg-surface2 text-text rounded-lg
 									font-medium transition hover:cursor-pointer"
 							>
@@ -1432,7 +1432,7 @@ templ fixtureScheduleActions(
 							</div>
 							<button
 								type="button"
-								@click={ fmt.Sprintf("if (!$el.parentElement.querySelector('select').value) { return; } window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Postpone Fixture', message: 'Are you sure you want to postpone this fixture? The current schedule will be cleared.', action: () => htmx.ajax('POST', '/fixtures/%d/schedule/postpone', { swap: 'none', values: { reschedule_reason: $el.parentElement.querySelector('select').value } }) } }))", fixture.ID) }
+								onclick={ templ.JSUnsafeFuncCall(fmt.Sprintf("if (!this.parentElement.querySelector('select').value) { return; } window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Postpone Fixture', message: 'Are you sure you want to postpone this fixture? The current schedule will be cleared.', action: () => htmx.ajax('POST', '/fixtures/%d/schedule/postpone', { swap: 'none', values: { reschedule_reason: this.parentElement.querySelector('select').value } }) } }))", fixture.ID)) }
 								class="w-full px-4 py-2 bg-peach hover:bg-peach/80 text-mantle rounded-lg
 									font-medium transition hover:cursor-pointer"
 							>
@@ -1459,7 +1459,7 @@ templ fixtureScheduleActions(
 							</div>
 							<button
 								type="button"
-								@click={ fmt.Sprintf("if (!$el.parentElement.querySelector('select').value) { return; } window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Declare Forfeit', message: 'This action is IRREVERSIBLE. The fixture schedule will be permanently cancelled. Are you sure?', action: () => htmx.ajax('POST', '/fixtures/%d/schedule/cancel', { swap: 'none', values: { reschedule_reason: $el.parentElement.querySelector('select').value } }) } }))", fixture.ID) }
+								onclick={ templ.JSUnsafeFuncCall(fmt.Sprintf("if (!this.parentElement.querySelector('select').value) { return; } window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Declare Forfeit', message: 'This action is IRREVERSIBLE. The fixture schedule will be permanently cancelled. Are you sure?', action: () => htmx.ajax('POST', '/fixtures/%d/schedule/cancel', { swap: 'none', values: { reschedule_reason: this.parentElement.querySelector('select').value } }) } }))", fixture.ID)) }
 								class="w-full px-4 py-2 bg-red hover:bg-red/80 text-mantle rounded-lg
 									font-medium transition hover:cursor-pointer"
 							>
diff --git a/internal/view/seasonsview/fixture_review_result.templ b/internal/view/seasonsview/fixture_review_result.templ
index 2921621..66098e4 100644
--- a/internal/view/seasonsview/fixture_review_result.templ
+++ b/internal/view/seasonsview/fixture_review_result.templ
@@ -160,14 +160,14 @@ templ FixtureReviewResultPage(
 								Finalize Result
 							</button>
 						</form>
-						<button
-							type="button"
-							@click={ fmt.Sprintf("window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Discard Result', message: 'Are you sure you want to discard this result? You will need to re-upload the match logs.', action: () => htmx.ajax('POST', '/fixtures/%d/results/discard', { swap: 'none' }) } }))", fixture.ID) }
-							class="px-6 py-3 bg-red hover:bg-red/80 text-mantle rounded-lg
-								font-medium transition hover:cursor-pointer text-lg"
-						>
-							Discard & Re-upload
-						</button>
+					<button
+						type="button"
+						onclick={ templ.JSUnsafeFuncCall(fmt.Sprintf("window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Discard Result', message: 'Are you sure you want to discard this result? You will need to re-upload the match logs.', action: () => htmx.ajax('POST', '/fixtures/%d/results/discard', { swap: 'none' }) } }))", fixture.ID)) }
+						class="px-6 py-3 bg-red hover:bg-red/80 text-mantle rounded-lg
+							font-medium transition hover:cursor-pointer text-lg"
+					>
+						Discard & Re-upload
+					</button>
 					</div>
 				</div>
 			</div>
diff --git a/internal/view/seasonsview/leagues_section.templ b/internal/view/seasonsview/leagues_section.templ
index f15bc2f..7a16b8a 100644
--- a/internal/view/seasonsview/leagues_section.templ
+++ b/internal/view/seasonsview/leagues_section.templ
@@ -35,7 +35,7 @@ templ LeaguesSection(season *db.Season, allLeagues []*db.League) {
 									if canRemoveLeague {
 										<button
 											type="button"
-											@click={ "window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Remove League', message: 'Are you sure you want to remove " + league.Name + " from this season?', action: () => htmx.ajax('DELETE', '/seasons/" + season.ShortName + "/leagues/" + league.ShortName + "', { target: '#leagues-section', swap: 'outerHTML' }) } }))" }
+											onclick={ templ.JSUnsafeFuncCall("window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Remove League', message: 'Are you sure you want to remove " + league.Name + " from this season?', action: () => htmx.ajax('DELETE', '/seasons/" + season.ShortName + "/leagues/" + league.ShortName + "', { target: '#leagues-section', swap: 'outerHTML' }) } }))") }
 											class="text-red hover:text-red/75 hover:cursor-pointer ml-1"
 										>
 											<svg class="w-4 h-4" fill="none" stroke="currentColor" viewBox="0 0 24 24">
diff --git a/internal/view/seasonsview/series_review_result.templ b/internal/view/seasonsview/series_review_result.templ
index fde65c1..6aaa8c3 100644
--- a/internal/view/seasonsview/series_review_result.templ
+++ b/internal/view/seasonsview/series_review_result.templ
@@ -126,7 +126,7 @@ templ SeriesReviewResultPage(
 						</button>
 						<button
 							type="button"
-							@click={ fmt.Sprintf("window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Discard All Results', message: 'Are you sure you want to discard all uploaded results? You will need to re-upload the match logs for every game.', action: () => htmx.ajax('POST', '/series/%d/results/discard', { swap: 'none' }) } }))", series.ID) }
+							onclick={ templ.JSUnsafeFuncCall(fmt.Sprintf("window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Discard All Results', message: 'Are you sure you want to discard all uploaded results? You will need to re-upload the match logs for every game.', action: () => htmx.ajax('POST', '/series/%d/results/discard', { swap: 'none' }) } }))", series.ID)) }
 							class="px-6 py-3 bg-red hover:bg-red/80 text-mantle rounded-lg
 								font-medium transition hover:cursor-pointer text-lg"
 						>
diff --git a/internal/view/seasonsview/series_schedule.templ b/internal/view/seasonsview/series_schedule.templ
index 724451c..08041f5 100644
--- a/internal/view/seasonsview/series_schedule.templ
+++ b/internal/view/seasonsview/series_schedule.templ
@@ -98,7 +98,7 @@ templ seriesScheduleStatus(
 						<div class="flex justify-center mt-4">
 							<button
 								type="button"
-								@click={ "window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Withdraw Proposal', message: 'Are you sure you want to withdraw your proposed time?', action: () => htmx.ajax('POST', '/series/" + fmt.Sprint(series.ID) + "/schedule/" + fmt.Sprint(current.ID) + "/withdraw', { swap: 'none' }) } }))" }
+								onclick={ templ.JSUnsafeFuncCall("window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Withdraw Proposal', message: 'Are you sure you want to withdraw your proposed time?', action: () => htmx.ajax('POST', '/series/" + fmt.Sprint(series.ID) + "/schedule/" + fmt.Sprint(current.ID) + "/withdraw', { swap: 'none' }) } }))") }
 								class="px-4 py-2 bg-surface1 hover:bg-surface2 text-text rounded-lg
 									font-medium transition hover:cursor-pointer"
 							>
@@ -320,7 +320,7 @@ templ seriesScheduleActions(
 							</div>
 							<button
 								type="button"
-								@click={ fmt.Sprintf("if (!$el.parentElement.querySelector('select').value) { return; } window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Postpone Series', message: 'Are you sure you want to postpone this series? The current schedule will be cleared.', action: () => htmx.ajax('POST', '/series/%d/schedule/postpone', { swap: 'none', values: { reschedule_reason: $el.parentElement.querySelector('select').value } }) } }))", series.ID) }
+								onclick={ templ.JSUnsafeFuncCall(fmt.Sprintf("if (!this.parentElement.querySelector('select').value) { return; } window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Postpone Series', message: 'Are you sure you want to postpone this series? The current schedule will be cleared.', action: () => htmx.ajax('POST', '/series/%d/schedule/postpone', { swap: 'none', values: { reschedule_reason: this.parentElement.querySelector('select').value } }) } }))", series.ID)) }
 								class="w-full px-4 py-2 bg-peach hover:bg-peach/80 text-mantle rounded-lg
 									font-medium transition hover:cursor-pointer"
 							>
@@ -354,7 +354,7 @@ templ seriesScheduleActions(
 							</div>
 							<button
 								type="button"
-								@click={ fmt.Sprintf("const reason = $el.parentElement.querySelector('input[name=reschedule_reason]').value; if (!reason) { return; } window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Cancel Schedule', message: 'Are you sure you want to cancel this schedule?', action: () => htmx.ajax('POST', '/series/%d/schedule/cancel', { swap: 'none', values: { reschedule_reason: reason } }) } }))", series.ID) }
+								onclick={ templ.JSUnsafeFuncCall(fmt.Sprintf("var reason = this.parentElement.querySelector('input[name=reschedule_reason]').value; if (!reason) { return; } window.dispatchEvent(new CustomEvent('confirm-action', { detail: { title: 'Cancel Schedule', message: 'Are you sure you want to cancel this schedule?', action: () => htmx.ajax('POST', '/series/%d/schedule/cancel', { swap: 'none', values: { reschedule_reason: reason } }) } }))", series.ID)) }
 								class="w-full px-4 py-2 bg-red hover:bg-red/80 text-mantle rounded-lg
 									font-medium transition hover:cursor-pointer"
 							>