Merge pull request #61 from GispoCoding/Lahti-95-Lue-csv-xslx-sijaan

Lahti: Muuta importskripti lukemaan csv-tiedostoja xslx sijaan

db/migrations/R__import_koodisto_osapuolenlaji.sql

@@ -1,4 +1,5 @@
 insert into jkr_koodistot.osapuolenrooli(id, selite) values
     (1,'Omistaja'),
-    (2,'Vanhin asukas')
+    (2,'Vanhin asukas'),
+    (3,'Asiakas')
 on conflict (id) do update set selite = excluded.selite;

jkrimporter/providers/db/services/buildings.py

@@ -316,7 +316,7 @@ def _find_by_address(session: "Session", haltija: "Yhteystieto"):
         select(Rakennus)
         .join(Osoite)
         .join(Katu)
-        .join(RakennuksenVanhimmat, outer=True)  # allow vapaa-ajanrakennukset
+        .outerjoin(RakennuksenVanhimmat)  # allow vapaa-ajanrakennukset
         .where(
             Osoite.posti_numero == haltija.osoite.postinumero,
             sqlalchemyFunc.lower(Katu.katunimi_fi) == katunimi_lower,

jkrimporter/providers/lahti/models.py

@@ -1,6 +1,7 @@
 import datetime
 import re
 from datetime import date
+from dateutil.parser import parse as date_parser
 from enum import Enum
 from typing import Dict, Optional, Union
 
@@ -118,6 +119,9 @@ def fix_posti(value: Union[str, int]):
     @validator("koko", "paino", pre=True)
     def parse_float(value: Union[float, str]):
         # If float wasn't parsed, let's parse them
+        # Return none if empty string was parsed
+        if value == '':
+            return None
         if type(value) is str:
             # we might have . or , as the separator
             return float(value.replace(",", "."))
@@ -127,6 +131,14 @@ def parse_float(value: Union[float, str]):
     def parse_jatelaji(value: str):
         if value == "Sekaj":
             value = "Sekajäte"
+        if value == "Biojäte":
+            value = "Bio"
+        if value == "Kartonkipakkaus":
+            value = "Kartonki"
+        if value == "Muovipakkaus":
+            value = "Muovi"
+        if value == "Lasipakkaus":
+            value = "Lasi"
         return value.title()
 
     @validator("Pvmalk", "Pvmasti", pre=True)
@@ -142,8 +154,8 @@ def parse_date(value: Union[date, str]):
         "tyhjennysvali", "Voimassaoloviikotalkaen", "Voimassaoloviikotasti", pre=True
     )
     def fix_na(value: str):
-        # Many fields may have strings such as #N/A. Parse them to None.
-        if value == "#N/A":
+        # Many fields may have strings such as #N/A or empty string. Parse them to None.
+        if value == "#N/A" or value == '':
             return None
         return value
 
@@ -154,3 +166,12 @@ def parse_kuntatunnus(value: str):
         if value:
             return int(value)
         return None
+
+    @validator("astiamaara", pre=True, always=True)
+    def parse_decimal_separator(cls, value):
+        if isinstance(value, str):
+            # There is atleast one case where SUM(astiamaara) is "0,12"
+            # Not sure what to do here, doesn't make sense that 0.12 containers have been emptied.
+            # But then again, should this be converted to 0, 1 or 12?
+            value = float(value.replace(',', '.'))
+        return value

jkrimporter/providers/lahti/siirtotiedosto.py

@@ -1,12 +1,10 @@
 import logging
+import csv
 from pathlib import Path
 
-from openpyxl.reader.excel import load_workbook
-
-from jkrimporter.datasheets import ExcelCombinedFileSheetCollection, SiirtotiedostoSheet
+from jkrimporter.datasheets import SiirtotiedostoSheet
 from jkrimporter.providers.lahti.models import Asiakas
 
-
 logger = logging.getLogger(__name__)
 
 
@@ -17,26 +15,29 @@ def _obj_from_dict(data):
 
 
 class LahtiSiirtotiedosto:
-    # Lahti has no set sheet names. It has a directory with different sheets
-    # for different providers, all having identical format.
-
     def __init__(self, path):
-        self._sheet_collection = ExcelCombinedFileSheetCollection(path)
+        self._path = path
 
     @classmethod
     def readable_by_me(cls, path):
-        p = Path(path)
-        for f in p.iterdir():
-            if f.is_file() and f.suffix == ".xlsx":
-                try:
-                    workbook = load_workbook(filename=f, data_only=True, read_only=True)
-                    sheets = workbook.sheetnames
-                    if "in" in sheets:
-                        return True
-                except Exception:
-                    pass
+        directory = Path(path)
+        for file in directory.iterdir():
+            if file.is_file() and file.suffix == ".csv":
+                return True
         return False
 
     @property
     def asiakastiedot(self):
-        return AsiakastiedotSheet(self._sheet_collection)
+        all_data = []
+
+        # Iterate through all CSV files in the directory
+        for csv_file_path in Path(self._path).glob("*.csv"):
+            with open(csv_file_path, mode="r", encoding="cp1252", newline="") as csv_file:
+                csv_reader = csv.DictReader(csv_file, delimiter=";", quotechar='"')
+                data_list = [row for row in csv_reader]
+                all_data.extend(data_list)
+
+        # Convert to a list of Asiakas objects
+        asiakas_list = [Asiakas.parse_obj(data) for data in all_data]
+
+        return asiakas_list

pyproject.toml

@@ -30,6 +30,7 @@ shapely = "^1.8.2"
 typer = "^0.6.1"
 python-dotenv = "^0.20.0"
 addrparser = "^0.1.0"
+python-dateutil = "^2.8.2"
 
 [tool.poetry.dev-dependencies]
 flake8 = "^5.0.4"

tests/data/test_lahti_siirtotiedosto/kuljetustiedot_csv.csv

@@ -0,0 +1,4 @@
+UrakoitsijaId;x-koordinaatti;y.koordinaattori;Eranro;UrakoitsijankohdeId;Rakennustunnus/Kiinteistotunnus;Kiinteistonkatuosoite;Kiinteistonposti;kimppa;kimppaid;Haltijannimi;Haltijanyhteyshlo;Haltijankatuosoite;Haltijanposti;Haltijanmaakoodi;Haltijanulkomaanpaikkakunta;Pvmalk;Pvmasti;tyyppiIdEWC;tapahtumannimi;COUNT(kaynnit);SUM(astiamaara);koko;SUM(paino);tyhjennysvali;kertaaviikossa;Voimassaoloviikotalkaen;Voimassaoloviikotasti;Voimassapmvalkaen;Voimassapvmasti;Voimassaoloviikotalkaen2;Voimassaoloviikotasti2;tyhjennysvali2;kertaaviikossa2;PalveluKuuluukokimppaan;Kimpanjakoosuus;palveluKimppakohdeId;KimpanNimi;Kimpasta vastaava;Kimpankatuosoite;Kimpanposti;Kuntatun;Keskeytysalkaen;Keskeytysasti
+0000000-9;6761402;427353;1234AB;01-0000001-00;123456789A;HARJUKATU 44;15100 LAHTI;;;ASUNTO OY KAHDEN LAULUMUISTO;HEIKKI LEHMUSTO;VESIJÄRVENKATU 8;15100 LAHTI;FI;;1.1.2023;31.12.2023;Sekaj;660 L SEKAJÄTEASTIA TYHJENNYS;123;4;0,66;10;1;2;1;53;1.1.1900;31.12.2099;;;;;;;;;;;;398;1.2.2023;15.2.2023
+0000000-9;;;;01-0000123-01;134567890B;KIRKKO�YR��NTIE 1D;16200 ARTJ�RVI;;;LINDROTH OY;JOHN LINDROTH;KIRKKO�YR��NTIE 1D;16200 ARTJ�RVI;FI;;1.1.2023;31.12.2023;Energia;660 L ENERGIAJ�TEASTIA TYHJENNYS;66;2;0,66;8;1;;1;53;1.1.1900;31.12.2099;;;;;;;01-0000999-99;KIRKKO�YR��N ENERGIAKIMPPA;01-1110999-01;KIRKKO�YR��NTIE 11;16200 ARTJ�RVI;560;;
+0000000-9;;;;01-0000123-02;100456789B;KUVAKALLIONTIE 1;15230 LAHTI;;;RIKU FORSSTR�M;RIKU FORSSTR�M;KUVAKALLIONTIE 1;15230 LAHTI;FI;;1.1.2023;31.12.2023;Energia;660 L ENERGIAJ�TEASTIA TYHJENNYS;22;2;0,66;7;1;;1;53;1.1.1900;31.12.2099;;;;;;;01-0000999-99;KIRKKO�YR��N ENERGIAKIMPPA;01-1110999-01;KIRKKO�YR��NTIE 11;16200 ARTJ�RVI;560;;

tests/test_data_import.py

@@ -24,7 +24,8 @@ def engine():
 
 def test_osapuolenrooli(engine):
     osapuolenroolit = [(1, 'Omistaja'),
-                       (2, 'Vanhin asukas')]
+                       (2, 'Vanhin asukas'),
+                       (3, 'Asiakas')]
     session = Session(engine)
     result = session.execute(select([Osapuolenrooli.id, Osapuolenrooli.selite]))
     assert [tuple(row) for row in result] == osapuolenroolit

tests/test_lahti_siirtotiedosto.py

@@ -32,7 +32,9 @@ def test_readable(datadir):
 
 def test_kohteet(datadir):
     asiakastiedot = LahtiSiirtotiedosto(datadir).asiakastiedot
-    assert 'UrakoitsijaId' in asiakastiedot.headers
+    header_row = asiakastiedot[0]
+    headers = header_row.dict().keys()
+    assert 'UrakoitsijaId' in headers
 
 
 def test_import_data(engine, datadir):