From 075665c588989ed0decdfb20d83f32b33eed4639 Mon Sep 17 00:00:00 2001
From: Marius Peter <marius.peter@tutanota.com>
Date: Fri, 29 Aug 2025 14:22:37 +0200
Subject: Properly implement bed and raft management logic.

---
 app/views/rafts/_form.html.erb  |  23 +++++++++
 app/views/rafts/edit.html.erb   |   5 ++
 app/views/rafts/editor.html.erb | 108 ----------------------------------------
 3 files changed, 28 insertions(+), 108 deletions(-)
 create mode 100644 app/views/rafts/_form.html.erb
 create mode 100644 app/views/rafts/edit.html.erb
 delete mode 100644 app/views/rafts/editor.html.erb

(limited to 'app/views/rafts')

diff --git a/app/views/rafts/_form.html.erb b/app/views/rafts/_form.html.erb
new file mode 100644
index 0000000..7b5d382
--- /dev/null
+++ b/app/views/rafts/_form.html.erb
@@ -0,0 +1,23 @@
+<%= form_with(model: raft) do |form| %>
+    <% if raft.errors.any? %>
+	<div style="color: red">
+	    <h2><%= pluralize(bed.errors.count, "error") %> prohibited this raft from being saved:</h2>
+
+	    <ul>
+		<% raft.errors.each do |error| %>
+		    <li><%= error.full_message %></li>
+		<% end %>
+	    </ul>
+	</div>
+    <% end %>
+
+    <div class="my-3">
+	<label class="form-label">Crop</label>
+	<%= form.collection_select :crop_id, @crops, :id, :name,
+	   { include_blank: "Unassigned" }, { class: "form-select" } %>
+    </div>
+
+    <div>
+	<%= form.submit class: "btn btn-primary" %>
+    </div>
+<% end %>
diff --git a/app/views/rafts/edit.html.erb b/app/views/rafts/edit.html.erb
new file mode 100644
index 0000000..bf975dc
--- /dev/null
+++ b/app/views/rafts/edit.html.erb
@@ -0,0 +1,5 @@
+<% content_for :title, "Editing bed #{@raft.bed.location}, raft #{@raft.location}" %>
+
+<h1 class="display-1">Editing bed <%= @raft.bed.location %>, raft <%= @raft.location %></h1>
+
+<%= render "form", raft: @raft %>
diff --git a/app/views/rafts/editor.html.erb b/app/views/rafts/editor.html.erb
deleted file mode 100644
index cde56d8..0000000
--- a/app/views/rafts/editor.html.erb
+++ /dev/null
@@ -1,108 +0,0 @@
-<div class="d-flex justify-content-between align-items-center mb-3">
-    <h1 class="display-1">Rafts editor</h1>
-    <%= link_to "Back to dashboard", root_path, class: "btn btn-outline-secondary" %>
-</div>
-
-<!-- Bulk assign all rafts -->
-<div class="card">
-    <div class="card-header">Assign ALL rafts</div>
-    <div class="card-body">
-	<%= form_with url: assign_all_rafts_path, method: :patch, class: "row g-2", data: { turbo: false } do %>
-	    <div class="col-12 col-md-6">
-		<select class="form-select" name="crop_id">
-		    <option value="">— Unassigned (clear) —</option>
-		    <% @crops.each do |crop| %>
-			<option value="<%= crop.id %>"><%= crop.name %></option>
-		    <% end %>
-		</select>
-	    </div>
-	    <div class="col-12 col-md-auto">
-		<button class="btn btn-primary">Apply to all</button>
-	    </div>
-	<% end %>
-    </div>
-</div>
-
-<div class="table-responsive">
-    <table class="table table-sm table-striped text-center align-middle my-3">
-	<thead>
-	    <tr>
-		<th class="text-start">Bed</th>
-		<% max_cols = @beds.map { |b| b.rafts.count }.max %>
-		<% (1..max_cols).each do |i| %>
-		    <th>R<%= i %></th>
-		<% end %>
-	    </tr>
-	</thead>
-	<tbody>
-	    <% @beds.each do |bed| %>
-		<tr>
-		    <th><%= bed.location %></th>
-		    <% bed.rafts.order(:location).each do |raft| %>
-			<td>
-			    <% if raft.crop %>
-				<%= raft.crop.name %>
-			    <% else %>
-				—
-			    <% end %>
-			</td>
-		    <% end %>
-		</tr>
-	    <% end %>
-	</tbody>
-    </table>
-</div>
-
-<!-- Per-bed tables -->
-<% @beds.each do |bed| %>
-    <div class="card mb-3" id="bed-<%= bed.id %>">
-	<div class="card-header d-flex flex-wrap gap-2 align-items-center">
-	    <span class="me-auto">Bed <strong>#<%= bed.location %></strong></span>
-	    <%= form_with url: assign_bed_rafts_path, method: :patch, class: "d-flex gap-2 align-items-center", data: { turbo: false } do %>
-		<input type="hidden" name="bed_id" value="<%= bed.id %>">
-		<select class="form-select" name="crop_id">
-		    <option value="">— Unassigned (clear) —</option>
-		    <% @crops.each do |crop| %>
-			<option value="<%= crop.id %>"><%= crop.name %></option>
-		    <% end %>
-		</select>
-		<button class="btn btn-primary">Apply</button>
-	    <% end %>
-	</div>
-	<div class="card-body p-0">
-	    <div class="table-responsive">
-		<table class="table table-sm mb-0 align-middle">
-		    <thead class="table-light">
-			<tr>
-			    <th class="text-nowrap text-center">Raft</th>
-			    <th>Crop</th>
-			    <th class="text-end">Actions</th>
-			</tr>
-		    </thead>
-		    <tbody>
-			<% bed.rafts.order(:location).each do |raft| %>
-			    <tr>
-				<td class="text-center"><strong><%= raft.location %></strong></td>
-				<td style="max-width: 280px;">
-				    <%= form_with url: assign_one_raft_path(raft), method: :patch, class: "d-flex gap-2", data: { turbo: false } do %>
-					<select class="form-select" name="crop_id">
-					    <option value="">— Unassigned (clear) —</option>
-					    <% @crops.each do |crop| %>
-						<option value="<%= crop.id %>" <%= "selected" if raft.crop_id == crop.id %>><%= crop.name %></option>
-					    <% end %>
-					</select>
-					<button class="btn btn-outline-primary">Save</button>
-				    <% end %>
-				</td>
-				<td class="text-end">
-				    <%= button_to "Clear", assign_one_raft_path(raft, crop_id: ""), method: :patch,
-						  form: { data: { turbo: false } }, class: "btn btn-outline-secondary btn-sm" %>
-				</td>
-			    </tr>
-			<% end %>
-		    </tbody>
-		</table>
-	    </div>
-	</div>
-    </div>
-<% end %>
-- 
cgit v1.2.3