Raten ueber Lebensphasen uebernehmen + Rate in der Verlaufsgrafik
Deploy App / deploy (push) Successful in 1m6s

(1) Beim Aendern eines Ratenfelds fragt das Panel nach der Reichweite:
nur diese Phase (Vorgabe), diese + folgende, alle Phasen. Gilt fuer
expectedReturn, valueGrowth, interestRate und teuerungsausgleich.

Inline statt Modal -- das Zahlenfeld loest je Tastendruck aus. Die
Zielphasen behalten ihre uebrigen Werte (der Endpunkt ersetzt den ganzen
Satz; ein Kopieren des Entwurfs haette dort Betraege geloescht).

(2) ElementYearPoint fuehrt neu `rate` mit -- additiv, nur durchgereicht.
Verlaufsgrafik zeigt sie auf zweiter Y-Achse als Stufenlinie.

Golden Tests unveraendert. Spezifikation 0.20, 17 Tests (164 -> 181).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
2026-07-19 22:36:05 +02:00
parent d023534a03
commit 4343d50aa4
7 changed files with 482 additions and 11 deletions
+49 -5
View File
@@ -396,6 +396,17 @@ export function ElementDetailDialog({
? "Eigenkapital"
: "Wert";
// Zweite Achse nur, wenn dieses Element überhaupt eine Rate trägt (AHV und Schulden nicht).
const rateLabel =
category === "REAL_ESTATE"
? "Wertsteigerung"
: isFlow
? category === "INCOME"
? "Lohnentwicklung"
: "Reale Mehrausgaben"
: "Erwartete Rendite";
const hasRate = points.some((p) => typeof p.rate === "number");
const verlauf = (
<>
{points.length === 0 ? (
@@ -406,16 +417,49 @@ export function ElementDetailDialog({
<LineChart data={points} margin={{ top: 8, right: 16, left: 8, bottom: 8 }}>
<CartesianGrid strokeDasharray="3 3" className="stroke-border" />
<XAxis dataKey="age" type="number" domain={["dataMin", "dataMax"]} tick={{ fontSize: 11 }} tickFormatter={(v) => `${v} J.`} />
<YAxis tick={{ fontSize: 11 }} tickFormatter={(v) => Intl.NumberFormat("de-CH", { notation: "compact" }).format(v)} />
<Tooltip formatter={(v, n) => [typeof v === "number" ? formatChf(v) : v, n]} labelFormatter={(v) => `Alter ${v}`} />
<YAxis yAxisId="chf" tick={{ fontSize: 11 }} tickFormatter={(v) => Intl.NumberFormat("de-CH", { notation: "compact" }).format(v)} />
{hasRate && (
<YAxis
yAxisId="rate"
orientation="right"
tick={{ fontSize: 11 }}
tickFormatter={(v) => `${v} %`}
// Etwas Luft nach oben und unten, damit eine konstante Rate nicht als
// Linie direkt auf der Achse klebt.
domain={([min, max]: readonly [number, number]) => [Math.min(0, min - 1), max + 1]}
/>
)}
<Tooltip
formatter={(v, n) =>
typeof v !== "number" ? [v, n] : n === rateLabel ? [`${v} %`, n] : [formatChf(v), n]
}
labelFormatter={(v) => `Alter ${v}`}
/>
<Legend wrapperStyle={{ fontSize: 12 }} />
<Line dataKey="value" name={valueLabel} stroke="var(--accent)" strokeWidth={2} dot={false} isAnimationActive={false} />
<Line yAxisId="chf" dataKey="value" name={valueLabel} stroke="var(--accent)" strokeWidth={2} dot={false} isAnimationActive={false} />
{MULTI_SERIES.includes(category) && (
<>
<Line dataKey="propertyValue" name="Verkehrswert" stroke="#0ea5e9" strokeWidth={1.5} dot={false} isAnimationActive={false} />
<Line dataKey="mortgage" name="Restschuld" stroke="#dc2626" strokeWidth={1.5} strokeDasharray="5 3" dot={false} isAnimationActive={false} />
<Line yAxisId="chf" dataKey="propertyValue" name="Verkehrswert" stroke="#0ea5e9" strokeWidth={1.5} dot={false} isAnimationActive={false} />
<Line yAxisId="chf" dataKey="mortgage" name="Restschuld" stroke="#dc2626" strokeWidth={1.5} strokeDasharray="5 3" dot={false} isAnimationActive={false} />
</>
)}
{hasRate && (
// Stufenlinie, nicht interpoliert: Die Rate ist innerhalb einer Phase
// konstant und springt an der Phasengrenze. Eine weiche Kurve würde einen
// gleitenden Übergang suggerieren, den die Berechnung nicht macht.
<Line
yAxisId="rate"
type="stepAfter"
dataKey="rate"
name={rateLabel}
stroke="#d97706"
strokeWidth={1.5}
strokeDasharray="4 2"
dot={false}
connectNulls
isAnimationActive={false}
/>
)}
</LineChart>
</ResponsiveContainer>
</div>
+75 -1
View File
@@ -6,6 +6,14 @@ import { AlertTriangle } from "lucide-react";
import { FieldLabel, MoneyField, NumberField, SelectField, TextField } from "@/components/FormField";
import { formatChf } from "@/lib/format";
import { api } from "@/lib/api-client";
import {
buildRateWrites,
changedRateFields,
scopeQuestion,
SCOPE_OPTIONS,
type PhaseRef,
type RateScope,
} from "@/lib/ratefields";
import { ahvAnnualPension, ahvMdje, type AhvCareer } from "@/lib/calculations";
import { CATEGORY_LABELS, num } from "@/lib/elements";
import {
@@ -49,6 +57,10 @@ interface Props {
context: CellContext;
phaseData: PhaseData;
transitionData: TransitionData;
// Alle Phasen des Szenarios und die dort bereits erfassten Werte -- Grundlage für die
// Übernahme einer geänderten Rate auf weitere Phasen (siehe SPEZIFIKATION 3.6.11).
allPhases?: PhaseRef[];
phaseDataByPhase?: Record<string, PhaseData>;
onSaved: () => void;
onDeleteElement: () => void;
}
@@ -886,14 +898,29 @@ export function ElementTransitionFields({
}
}
export function ElementDetail({ element, context, phaseData, transitionData, onSaved, onDeleteElement }: Props) {
export function ElementDetail({
element,
context,
phaseData,
transitionData,
allPhases = [],
phaseDataByPhase = {},
onSaved,
onDeleteElement,
}: Props) {
const [pd, setPd] = useState<PhaseData>({ ...phaseData });
const [td, setTd] = useState<TransitionData>({ ...transitionData });
const [saving, setSaving] = useState(false);
const [error, setError] = useState<string | null>(null);
// Reichweite einer geänderten Rate. Vorgabe ist das bisherige Verhalten: nur diese Phase.
const [scope, setScope] = useState<RateScope>("THIS");
const isTransition = context.kind === "transition";
// Welche Ratenfelder hat der Nutzer verändert? Nur dann wird überhaupt gefragt.
const changedRates = isTransition ? [] : changedRateFields(element.category, phaseData, pd);
const showScope = changedRates.length > 0 && allPhases.length > 1;
async function save() {
setSaving(true);
setError(null);
@@ -914,6 +941,21 @@ export function ElementDetail({ element, context, phaseData, transitionData, onS
delete payload.amount;
}
await api.put(`/api/elements/${element.id}/phase/${context.phaseId}`, payload);
// Geänderte Raten auf die gewählten weiteren Phasen übertragen. Die bestehenden
// Werte der Zielphasen bleiben erhalten -- übernommen wird ausschliesslich die Rate.
// Alle Schreibvorgänge fallen in dieselbe Bearbeitungssitzung und ergeben daher
// EINE Nebenversion, nicht eine je Phase (siehe 3.8.1).
for (const w of buildRateWrites(
allPhases,
context.phaseId,
scope,
changedRates,
pd,
phaseDataByPhase
)) {
await api.put(`/api/elements/${element.id}/phase/${w.phaseId}`, w.data);
}
}
onSaved();
} catch (e) {
@@ -959,6 +1001,38 @@ export function ElementDetail({ element, context, phaseData, transitionData, onS
)}
</div>
{/* Rückfrage zur Reichweite. Erscheint erst, wenn eine Rate tatsächlich verändert
wurde -- und bewusst hier statt als Modal beim Tippen: Das Zahlenfeld löst bei
jedem Tastendruck aus, ein Dialog erschiene bei «5.2» viermal. */}
{showScope && (
<div className="rounded-xl border border-attention bg-attention-soft p-3">
<p className="mb-2 text-xs text-attention-soft-fg">
{scopeQuestion(changedRates, allPhases.length)}
</p>
<div className="inline-flex flex-wrap gap-0.5 rounded-lg border border-border bg-surface p-0.5 text-xs">
{SCOPE_OPTIONS.map((o) => (
<button
key={o.value}
type="button"
onClick={() => setScope(o.value)}
className={`rounded-md px-2.5 py-1 font-medium transition-colors ${
scope === o.value ? "bg-accent text-accent-fg" : "text-muted hover:text-fg"
}`}
>
{o.label}
</button>
))}
</div>
<p className="mt-1.5 text-[11px] text-attention-soft-fg/80">
{scope === "THIS"
? "Andere Phasen bleiben unverändert."
: `Wird beim Speichern in ${
buildRateWrites(allPhases, context.phaseId, scope, changedRates, pd, phaseDataByPhase).length
} weitere Phase(n) übertragen. Beträge und Raten dort bleiben erhalten nur der geänderte Wert wird gesetzt.`}
</p>
</div>
)}
{error && <p className="text-sm text-danger">{error}</p>}
<div>
<button
+4
View File
@@ -863,6 +863,10 @@ export function PlanView({
element={element}
context={buildPhaseContext(phase, element)}
phaseData={element.phaseValues[phase.id] ?? {}}
// Für die Übernahme von Raten über Phasen hinweg: alle Phasen und die dort
// bereits erfassten Werte (sie müssen beim Übertragen erhalten bleiben).
allPhases={plan.phases.map((p) => ({ id: p.id, sequenceNumber: p.sequenceNumber }))}
phaseDataByPhase={element.phaseValues}
transitionData={{}}
onSaved={closeAndRefresh}
onDeleteElement={() => {