search

punosmobile / jkr-lahti

Lahden jätteenkuljetusrekisteri
GNU General Public License v3.0
1 stars 0 forks source link

Kohteiden luonnin korjaus DVV-päivitysajon jälkeen #86

Closed Mtk112 closed 1 year ago

Mtk112 commented 1 year ago

Traceback (most recent call last): File "C:\Projektit\jkr-core\jkrimporter\providers\db\dbprovider.py", line 331, in write_dvv_kohteet import_dvv_kohteet(session, poimintapvm, loppupvm, perusmaksutiedosto) File "C:\Projektit\jkr-core\jkrimporter\providers\db\dbprovider.py", line 228, in import_dvv_kohteet paritalo_kohteet = get_or_create_paritalo_kohteet(session, poimintapvm, loppupvm) File "C:\Projektit\jkr-core\jkrimporter\providers\db\services\kohde.py", line 1274, in get_or_create_paritalo_kohteet return get_or_create_kohteet_from_vanhimmat( File "C:\Projektit\jkr-core\jkrimporter\providers\db\services\kohde.py", line 892, in get_or_create_kohteet_from_vanhimmat {dvv_rakennustiedot[vanhin.rakennus_id]}, KeyError: 66405

ismogis commented 1 year ago

Sain toistettua. Testitapauksista puuttuu paritalokohde ja kohteiden luonnin testaus päivityksen jälkeen. Lisätään ensin paritalokohde dataan, ja tehdään tarvittaessa testaus päivityksen jälkeiselle kohteiden luonnille.

ismogis commented 1 year ago

Lisätty testeihin paritalokohde, DVV-päivitys ja kohteiden päivitys. Ongelma ei esiinny näissä testeissä. Tarkistetaan, onko ongelman aiheuttaneeseen kohteeseen tullut muutosta DVV-versioiden välillä vai tuleeko virhe vain silloin, kun kohteet luodaan ensimmäistä kertaa DVV-päivityksen jälkeen.

ismogis commented 1 year ago

Ongelman aiheuttaneeseen kohteeseen ei ole tullut muutoksia DVV-versioiden välillä. Ongelma ei myöskään toistu testidatalla, vaikka kohteet loisi ensimmäistä kertaa DVV-päivityksen jälkeen. Tässä vaiheessa epäselvää, miten testitapaus poikkeaa oikeasta datasta.

ismogis commented 1 year ago

Testitapaus ei poikkea oikeasta datasta, vaan ongelma liittynee pelkästään siihen, ettei import_dvv_kohteet-funktiolle anneta loppupäivämäärää.

ismogis commented 1 year ago

Korjattu kohteen voimassaolon tarkistus, kun loppupäivämäärää ei anneta, mutta kaatuu oikealla datalla edelleen johonkin paritalokohteeseen (eri kuin aiemmin). Skriptissä erehdytään luulemaan paritalokohteeksi kohdetta, joka on oikeasti yhden asunnon talo.

ismogis commented 1 year ago

Ilmeisesti kohde tunnistuu väärin paritalokohteeksi, jos kohteen osapuoli on osapuolena myös ei-paritalokohteessa.