internal/embedfs/web/css/output.css

@@ -489,9 +489,6 @@
   .w-20 {
     width: calc(var(--spacing) * 20);
   }
-  .w-24 {
-    width: calc(var(--spacing) * 24);
-  }
   .w-26 {
     width: calc(var(--spacing) * 26);
   }

internal/handlers/fixtures.go

@@ -2,6 +2,7 @@ package handlers
 
 import (
 	"context"
+	"fmt"
 	"net/http"
 	"strconv"
 
@@ -69,6 +70,7 @@ func UpdateFixtures(
 		leagueShortName := getter.String("league_short_name").TrimSpace().Required().Value
 		allocations := getter.GetMaps("allocations")
 		if !getter.ValidateAndNotify(s, w, r) {
+			w.WriteHeader(http.StatusBadRequest)
 			return
 		}
 		updates, err := mapUpdates(allocations)
@@ -91,6 +93,7 @@ func UpdateFixtures(
 			}
 			var valid bool
 			fixtures, valid = updateFixtures(fixtures, updates)
+			fmt.Println(len(fixtures))
 			if !valid {
 				notify.Warn(s, w, r, "Invalid game weeks", "A game week is missing or has no games", nil)
 				return false, nil
@@ -158,11 +161,28 @@ func updateFixtures(fixtures []*db.Fixture, updates map[int]int) ([]*db.Fixture,
 	gameWeeks := map[int]int{}
 	for _, fixture := range fixtures {
 		if gameWeek, exists := updates[fixture.ID]; exists {
-			fixture.GameWeek = &gameWeek
+			var newValue *int
+			var oldValue int
+			if fixture.GameWeek != nil {
+				oldValue = *fixture.GameWeek
+			} else {
+				oldValue = 0
+			}
+			if gameWeek == 0 {
+				newValue = nil
+			} else {
+				newValue = &gameWeek
+			}
+			if gameWeek != oldValue {
+				fixture.GameWeek = newValue
 				updated = append(updated, fixture)
 			}
+			// fuck i hate pointers sometimes
+		}
+		if fixture.GameWeek != nil {
 			gameWeeks[*fixture.GameWeek]++
 		}
+	}
 	for i := range len(gameWeeks) {
 		count, exists := gameWeeks[i+1]
 		if !exists || count < 1 {

internal/validation/forms.go

@@ -3,7 +3,6 @@ package validation
 import (
 	"net/http"
 	"regexp"
-	"strconv"
 	"strings"
 
 	"git.haelnorr.com/h/golib/hws"
@@ -32,18 +31,24 @@ func (f *FormGetter) GetList(key string) []string {
 }
 
 func (f *FormGetter) GetMaps(key string) []map[string]string {
-	var result []map[string]string
-	for key, values := range f.r.Form {
-		re := regexp.MustCompile(key + "\\[([0-9]+)\\]\\[([a-zA-Z]+)\\]")
-		matches := re.FindStringSubmatch(key)
+	results := map[string]map[string]string{}
+	re := regexp.MustCompile(key + "\\[([0-9]+)\\]\\[([a-zA-Z_]+)\\]")
+	for k, v := range f.r.Form {
+		matches := re.FindStringSubmatch(k)
 		if len(matches) >= 3 {
-			index, _ := strconv.Atoi(matches[1])
-			for index >= len(result) {
-				result = append(result, map[string]string{})
+			realKey := matches[1]
+			field := matches[2]
+			value := strings.Join(v, ",")
+			if _, exists := results[realKey]; !exists {
+				results[realKey] = map[string]string{}
 			}
-			result[index][matches[2]] = values[0]
+			results[realKey][field] = value
 		}
 	}
+	result := []map[string]string{}
+	for _, v := range results {
+		result = append(result, v)
+	}
 	return result
 }
 

internal/view/seasonsview/season_league_fixtures_manage.templ

@@ -60,18 +60,19 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 		<div class="flex items-center gap-3 mb-6">
 			<!-- Generate -->
 			<div class="flex items-center gap-2">
-				<input
-					type="number"
+				<label class="text-sm text-subtext0">Round:</label>
+				<select
 					x-model.number="generateRounds"
-					min="1"
-					max="20"
-					placeholder="Rounds"
-					class="w-24 py-2 px-3 rounded-lg text-sm bg-base border-2 border-overlay0
+					class="py-2 px-3 rounded-lg text-sm bg-base border-2 border-overlay0
 						focus:border-blue outline-none text-text"
-				/>
+				>
+					<template x-for="r in availableRounds" :key="r">
+						<option :value="r" x-text="r"></option>
+					</template>
+				</select>
 				<button
 					@click="generate()"
-					:disabled="isGenerating || generateRounds < 1"
+					:disabled="isGenerating || availableRounds.length === 0"
 					class="rounded-lg px-4 py-2 hover:cursor-pointer text-center text-sm
 						bg-blue hover:bg-blue/80 text-mantle transition
 						disabled:bg-blue/40 disabled:cursor-not-allowed"
@@ -79,32 +80,40 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 					<span x-text="isGenerating ? 'Generating...' : 'Generate Round'"></span>
 				</button>
 			</div>
-			<!-- Clear All -->
+			<!-- Delete All -->
 			<button
 				x-show="allFixtures.length > 0"
-				@click="clearAll()"
+				@click="deleteAll()"
 				class="rounded-lg px-4 py-2 hover:cursor-pointer text-center text-sm
 					bg-red hover:bg-red/80 text-mantle transition"
 			>
-				Clear All
+				Delete All
+			</button>
+			<!-- Save / Reset -->
+			<div x-show="unsavedChanges" class="flex items-center gap-2 ml-auto">
+				<span
+					x-show="!canSave()"
+					class="text-yellow text-xs"
+				>
+					All game weeks must have at least 1 fixture
+				</span>
+				<button
+					@click="reset()"
+					class="rounded-lg px-4 py-2 hover:cursor-pointer text-center text-sm
+						bg-surface1 hover:bg-surface2 text-text transition"
+				>
+					Reset
 				</button>
-			<!-- Save -->
 				<button
-				x-show="unsavedChanges"
 					@click="save()"
 					:disabled="isSaving || !canSave()"
 					class="rounded-lg px-4 py-2 hover:cursor-pointer text-center text-sm
 						bg-green hover:bg-green/75 text-mantle transition
-					disabled:bg-green/40 disabled:cursor-not-allowed ml-auto"
+						disabled:bg-green/40 disabled:cursor-not-allowed"
 				>
 					<span x-text="isSaving ? 'Saving...' : 'Save'"></span>
 				</button>
-			<span
-				x-show="unsavedChanges && !canSave()"
-				class="text-yellow text-xs ml-2"
-			>
-				All game weeks must have at least 1 fixture
-			</span>
+			</div>
 		</div>
 		<!-- Main content panels -->
 		<div class="grid grid-cols-1 lg:grid-cols-3 gap-4">
@@ -114,7 +123,7 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 				<div x-show="selectedGameWeek === null">
 					<h3 class="text-lg font-bold text-text mb-4">Game Weeks</h3>
 					<div x-show="allGameWeekNumbers.length === 0" class="text-subtext0 text-sm italic mb-4">
-						No game weeks yet. Add one to start allocating fixtures.
+						No game weeks yet. Generate fixtures to get started.
 					</div>
 					<div class="space-y-2">
 						<template x-for="week in allGameWeekNumbers" :key="week">
@@ -271,9 +280,12 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 		</div>
 		<!-- Alpine.js component -->
 		<script>
-			document.addEventListener('alpine:init', () => {
-				Alpine.data('fixturesManager', (initialFixtures, seasonShortName, leagueShortName) => ({
+			document.addEventListener("alpine:init", () => {
+				Alpine.data(
+					"fixturesManager",
+					(initialFixtures, seasonShortName, leagueShortName) => ({
 						allFixtures: initialFixtures || [],
+						_initialFixtures: JSON.parse(JSON.stringify(initialFixtures || [])),
 						seasonShortName: seasonShortName,
 						leagueShortName: leagueShortName,
 
@@ -282,7 +294,12 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 						unsavedChanges: false,
 						isSaving: false,
 						isGenerating: false,
-					generateRounds: 1,
+						generateRounds: null,
+
+						init() {
+							this.generateRounds =
+								this.availableRounds.length > 0 ? this.availableRounds[0] : 1;
+						},
 
 						// Drag state
 						draggedFixture: null,
@@ -291,7 +308,7 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 						// Computed
 						get unallocatedFixtures() {
 							return this.allFixtures
-							.filter(f => f.gameWeek === null)
+								.filter((f) => f.gameWeek === null)
 								.sort((a, b) => a.round - b.round || a.id - b.id);
 						},
 
@@ -309,17 +326,41 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 							return [...weeks].sort((a, b) => a - b);
 						},
 
+						get existingRounds() {
+							const rounds = new Set();
+							for (const f of this.allFixtures) {
+								rounds.add(f.round);
+							}
+							return rounds;
+						},
+
+						get availableRounds() {
+							const taken = this.existingRounds;
+							const maxTaken = taken.size > 0 ? Math.max(...taken) : 0;
+							const limit = maxTaken + 5;
+							const available = [];
+							for (let i = 1; i <= limit; i++) {
+								if (!taken.has(i)) available.push(i);
+							}
+							return available;
+						},
+
 						// Track empty weeks that user created
-					_emptyWeeks: [],
+						// Default to [1] if no fixtures have game weeks assigned
+						_emptyWeeks: (initialFixtures || []).some(
+							(f) => f.gameWeek !== null,
+						)
+							? []
+							: [1],
 
 						getGameWeekFixtures(week) {
 							return this.allFixtures
-							.filter(f => f.gameWeek === week)
+								.filter((f) => f.gameWeek === week)
 								.sort((a, b) => a.round - b.round || a.id - b.id);
 						},
 
 						getFixtureCount(week) {
-						return this.allFixtures.filter(f => f.gameWeek === week).length;
+							return this.allFixtures.filter((f) => f.gameWeek === week).length;
 						},
 
 						getPreview(week) {
@@ -336,7 +377,7 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 
 						deleteGameWeek(week) {
 							if (this.getFixtureCount(week) > 0) return;
-						this._emptyWeeks = this._emptyWeeks.filter(w => w !== week);
+							this._emptyWeeks = this._emptyWeeks.filter((w) => w !== week);
 							if (this.selectedGameWeek === week) {
 								this.selectedGameWeek = null;
 							}
@@ -354,8 +395,8 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 						// Drag and drop
 						onDragStart(event, fixture) {
 							this.draggedFixture = fixture;
-						event.dataTransfer.effectAllowed = 'move';
-						event.dataTransfer.setData('text/plain', fixture.id);
+							event.dataTransfer.effectAllowed = "move";
+							event.dataTransfer.setData("text/plain", fixture.id);
 						},
 
 						onDragEnd() {
@@ -366,15 +407,17 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 						onDrop(target) {
 							if (!this.draggedFixture) return;
 
-						const fixture = this.allFixtures.find(f => f.id === this.draggedFixture.id);
+							const fixture = this.allFixtures.find(
+								(f) => f.id === this.draggedFixture.id,
+							);
 							if (!fixture) return;
 
-						if (target === 'unallocated') {
+							if (target === "unallocated") {
 								fixture.gameWeek = null;
 							} else {
 								fixture.gameWeek = target;
 								// Remove from empty weeks if it now has fixtures
-							this._emptyWeeks = this._emptyWeeks.filter(w => w !== target);
+								this._emptyWeeks = this._emptyWeeks.filter((w) => w !== target);
 							}
 
 							this.unsavedChanges = true;
@@ -383,7 +426,7 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 						},
 
 						unallocateFixture(fixture) {
-						const f = this.allFixtures.find(ff => ff.id === fixture.id);
+							const f = this.allFixtures.find((ff) => ff.id === fixture.id);
 							if (f) {
 								const oldWeek = f.gameWeek;
 								f.gameWeek = null;
@@ -405,21 +448,36 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 							return true;
 						},
 
+						reset() {
+							this.allFixtures = JSON.parse(
+								JSON.stringify(this._initialFixtures),
+							);
+							this._emptyWeeks = this._initialFixtures.some(
+								(f) => f.gameWeek !== null,
+							)
+								? []
+								: [1];
+							this.selectedGameWeek = null;
+							this.unsavedChanges = false;
+						},
+
 						// Server actions
 						generate() {
 							if (this.generateRounds < 1) return;
 							this.isGenerating = true;
 
 							const form = new FormData();
-						form.append('season_short_name', this.seasonShortName);
-						form.append('league_short_name', this.leagueShortName);
-						form.append('round', this.generateRounds);
+							form.append("season_short_name", this.seasonShortName);
+							form.append("league_short_name", this.leagueShortName);
+							form.append("round", this.generateRounds);
 
-						htmx.ajax('POST', '/fixtures/generate', {
-							target: '#manage-fixtures-content',
-							swap: 'outerHTML',
-							values: Object.fromEntries(form)
-						}).finally(() => {
+							htmx
+								.ajax("POST", "/fixtures/generate", {
+									target: "#manage-fixtures-content",
+									swap: "outerHTML",
+									values: Object.fromEntries(form),
+								})
+								.finally(() => {
 									this.isGenerating = false;
 								});
 						},
@@ -428,49 +486,66 @@ templ SeasonLeagueManageFixtures(season *db.Season, league *db.League, fixtures
 							if (!this.canSave()) return;
 							this.isSaving = true;
 
-						const form = new FormData();
-						form.append('season_short_name', this.seasonShortName);
-						form.append('league_short_name', this.leagueShortName);
+							const params = new URLSearchParams();
+							params.append("season_short_name", this.seasonShortName);
+							params.append("league_short_name", this.leagueShortName);
 
 							this.allFixtures.forEach((f, i) => {
-							form.append('allocations[' + i + '][id]', f.id);
-							form.append('allocations[' + i + '][game_week]', f.gameWeek !== null ? f.gameWeek : 0);
+								params.append("allocations[" + i + "][id]", f.id);
+								params.append(
+									"allocations[" + i + "][game_week]",
+									f.gameWeek !== null ? f.gameWeek : 0,
+								);
 							});
 
-						fetch('/fixtures/update-game-weeks', {
-							method: 'POST',
-							body: form
-						}).then(response => {
+							fetch("/fixtures/update-game-weeks", {
+								method: "POST",
+								headers: {
+									"Content-Type": "application/x-www-form-urlencoded",
+								},
+								body: params.toString(),
+							})
+								.then((response) => {
 									if (response.ok) {
 										this.unsavedChanges = false;
 									}
 									this.isSaving = false;
-						}).catch(() => {
+								})
+								.catch(() => {
 									this.isSaving = false;
 								});
 						},
 
-					clearAll() {
+						deleteAll() {
 							const seasonShort = this.seasonShortName;
 							const leagueShort = this.leagueShortName;
 
-						window.dispatchEvent(new CustomEvent('confirm-action', {
+							window.dispatchEvent(
+								new CustomEvent("confirm-action", {
 									detail: {
-								title: 'Clear All Fixtures',
-								message: 'This will delete all fixtures for this league. This action cannot be undone.',
+										title: "Delete All Fixtures",
+										message:
+											"This will delete all fixtures for this league. This action cannot be undone.",
 										action: () => {
-									htmx.ajax('DELETE',
-										'/seasons/' + seasonShort + '/leagues/' + leagueShort + '/fixtures',
+											htmx.ajax(
+												"DELETE",
+												"/seasons/" +
+													seasonShort +
+													"/leagues/" +
+													leagueShort +
+													"/fixtures",
 												{
-											target: '#manage-fixtures-content',
-											swap: 'outerHTML'
-										}
-									);
-								}
-							}
-						}));
+													target: "#manage-fixtures-content",
+													swap: "outerHTML",
 												},
-				}));
+											);
+										},
+									},
+								}),
+							);
+						},
+					}),
+				);
 			});
 		</script>
 	</div>