punosmobile / jkr-lahti

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

Erottele kohteiden luonnissa ns. merkitsevät ja ei-merkitsevät rakennukset #74

Closed Rikuoja closed 1 year ago

Rikuoja commented 1 year ago

Tällä hetkellä kaikkia kiinteistön rakennuksia tarkastellaan samalla tavalla, ja niistä luodaan uusia kohteita saman logiikan mukaisesti.

Tehdään tähän issuen https://github.com/GispoCoding/jkr-lahti/issues/64 edellyttämä muutos: jaetaan rakennukset merkitseviin ja ei-merkitseviin rakennuksiin. Tehtävä erillinen funktio, joka tarkistaa onko rakennus merkitsevä:

Näiden rakennusten kohteille voidaan aina lisätä tai poistaa muita (ei-merkitseviä) rakennuksia.

Jos merkitsevä rakennus lakkaa olemasta, omistaja muuttuu, tulee lisää merkitseviä rakennuksia jne., luodaan aina uusi kohde.

TiiaY commented 1 year ago

@Rikuoja @LKajan @ltalve Ei tarvitse merkitä näitä, jos kohteet luotu alkuperäisen määrittelyn mukaisesti: https://lahti.sharepoint.com/:w:/r/sites/LahtiJtehuollonjrjestelmprojekti-Materiaalit/Jaetut%20asiakirjat/Materiaalit/Lahden%20m%C3%A4%C3%A4rittelyt/M%C3%84%C3%84RITTELY%20kohteen%20muodostus%20JKRss%C3%A4%207.12.2022.docx?d=w60340f9d5e30432f8d3f5b3b45e27a77&csf=1&web=1&e=eza47d

Määrittelyssä on eritelty rakennukset tyypeittäin, eikä kaikkia rakennuksia kohdella samalla lailla.

Rikuoja commented 1 year ago

Tällä hetkellä virhe tulee juuri siitä, että tontin ylimääräisiä rakennuksia kohdellaan eri tavalla sen mukaan, milloin itse alkuperäinen kohde muodostetaan. Ja silloin voi käydä niin, että näistä kohteen ulkopuolelle jäävistä ei-merkitsevistä rakennuksista (autotallit) tulee ylimääräisiä virheellisiä kohteita, kuten jo huomasitte kohdeongelmia listatessa. Eli käytännössä käy niin, että katsotaan rakennuksia, joilla on asukas, ja sitten vahingossa näiden kanssa samalla kiinteistöllä olevat rakennukset, joissa ei ole huoneistoa, lisätään mukaan kohteidenluontilistaan, vaikka ei pitäisi. Eli ihan logiikkabugi tuossa meidän toteutuksessa.

Tein siitä oman issuen https://github.com/GispoCoding/jkr-core/issues/37 , nämä kaksi on helpointa korjata samalla kertaa, niin ettei näistä ei-merkitsevistä rakennuksista synny koskaan kohteita.

Rikuoja commented 1 year ago

Tällä hetkellä on tosiaan katsottu tuota rakennuksen asukasta, ja sen jälkeen ei ole huomattu katsoa enää kiinteistön muiden rakennusten rakennustyyppejä, siitä johtuu tämä virhe. @TiiaY Onko tuo rakennustyyppilista nyt lopullinen? Kysyin tuolla toisessa issuessa https://github.com/GispoCoding/jkr-lahti/issues/64

Alempana kuitenkin on erilainen ehto (kohteet joissa 011 tai 041+ luonnollisen henkilön omistamat). Kumpi näistä on merkitsevän rakennuksen määritelmä? Onko tässä kaksi eri määritelmää merkitsevälle rakennukselle eri vaiheissa?

Rikuoja commented 1 year ago

@TiiaY Lisäksi vielä yksi kysymys saunoista: tuolla toisessa dokumentissa esiintyi saunoja, jotka oli virheellisesti liitetty kohteelle. Tarkoittaako tämä siis sitä, että samalla kiinteistöllä

a) saunoja ei pitäisi koskaan liittää kohteelle, b) saunat voi liittää kohteelle, jos niissä ei ole asukasta, c) saunat pitäisi aina liittää kohteelle vai d) jotain muuta, vähän muiden rakennusten tyypistä riippuen?

Eli siis yritän ehkä kysyä, onko sauna erikoistapaus (merkitsevä tai ei-merkitsevä) vähän tilanteen mukaan? Vai onko sauna aina oma merkitsevä rakennuksensa, kuten alussa lukee? Yritän jotenkin hahmottaa, onko saunoille omia sääntöjä, jotka pitäisi tarkistaa erikseen.

Rikuoja commented 1 year ago

Lisäsin saunoista oman issuensa https://github.com/GispoCoding/jkr-core/issues/41 . Siellä voidaan jatkaa keskustelua tilanteista, jossa saunoista pitää tehdä erillisiä kohteita. Mutta ilmeisesti jos kiinteistöllä on muita asuinrakennuksia, saunoista ei halutakaan omia kohteitaan.

ltalve commented 1 year ago

Q&A-sessiossa päätetty: Merkitsevät rakennukset ovat joka puolella aina samat.

Rikuoja commented 1 year ago

Fixed in https://github.com/GispoCoding/jkr-core/commit/96ae30dede0dad5e1e98b1134e8d8abeb1b64725