wip: Try to fix Heizkostenabrechnung

deeplex · May 30, 2024 · a374866 · a374866
1 parent 9e96b46
commit a374866
Show file tree

Hide file tree

Showing 8 changed files with 34 additions and 14 deletions.
diff --git a/.vscode/launch.json b/.vscode/launch.json
@@ -64,7 +64,7 @@
                 "ASPNETCORE_ENVIRONMENT": "Development",
                 "DATABASE_HOST": "walter_dev_db",
                 "DATABASE_PORT": "5432",
-                "DATABASE_NAME": "saverdb",
+                "DATABASE_NAME": "walter_dev_generic_db",
                 "DATABASE_USER": "postgres",
                 "DATABASE_PASS": "postgres",
                 "WALTER_PASSWORD": "postgres",

diff --git a/Deeplex.Saverwalter.BetriebskostenabrechnungService/Abrechnungseinheit.cs b/Deeplex.Saverwalter.BetriebskostenabrechnungService/Abrechnungseinheit.cs
@@ -156,10 +156,9 @@ private double GetVerbrauchAnteil(Betriebskostenrechnung rechnung, List<Note> no
 
             var verbrauchAnteil = verbrauchAnteile.First();
 
-
             if (verbrauchAnteil.Anteil.Count > 1)
             {
-                notes.Add($"Verbrauch von Rechnung {rechnung.Umlage.Typ} enthält mehr als einen Zählertypen",
+                notes.Add($"Verbrauch von Rechnung {rechnung.Umlage.Typ.Bezeichnung} enthält mehr als einen Zählertypen",
                     Severity.Error);
 
                 return 0;

diff --git a/Deeplex.Saverwalter.BetriebskostenabrechnungService/Utils.cs b/Deeplex.Saverwalter.BetriebskostenabrechnungService/Utils.cs
@@ -92,6 +92,10 @@ public static double GetKaltMiete(Vertrag vertrag, Zeitraum zeitraum)
                     {
                         return 0;
                     }
+                    else if (version.Beginn > zeitraum.Abrechnungsende)
+                    {
+                        return 0;
+                    }
                     else
                     {
                         var last = Min(versionEnde ?? zeitraum.Abrechnungsende, zeitraum.Abrechnungsende).Month;

diff --git a/Deeplex.Saverwalter.BetriebskostenabrechnungService/Verbrauchanteil.cs b/Deeplex.Saverwalter.BetriebskostenabrechnungService/Verbrauchanteil.cs
@@ -50,6 +50,10 @@ public VerbrauchAnteil(Umlage umlage, Wohnung wohnung, Zeitraum zeitraum, List<N
                         continue;
                     }
                     // Done because the first entry should not be in the last month.
+                    if (zaehler.Ende < zeitraum.Abrechnungsbeginn)
+                    {
+                        continue;
+                    }
                     var daysToFirstZaehler = zaehler.Staende
                         .OrderBy(stand => stand.Stand)
                         .FirstOrDefault()?
@@ -60,7 +64,11 @@ public VerbrauchAnteil(Umlage umlage, Wohnung wohnung, Zeitraum zeitraum, List<N
                     {
                         continue;
                     }
-                    var verbrauch = new Verbrauch(zaehler, zeitraum.Abrechnungsbeginn, zeitraum.Abrechnungsende, notes);
+                    var verbrauch = new Verbrauch(
+                        zaehler,
+                        zeitraum.Abrechnungsbeginn,
+                        zeitraum.Abrechnungsende,
+                        notes);
                     AlleZaehler[unit].Add(verbrauch);
                     AlleVerbrauch[unit] += verbrauch.Delta;
 

diff --git a/Deeplex.Saverwalter.WebAPI/Services/DbServices/UmlageDbService.cs b/Deeplex.Saverwalter.WebAPI/Services/DbServices/UmlageDbService.cs
@@ -180,7 +180,11 @@ private void SetOptionalValues(Umlage entity, UmlageEntry entry)
             {
                 if (hkvo.Id == 0)
                 {
-                    var newHKVO = new HKVO(hkvo.HKVO_P7 / 100, hkvo.HKVO_P8 / 100, (HKVO_P9A2)hkvo.HKVO_P9.Id, hkvo.Strompauschale / 100)
+                    var newHKVO = new HKVO(
+                        ((double)hkvo.HKVO_P7) / 100,
+                        ((double)hkvo.HKVO_P8) / 100,
+                        (HKVO_P9A2)hkvo.HKVO_P9.Id,
+                        ((double)hkvo.Strompauschale) / 100)
                     {
                         Betriebsstrom = Ctx.Umlagen.Single(e => e.UmlageId == hkvo.Stromrechnung.Id)
                     };
@@ -190,10 +194,10 @@ private void SetOptionalValues(Umlage entity, UmlageEntry entry)
                 else
                 {
                     var oldHKVO = Ctx.HKVO.Single(e => e.HKVOId == hkvo.Id);
-                    oldHKVO.HKVO_P7 = (double)hkvo.HKVO_P7 / 100;
-                    oldHKVO.HKVO_P8 = (double)hkvo.HKVO_P8 / 100;
+                    oldHKVO.HKVO_P7 = ((double)hkvo.HKVO_P7) / 100;
+                    oldHKVO.HKVO_P8 = ((double)hkvo.HKVO_P8) / 100;
                     oldHKVO.HKVO_P9 = (HKVO_P9A2)hkvo.HKVO_P9.Id;
-                    oldHKVO.Strompauschale = (double)hkvo.Strompauschale / 100;
+                    oldHKVO.Strompauschale = ((double)hkvo.Strompauschale) / 100;
                     oldHKVO.Betriebsstrom = Ctx.Umlagen.Single(e => e.UmlageId == hkvo.Stromrechnung.Id);
                     Ctx.HKVO.Update(oldHKVO);
                 }

diff --git a/Deeplex.Saverwalter.WebAPI/svelte/src/components/details/WalterUmlageHKVO.svelte b/Deeplex.Saverwalter.WebAPI/svelte/src/components/details/WalterUmlageHKVO.svelte
@@ -41,7 +41,9 @@ along with this program. If not, see <https://www.gnu.org/licenses/>.
 
     let visible: boolean = false;
     let oldHKVO: Partial<WalterHKVOEntry> = {};
-    let selectableUmlagen: Promise<WalterSelectionEntry[]>;
+    let selectableUmlagen: Promise<WalterSelectionEntry[]> = new Promise(
+        () => []
+    );
 
     onMount(() => {
         visible = !!entry.hKVO;
@@ -120,6 +122,7 @@ along with this program. If not, see <https://www.gnu.org/licenses/>.
     function change(e: Event, p9a2: WalterSelectionEntry[]) {
         if ((e.target as HTMLInputElement).checked) {
             entry.hKVO = {
+                id: oldHKVO.id || 0,
                 hkvO_P7: oldHKVO.hkvO_P7 || 50,
                 hkvO_P8: oldHKVO.hkvO_P8 || 50,
                 hkvO_P9: oldHKVO.hkvO_P9 || p9a2[1],
@@ -132,13 +135,13 @@ along with this program. If not, see <https://www.gnu.org/licenses/>.
     }
 </script>
 
-{#if entry.schluessel?.id === '3'}
+{#if `${entry.schluessel?.id}` === '3'}
     <Tile>
         <Row>
             {#await hkvo_p9a2 then p9a2}
                 <div>
                     <Checkbox
-                        checked={visible}
+                        bind:checked={visible}
                         on:change={(e) => change(e, p9a2)}
                     />
                 </div>

diff --git a/Deeplex.Saverwalter.WebAPI/svelte/src/lib/WalterUmlage.ts b/Deeplex.Saverwalter.WebAPI/svelte/src/lib/WalterUmlage.ts
@@ -58,8 +58,9 @@ export class WalterUmlageEntry extends WalterApiHandler {
         const selectedZaehler = json.selectedZaehler?.map(
             WalterSelectionEntry.fromJson
         );
+        // TODO: Check naming?
         const hkvo =
-            json.hKVO && WalterHKVOEntry.fromJson(json.hKVO as WalterHKVOEntry);
+            json.hkvo && WalterHKVOEntry.fromJson(json.hkvo as WalterHKVOEntry);
         const zaehler = json.zaehler?.map(WalterZaehlerEntry.fromJson);
         const permissions =
             json.permissions && WalterPermissions.fromJson(json.permissions);

diff --git a/Deeplex.Saverwalter.WebAPI/svelte/src/routes/(app)/umlagen/[id]/+page.svelte b/Deeplex.Saverwalter.WebAPI/svelte/src/routes/(app)/umlagen/[id]/+page.svelte
@@ -24,7 +24,8 @@ along with this program. If not, see <https://www.gnu.org/licenses/>.
         WalterUmlage,
         WalterZaehlerList,
         WalterLinks,
-        WalterLinkTile
+        WalterLinkTile,
+        WalterZaehler
     } from '$walter/components';
     import { convertDateCanadian } from '$walter/services/utils';
     import {
@@ -94,7 +95,7 @@ along with this program. If not, see <https://www.gnu.org/licenses/>.
             rows={data.entry.betriebskostenrechnungen}
         />
         <!-- Only show if Schlüssel is "nach Verbrauch" -->
-        {#if data.entry?.schluessel?.id === '3'}
+        {#if `${data.entry?.schluessel?.id}` === '3'}
             <WalterZaehlerList
                 fetchImpl={data.fetchImpl}
                 title="Zähler"