From aa967fbf69f8caaa1838c2c5883c0b5b7bf1720b Mon Sep 17 00:00:00 2001
From: kisfenyo <nagyfenyvesi.viktor@gmail.com>
Date: Mon, 13 Jul 2026 09:37:05 +0200
Subject: [PATCH] =?UTF-8?q?controller:=20Felhom-offsite=20status=20card=20?=
 =?UTF-8?q?=E2=80=94=20three=20honest=20states=20from=20local=20data=20(op?=
 =?UTF-8?q?t-in=20pitch=20/=20Akt=C3=ADv-nincs-kijel=C3=B6lt=20/=20the=20r?=
 =?UTF-8?q?egular=20status=20block);=20display-only=20by=20design=20(decis?=
 =?UTF-8?q?ion=202:=20never=20a=20state=20change),=20state-tested=20incl.?=
 =?UTF-8?q?=20the=20no-form/no-button=20invariant?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Claude-Session: https://claude.ai/code/session_01GzammAMzsJTgpQHqxwM2bC
---
 controller/internal/web/backups_split_test.go | 53 +++++++++++++++++++
 .../web/templates/backups_remote.html         |  9 ++++
 2 files changed, 62 insertions(+)

diff --git a/controller/internal/web/backups_split_test.go b/controller/internal/web/backups_split_test.go
index e513238..b26fc15 100644
--- a/controller/internal/web/backups_split_test.go
+++ b/controller/internal/web/backups_split_test.go
@@ -85,6 +85,59 @@ func TestBackupsSplit_CrossPageAnchors(t *testing.T) {
 	}
 }
 
+// Felhom-offsite status card (v0.124.0, decision 2): three honest states, display-only —
+// the card itself must never carry a form or button (no state change from the card, ever).
+func TestBackupsRemote_OffsiteStatusCardStates(t *testing.T) {
+	cardOf := func(html string) string {
+		i := strings.Index(html, `id="felhom-offsite-card"`)
+		if i < 0 {
+			return ""
+		}
+		j := strings.Index(html[i:], "</div>")
+		return html[i : i+j]
+	}
+
+	// State 1: no target configured → the opt-in pitch (and no "Aktív" headline).
+	data := splitTestData()
+	data["OffboxConfigured"] = false
+	data["Offbox"] = nil
+	data["OffboxToggledCount"] = 0
+	html := renderBackupPage(t, "backups_remote", data)
+	if !strings.Contains(html, "Felhom offsite tárhely — igényelhető szolgáltatás. Titkosított, házon kívüli másolat a mentéseidről. Érdeklődj az üzemeltetőnél.") {
+		t.Error("state 1: the opt-in pitch is missing")
+	}
+	if strings.Contains(html, "Aktív — nincs kijelölt alkalmazás") {
+		t.Error("state 1 must not render the state-2 headline")
+	}
+	if card := cardOf(html); strings.Contains(card, "<form") || strings.Contains(card, "<button") {
+		t.Error("the status card must be display-only (no form/button)")
+	}
+
+	// State 2: applied, zero apps toggled → the honest active-but-empty headline.
+	data = splitTestData()
+	data["OffboxToggledCount"] = 0
+	data["OffboxApps"] = []OffboxAppRow{{Name: "calibre-web", DisplayName: "Calibre-Web", Enabled: false}}
+	html = renderBackupPage(t, "backups_remote", data)
+	if !strings.Contains(html, "Aktív — nincs kijelölt alkalmazás") {
+		t.Error("state 2: 'Aktív — nincs kijelölt alkalmazás' missing")
+	}
+	if strings.Contains(html, "igényelhető szolgáltatás") {
+		t.Error("state 2 must not render the state-1 pitch")
+	}
+	if card := cardOf(html); strings.Contains(card, "<form") || strings.Contains(card, "<button") {
+		t.Error("the status card must be display-only (no form/button)")
+	}
+
+	// State 3: applied + toggled → NO card; the regular status block carries the state.
+	html = renderBackupPage(t, "backups_remote", splitTestData())
+	if strings.Contains(html, `id="felhom-offsite-card"`) {
+		t.Error("state 3 must not render the status card (the status block is the state)")
+	}
+	if !strings.Contains(html, "Utolsó távoli mentés") {
+		t.Error("state 3: the regular status block is missing")
+	}
+}
+
 // Route → handler wiring: each sub-route sets its own .Page id, so the sidebar child renders
 // active on the right page (backupMgr nil → the empty state renders, the nav still does).
 func TestBackupsSplit_RoutesSetPageIDs(t *testing.T) {
diff --git a/controller/internal/web/templates/backups_remote.html b/controller/internal/web/templates/backups_remote.html
index 07865a4..826d2c1 100644
--- a/controller/internal/web/templates/backups_remote.html
+++ b/controller/internal/web/templates/backups_remote.html
@@ -12,6 +12,15 @@
 {{template "backups_empty" .}}
 {{else}}
 
+<!-- Felhom-offsite status card (v0.124.0, decision 2): three honest states from LOCAL data only —
+     display + opt-in pointers, NEVER a state change (no form, no button in this card). State 3
+     (applied + apps toggled) is the existing status block below; the card renders nothing then. -->
+{{if not .OffboxConfigured}}
+<div class="alert alert-info" style="margin-bottom:1rem" id="felhom-offsite-card">Felhom offsite tárhely — igényelhető szolgáltatás. Titkosított, házon kívüli másolat a mentéseidről. Érdeklődj az üzemeltetőnél.</div>
+{{else if eq .OffboxToggledCount 0}}
+<div class="alert alert-info" style="margin-bottom:1rem" id="felhom-offsite-card">Aktív — nincs kijelölt alkalmazás</div>
+{{end}}
+
 <!-- Off-box (NAS) backup — Part B: encrypted restic repo over SFTP (the off-site 3-2-1 leg). -->
 <h3 id="offbox-section" class="backup-tier-divider">Távoli mentés (3. mentés) — titkosított, offsite</h3>
 <p class="form-hint" style="margin:-0.25rem 0 1rem">Az alkalmazás-mentések titkosított másolata egy távoli tárolóra — saját NAS vagy Felhom offsite tárhely — restic + SFTP kapcsolaton. A tároló csak titkosított adatot lát. Ez a 3-2-1 szabály „1 off-site" lába — független a helyi másodpéldánytól és a teljes rendszermentéstől.</p>