Rueckkaufswert am Vertrag sichtbar machen
Am Telefon mit der Gesellschaft braucht man die Zahl dort, wo man sie gefragt hat. Auf der Vertragsseite steht jetzt oben der zuletzt mitgeteilte Wert mit Stichtag und Quelle, darunter die Historie; in der Uebersicht traegt ein Wert, der aelter als ein Jahr ist, ein Abzeichen. Ohne Migration, und das ist keine Einsparung, sondern der Punkt: Die Werthistorie gibt es schon als Wertstand am Vermoegensposten, und der kann auf einen Vertrag zeigen. Ein Feld am Vertrag waere genau die zweite Wahrheit, die das Issue ausschliesst - dieselbe Zahl steht im Vermoegensverzeichnis, weil es dieselbe Zahl ist. Aufbau: - Die Ein-Jahres-Grenze als eine getestete Regel in finanzen/wertalter.ts. Sie stand bisher als "> 365" mitten in der Vermoegensseite; jetzt gilt sie am Vertrag genauso. - wertHistorie() fuehrt die Wertstaende aller mit dem Vertrag verknuepften Posten zu einer Reihe zusammen. - Wo ein Rueckkaufswert zu erwarten waere, aber keiner erfasst ist, steht "noch nicht erfasst" statt einer leeren Zelle. Zwei Fehler dabei behoben: - Vertrag und Vermoegensposten wurden ueber die ART des Postens zusammengefuehrt. Ein von Hand als "Sonstiges" angelegter Posten wurde nicht gefunden, und beim ersten Erfassen am Vertrag entstand still ein zweiter - Bericht und Vertrag haetten verschiedene Zahlen gezeigt. Massgeblich ist jetzt die Verknuepfung. - Beim Speichern eines Vermoegenspostens wurde geprueft, ob das verknuepfte Konto zur betreuten Person gehoert; bei Vertrag und Immobilie fehlte die Pruefung, beim Vertragsverweis einer Buchung ebenso. Bei mehreren betreuten Personen haette ein manipuliertes Formular ueber Aktengrenzen hinweg verknuepfen koennen. Geprueft: zehn Einheitstests, vier End-to-End-Tests - darunter der neu eingetragene Wert, der unveraendert im Vermoegensverzeichnis wiederauftaucht, und der Posten mit abweichender Art. Refs #8
This commit is contained in:
Florian Wilde
@@ -18,6 +18,7 @@ import {
|
||||
VERMOEGENS_ART_EINZAHL,
|
||||
} from '@/lib/finanzen/kategorien'
|
||||
import { vermoegenLaden } from '@/lib/berichte/gemeinsam'
|
||||
import { wertVeraltet } from '@/lib/finanzen/wertalter'
|
||||
import type { SozialhilfeBewertung } from '@prisma/client'
|
||||
|
||||
export const metadata = { title: 'Vermögen' }
|
||||
@@ -235,7 +236,7 @@ function Postentabelle({
|
||||
</TableCell>
|
||||
<TableCell className="whitespace-nowrap text-sm text-muted-foreground">
|
||||
{formatDatum(eintrag.bewertetAm)}
|
||||
{eintrag.alterTage > 365 && (
|
||||
{wertVeraltet(eintrag.alterTage) && (
|
||||
<Badge variant="gelb" className="ml-2">
|
||||
über ein Jahr alt
|
||||
</Badge>
|
||||
|
||||
@@ -18,7 +18,12 @@ import {
|
||||
kuendigungsplan,
|
||||
laufzeitAusVertrag,
|
||||
} from '@/lib/vertraege/kuendigung'
|
||||
import { anfrageStatus, rueckkaufswertAnschreiben } from '@/lib/vertraege/rueckkaufswert'
|
||||
import {
|
||||
anfrageStatus,
|
||||
rueckkaufswertAnschreiben,
|
||||
wertHistorie,
|
||||
} from '@/lib/vertraege/rueckkaufswert'
|
||||
import { wertalterBestimmen } from '@/lib/finanzen/wertalter'
|
||||
import {
|
||||
BEWERTUNG_BESCHRIFTUNG,
|
||||
BEWERTUNG_ERKLAERUNG,
|
||||
@@ -44,8 +49,9 @@ export default async function VertragDetail({ params }: { params: Promise<{ id:
|
||||
const vertrag = await db.vertrag.findFirst({
|
||||
where: { id, betreuteId: aktive.id, ...nichtGeloescht },
|
||||
include: {
|
||||
// Ueber die Verknuepfung, nicht ueber die Art - siehe wertHistorie().
|
||||
vermoegensposten: {
|
||||
where: { art: 'VERSICHERUNG', ...nichtGeloescht },
|
||||
where: nichtGeloescht,
|
||||
include: { wertstaende: { orderBy: { stichtag: 'desc' } } },
|
||||
},
|
||||
},
|
||||
@@ -76,7 +82,7 @@ export default async function VertragDetail({ params }: { params: Promise<{ id:
|
||||
heute,
|
||||
})
|
||||
|
||||
const wertstaende = vertrag.vermoegensposten[0]?.wertstaende ?? []
|
||||
const wertstaende = wertHistorie(vertrag.vermoegensposten)
|
||||
|
||||
return (
|
||||
<div className="space-y-6">
|
||||
@@ -226,6 +232,9 @@ export default async function VertragDetail({ params }: { params: Promise<{ id:
|
||||
tageSeitAnfrage={anfrage.tageSeitAnfrage}
|
||||
anschreiben={anschreiben}
|
||||
stichtagVorschlag={alsIsoTag(heute)}
|
||||
alterHinweis={
|
||||
wertalterBestimmen(wertstaende[0]?.stichtag ?? null, heute)?.hinweis ?? null
|
||||
}
|
||||
werte={wertstaende.map((w) => ({
|
||||
id: w.id,
|
||||
stichtag: w.stichtag.toISOString(),
|
||||
|
||||
@@ -22,10 +22,13 @@ import {
|
||||
import {
|
||||
BEWERTUNG_BESCHRIFTUNG,
|
||||
jahresbeitragCent,
|
||||
kannRueckkaufswertHaben,
|
||||
RHYTHMUS_BESCHRIFTUNG,
|
||||
VERTRAG_ART_BESCHRIFTUNG,
|
||||
VERTRAG_STATUS_BESCHRIFTUNG,
|
||||
} from '@/lib/vertraege/konstanten'
|
||||
import { juengsterWert } from '@/lib/vertraege/rueckkaufswert'
|
||||
import { wertalterBestimmen } from '@/lib/finanzen/wertalter'
|
||||
import { formatCent, formatDatum } from '@/lib/utils'
|
||||
|
||||
export const dynamic = 'force-dynamic'
|
||||
@@ -48,8 +51,10 @@ export default async function VertraegeUebersicht() {
|
||||
const vertraege = await db.vertrag.findMany({
|
||||
where: { betreuteId: aktive.id, ...nichtGeloescht },
|
||||
include: {
|
||||
// Ueber die Verknuepfung, nicht ueber die Art: Ein von Hand als
|
||||
// "Sonstiges" angelegter Posten meint denselben Vertrag.
|
||||
vermoegensposten: {
|
||||
where: { art: 'VERSICHERUNG', ...nichtGeloescht },
|
||||
where: nichtGeloescht,
|
||||
include: { wertstaende: { orderBy: { stichtag: 'desc' }, take: 1 } },
|
||||
},
|
||||
},
|
||||
@@ -71,6 +76,8 @@ export default async function VertraegeUebersicht() {
|
||||
})
|
||||
const ampel = kuendigungsampel(plan)
|
||||
|
||||
const letzterWert = juengsterWert(vertrag.vermoegensposten)
|
||||
|
||||
const jahr = jahresbeitragCent(vertrag.beitragCent, vertrag.rhythmus)
|
||||
if (jahr && vertrag.status === 'AKTIV') jahresbeitragSumme += jahr
|
||||
|
||||
@@ -97,11 +104,10 @@ export default async function VertraegeUebersicht() {
|
||||
beitragCent: centZuZahl(vertrag.beitragCent),
|
||||
rhythmus: vertrag.rhythmus,
|
||||
bewertung: vertrag.bewertungSozialhilfe,
|
||||
rueckkaufswertCent: centZuZahl(
|
||||
vertrag.vermoegensposten[0]?.wertstaende[0]?.wertCent ?? null,
|
||||
),
|
||||
rueckkaufswertStichtag:
|
||||
vertrag.vermoegensposten[0]?.wertstaende[0]?.stichtag.toISOString() ?? null,
|
||||
rueckkaufswertCent: centZuZahl(letzterWert?.wertCent ?? null),
|
||||
rueckkaufswertStichtag: letzterWert?.stichtag.toISOString() ?? null,
|
||||
rueckkaufswertAlter: wertalterBestimmen(letzterWert?.stichtag ?? null, heute),
|
||||
wertErwartet: kannRueckkaufswertHaben(vertrag.art),
|
||||
ampel,
|
||||
}
|
||||
})
|
||||
@@ -201,10 +207,21 @@ export default async function VertraegeUebersicht() {
|
||||
)}
|
||||
</TableCell>
|
||||
<TableCell className="text-right tabular-nums">
|
||||
{formatCent(zeile.rueckkaufswertCent)}
|
||||
{zeile.rueckkaufswertStichtag && (
|
||||
<span className="block text-xs text-muted-foreground">
|
||||
{formatDatum(zeile.rueckkaufswertStichtag)}
|
||||
{zeile.rueckkaufswertStichtag ? (
|
||||
<>
|
||||
{formatCent(zeile.rueckkaufswertCent)}
|
||||
<span className="block text-xs text-muted-foreground">
|
||||
{formatDatum(zeile.rueckkaufswertStichtag)}
|
||||
</span>
|
||||
{zeile.rueckkaufswertAlter?.veraltet && (
|
||||
<Badge variant="gelb" className="mt-1">
|
||||
{zeile.rueckkaufswertAlter.hinweis}
|
||||
</Badge>
|
||||
)}
|
||||
</>
|
||||
) : (
|
||||
<span className="text-sm font-normal text-muted-foreground">
|
||||
{zeile.wertErwartet ? 'noch nicht erfasst' : '–'}
|
||||
</span>
|
||||
)}
|
||||
</TableCell>
|
||||
|
||||
@@ -4,6 +4,7 @@ import Link from 'next/link'
|
||||
import { useActionState, useState } from 'react'
|
||||
import { Coins, FileDown, Mail, Plus } from 'lucide-react'
|
||||
import { Alert, AlertDescription, AlertTitle } from '@/components/ui/alert'
|
||||
import { Badge } from '@/components/ui/badge'
|
||||
import { Button } from '@/components/ui/button'
|
||||
import { Card, CardContent, CardDescription, CardHeader, CardTitle } from '@/components/ui/card'
|
||||
import { Input } from '@/components/ui/input'
|
||||
@@ -36,6 +37,7 @@ export function Rueckkaufswert({
|
||||
anschreiben,
|
||||
stichtagVorschlag,
|
||||
darfSchreiben,
|
||||
alterHinweis,
|
||||
}: {
|
||||
betreuteId: string
|
||||
vertragId: string
|
||||
@@ -46,6 +48,8 @@ export function Rueckkaufswert({
|
||||
anschreiben: string
|
||||
stichtagVorschlag: string
|
||||
darfSchreiben: boolean
|
||||
/** Kurztext, wenn der juengste Wert zu alt ist - sonst null. */
|
||||
alterHinweis: string | null
|
||||
}) {
|
||||
const [erfassen, erfassenAktion] = useActionState<Formularzustand, FormData>(
|
||||
rueckkaufswertErfassen,
|
||||
@@ -55,6 +59,9 @@ export function Rueckkaufswert({
|
||||
rueckkaufswertAnfordern,
|
||||
{},
|
||||
)
|
||||
// Die Liste kommt bereits juengster Wert zuerst von der Seite.
|
||||
const letzter = werte[0] ?? null
|
||||
|
||||
const [briefOffen, setBriefOffen] = useState(false)
|
||||
const [erfassenOffen, setErfassenOffen] = useState(false)
|
||||
|
||||
@@ -88,7 +95,7 @@ export function Rueckkaufswert({
|
||||
</Alert>
|
||||
)}
|
||||
|
||||
{werte.length === 0 ? (
|
||||
{!letzter ? (
|
||||
<Leerzustand
|
||||
titel="Noch kein Rückkaufswert erfasst"
|
||||
beschreibung="Fordere ihn bei der Gesellschaft an. Frag im selben Schreiben nach der beitragsfreien Versicherungssumme — sonst musst du für den Vergleich ein zweites Mal schreiben."
|
||||
@@ -103,7 +110,26 @@ export function Rueckkaufswert({
|
||||
}
|
||||
/>
|
||||
) : (
|
||||
<Table>
|
||||
<>
|
||||
{/* Der zuletzt mitgeteilte Wert gehoert nach oben: Wer beim
|
||||
Telefonat mit der Gesellschaft danach gefragt wird, hat
|
||||
keine Zeit, eine Historie zu lesen. */}
|
||||
<div className="rounded-md border p-4">
|
||||
<p className="text-sm text-muted-foreground">Zuletzt mitgeteilt</p>
|
||||
<p className="text-2xl font-semibold tabular-nums">
|
||||
{formatCent(letzter.wertCent)}
|
||||
</p>
|
||||
<p className="text-sm text-muted-foreground">
|
||||
zum Stichtag {formatDatum(letzter.stichtag)} · {letzter.wertquelle}
|
||||
</p>
|
||||
{alterHinweis && (
|
||||
<Badge variant="gelb" className="mt-2">
|
||||
{alterHinweis} — vor dem nächsten Bericht neu anfragen
|
||||
</Badge>
|
||||
)}
|
||||
</div>
|
||||
|
||||
<Table>
|
||||
<TableHeader>
|
||||
<TableRow>
|
||||
<TableHead>Stichtag</TableHead>
|
||||
@@ -126,7 +152,8 @@ export function Rueckkaufswert({
|
||||
</TableRow>
|
||||
))}
|
||||
</TableBody>
|
||||
</Table>
|
||||
</Table>
|
||||
</>
|
||||
)}
|
||||
|
||||
{darfSchreiben && (
|
||||
|
||||
@@ -0,0 +1,52 @@
|
||||
import { tageZwischen } from '@/lib/finanzen/datum'
|
||||
|
||||
/**
|
||||
* Wie alt ist ein Wert - und ab wann ist das ein Problem?
|
||||
*
|
||||
* Ein Rueckkaufswert ohne Datum ist schlimmer als kein Rueckkaufswert:
|
||||
* Man rechnet mit einer Zahl, die drei Jahre alt sein kann, und merkt es
|
||||
* erst, wenn der Bezirk nachfragt. Deshalb steht neben jedem Betrag der
|
||||
* Stichtag, und ab einem Jahr steht daneben, dass er alt ist.
|
||||
*
|
||||
* Die Grenze liegt bei 365 Tagen, weil der Bezirk jaehrlich prueft und
|
||||
* das Betreuungsgericht jaehrlich Rechnung legen laesst. Ein Wert, der
|
||||
* den letzten Jahresbericht ueberlebt hat, gehoert erneuert.
|
||||
*
|
||||
* Bewusst eine eigene Datei und nicht in vermoegen.ts: Die Regel gilt am
|
||||
* Vertrag genauso wie im Vermoegensverzeichnis, und sie soll an beiden
|
||||
* Stellen dieselbe sein - nicht zweimal als `> 365` hingeschrieben.
|
||||
*/
|
||||
|
||||
export const WERT_VERALTET_TAGE = 365
|
||||
|
||||
export type Wertalter = {
|
||||
/** Tage zwischen Stichtag und heute. Negativ bei kuenftigem Stichtag. */
|
||||
alterTage: number
|
||||
veraltet: boolean
|
||||
/** Kurztext fuer ein Abzeichen neben dem Betrag, sonst null. */
|
||||
hinweis: string | null
|
||||
}
|
||||
|
||||
/** Ist ein Wert dieses Alters erneuerungsbeduerftig? */
|
||||
export function wertVeraltet(alterTage: number): boolean {
|
||||
return alterTage > WERT_VERALTET_TAGE
|
||||
}
|
||||
|
||||
/**
|
||||
* Das Alter eines Werts zum Stichtag.
|
||||
*
|
||||
* Ohne Stichtag gibt es kein Alter - dann null, und die Oberflaeche
|
||||
* zeigt gar nichts an, statt eine Null zu behaupten.
|
||||
*/
|
||||
export function wertalterBestimmen(stichtag: Date | null, heute: Date): Wertalter | null {
|
||||
if (!stichtag) return null
|
||||
|
||||
const alterTage = tageZwischen(stichtag, heute)
|
||||
const veraltet = wertVeraltet(alterTage)
|
||||
|
||||
return {
|
||||
alterTage,
|
||||
veraltet,
|
||||
hinweis: veraltet ? 'über ein Jahr alt' : null,
|
||||
}
|
||||
}
|
||||
+1
-1
@@ -17,7 +17,7 @@
|
||||
*/
|
||||
|
||||
/** Muss zur "version" in der package.json passen. */
|
||||
export const VERSION = '1.7.0'
|
||||
export const VERSION = '1.8.0'
|
||||
|
||||
/**
|
||||
* Kurzer Git-Stand, beim Bauen hineingereicht:
|
||||
|
||||
@@ -114,3 +114,31 @@ export function anfrageStatus(
|
||||
tageSeitAnfrage: tage,
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Die Rueckkaufswert-Historie eines Vertrags, juengster Wert zuerst.
|
||||
*
|
||||
* Der Wert lebt nicht am Vertrag, sondern als Wertstand am
|
||||
* Vermoegensposten - genau derselbe, aus dem das Vermoegensverzeichnis
|
||||
* rechnet. Es gibt also keine zweite Zahl, die abweichen koennte, und
|
||||
* der Vertrag zeigt nur an, was ohnehin schon in der Akte steht.
|
||||
*
|
||||
* Verknuepft wird ueber die Vertrags-Id, nicht ueber die Art des Postens.
|
||||
* Wer einen Posten von Hand als "Sonstiges" anlegt und mit dem Vertrag
|
||||
* verknuepft, meint denselben Vertrag - er darf dadurch nicht eine
|
||||
* zweite, unsichtbare Wahrheit bekommen.
|
||||
*/
|
||||
export function wertHistorie<T extends { stichtag: Date }>(
|
||||
posten: { wertstaende: T[] }[],
|
||||
): T[] {
|
||||
return posten
|
||||
.flatMap((p) => p.wertstaende)
|
||||
.sort((a, b) => b.stichtag.getTime() - a.stichtag.getTime())
|
||||
}
|
||||
|
||||
/** Der zuletzt mitgeteilte Wert, oder null, wenn noch keiner vorliegt. */
|
||||
export function juengsterWert<T extends { stichtag: Date }>(
|
||||
posten: { wertstaende: T[] }[],
|
||||
): T | null {
|
||||
return wertHistorie(posten)[0] ?? null
|
||||
}
|
||||
|
||||
@@ -332,6 +332,15 @@ export async function buchungSpeichern(
|
||||
})
|
||||
if (!konto) return { fehler: 'Dieses Konto gehört nicht zu dieser betreuten Person.' }
|
||||
}
|
||||
|
||||
// Wie beim Konto: Der Vertrag muss zu dieser Akte gehoeren.
|
||||
if (werte.vertragId) {
|
||||
const vertrag = await db.vertrag.findFirst({
|
||||
where: { id: werte.vertragId, betreuteId, ...nichtGeloescht },
|
||||
select: { id: true },
|
||||
})
|
||||
if (!vertrag) return { fehler: 'Dieser Vertrag gehört nicht zu dieser betreuten Person.' }
|
||||
}
|
||||
if (werte.belegId) {
|
||||
const beleg = await db.dokument.findFirst({
|
||||
where: { id: werte.belegId, betreuteId, ...nichtGeloescht },
|
||||
@@ -634,6 +643,27 @@ export async function vermoegenspostenSpeichern(
|
||||
if (!konto) return { fehler: 'Dieses Konto gehört nicht zu dieser betreuten Person.' }
|
||||
}
|
||||
|
||||
// Dieselbe Pruefung fuer die beiden anderen Verweise. Sie fehlte:
|
||||
// An der Verknuepfung zum Vertrag haengt dessen Rueckkaufswert, und
|
||||
// die darf nicht in eine fremde Akte zeigen.
|
||||
if (werte.vertragId) {
|
||||
const vertrag = await db.vertrag.findFirst({
|
||||
where: { id: werte.vertragId, betreuteId, ...nichtGeloescht },
|
||||
select: { id: true },
|
||||
})
|
||||
if (!vertrag) return { fehler: 'Dieser Vertrag gehört nicht zu dieser betreuten Person.' }
|
||||
}
|
||||
|
||||
if (werte.immobilieId) {
|
||||
const immobilie = await db.immobilie.findFirst({
|
||||
where: { id: werte.immobilieId, betreuteId, ...nichtGeloescht },
|
||||
select: { id: true },
|
||||
})
|
||||
if (!immobilie) {
|
||||
return { fehler: 'Diese Immobilie gehört nicht zu dieser betreuten Person.' }
|
||||
}
|
||||
}
|
||||
|
||||
const felder = {
|
||||
bezeichnung: werte.bezeichnung,
|
||||
art: werte.art,
|
||||
|
||||
@@ -145,8 +145,13 @@ export async function vertragLoeschen(betreuteId: string, vertragId: string): Pr
|
||||
// -----------------------------------------------------------------------------
|
||||
|
||||
async function vermoegenspostenZumVertrag(betreuteId: string, vertragId: string, bezeichnung: string) {
|
||||
// Gesucht wird ueber die Verknuepfung, nicht ueber die Art. Sonst
|
||||
// entsteht neben einem von Hand als "Sonstiges" angelegten Posten ein
|
||||
// zweiter - und derselbe Vertrag haette zwei Wertreihen, von denen der
|
||||
// Bericht die eine und der Vertrag die andere zeigt.
|
||||
const vorhanden = await db.vermoegensposten.findFirst({
|
||||
where: { betreuteId, vertragId, art: 'VERSICHERUNG', ...nichtGeloescht },
|
||||
where: { betreuteId, vertragId, ...nichtGeloescht },
|
||||
orderBy: { erstelltAm: 'asc' },
|
||||
})
|
||||
if (vorhanden) return vorhanden
|
||||
return db.vermoegensposten.create({
|
||||
|
||||
Reference in New Issue
Block a user