DorkasV / jira-to-github-issues

0 stars 0 forks source link

Der må ikke gemmes geometrier i e_gis med små arealer #172

Open sync-by-unito[bot] opened 2 months ago

sync-by-unito[bot] commented 2 months ago

JAR samarbejdet skal blive enige om hvad det mindste areal må vær på en geometri som må gemmes i e_gis.

Valideringen af arealet skal ske i GIS delen af brugerfladen, ved at en infoboks præsenteres for brugen med beskeden:

"Du er ved at gemme en flade med en geometri på under 0,1 m2. Dette er ikke tilladt. Slet derfor geometrien for gå videre"

Skærmbillede med eksempel på hvorledes fejlen kan præsenteres er vedhæftet:

!Snap1.jpg|width=200,height=183!

Brugerne må ikke kunne trykke "Næste" under "Flader i lokaliteten", hvis der findes et areal på under 0,1 m2.


BDD Test

Titel: Forhindre at geometri-arealer < end konfigureret værdi (0,1 m2) kan gemmes.

Som sagsbehandler Ønsker jeg at de steder i JAR.web hvor DMP geometri-valideringen kaldes, tilføjes en kontrol, som forhindrer muligheden for at oprette geometri-arealer < end konfigureret værdi (0,1 m2) ved at disse geometrier slettes der ikke bliver gemt geometrier med arealer < end konfigureret værdi (0,1 m2).

Scenarie 1: Oprettelse, kopiering eller redigering af flade skal ikke kunne oprette et areal der er < end konfigureret værdi (0,1 m2). Givet sagsbehandler opretter, kopierer eller redigerer en flade, i en af de situationer hvor eksisterende DMP validerings-komponent bliver kaldt (JAR-1470) Når validerings-komponenten kaldes skal geometrier, udover DMP valideringen, også tjekkes for om arealet er < end konfigureret værdi (0,1 m2) Når en eller flere arealer er < end værdien, præsenteres sagsbehandler, på samme måde som ved DMP valideringsfejl, for besked om at det ikke er muligt at gemme flade(r) med areal < end konfigureret værdi Og at flade(r) med areal < end konfigureret værdi automatisk slettes hvis sagsbehander klikker videre.

Titel: Oprydning i geometri-arealer < end konfigureret værdi (0,1 m2).

Som databaseadministrator i en region Ønsker jeg at kunne afvikle et script på JAR-databasen geometrier med areal < end konfigureret værdi (0,1 m2) på lokaliteter slettes.

Scenarie 1: Afvikling af script på JAR-database Givet at databaseadministrator afvikler script på JAR-databasen i Live tilstand Når scriptet afvikles identificeres geometrier med areal < end konfigureret værdi (0,1 m2) på lokaliteter Og sletter disse identificerede geometrier.

┆Attachments: e_gis - arealer under 0,1 m2.sql | Snap1.jpg

sync-by-unito[bot] commented 2 months ago

➤ John Ryan Pedersen commented:

Arealet skal kunne konfigureres i opsætningen via konfigfilen

sync-by-unito[bot] commented 2 months ago

➤ Peter Riis commented:

i Administration >> Systemkonfiguration skal der opsættes en parameter på størrelsen af arealet som som der skal valideres på. Således kan én region vælger at der skal valideres på geometrier mindre end 0,1 m2, mens en anden kan vælge en værdi på 1 m2.

Dette er kun en GIS valideringsregel og der må natrligvis ikke kunne slettes geometrier på en lokalitet der er mindre end parameteren for arealets størrelse.

Der kunne måske laves en Unit test på baggrund af dette issue.

sync-by-unito[bot] commented 2 months ago

➤ Peter Riis commented:

Issue var assignet til mig, men jeg ved ikke hvad jeg skal gøre. Issue må være en del af den almindelige prioritering.

sync-by-unito[bot] commented 2 months ago

➤ Peter Riis commented:

Script til udtræk af de små arealer, som evt. kan slettes igennem et andet script eller manuelt er vedlagt

sync-by-unito[bot] commented 2 months ago

➤ Filip Bruman commented:

Peter Riis

Med “Valideringen af arealet skal ske i GIS delen af brugerfladen…“ får vi associationer til DMP geometrivalideringen (JAR-1309 og JAR-1379) .

Skal validering af areal ske i de samme situationer som der foretages DMP geometrivalidering?

sync-by-unito[bot] commented 2 months ago

➤ Peter Riis commented:

Det synes jeg lyder som en god idé.

Der skal ikke slettes arelaer > 0,1, men brugeren skal orienteres om arealerne findes og ikke kan gemmes. Så skal de selv fjerne dem ved at slette dem eller kombinere dem med andre arealer.

sync-by-unito[bot] commented 2 months ago

➤ Filip Bruman commented:

Peter Riis

Er der stadig et ønske at størrelse af mindste areal skal kunne konfigureres?

sync-by-unito[bot] commented 2 months ago

➤ Peter Riis commented:

Ja, det er der et ønske om at det skal kunne lade sig gøre.

sync-by-unito[bot] commented 2 months ago

➤ Devran Kucukyildiz commented:

Peter Riis Lige et afklarende spørgsmål: Skal arealet på geometrien udelukkende valideres i GIS-kortet?

I https://jar-cowi.atlassian.net/browse/JAR-1379 ( https://jar-cowi.atlassian.net/browse/JAR-1379|smart-link ), bliver validering f.eks. også foretaget ved “Opret fase”.

sync-by-unito[bot] commented 2 months ago

➤ Peter Riis commented:

Devran Kucukyildiz , jeg forstår ikke dit spørgsmål.

Validering af areal ske i de samme situationer som der foretages DMP geometrivalidering. Se tidligere i issuet.

info til brugerne præsenteres på samme måde som ved DMP geometrivalidering.

sync-by-unito[bot] commented 2 months ago

➤ Devran Kucukyildiz commented:

Peter Riis Valideringer implementeret i https://jar-cowi.atlassian.net/browse/JAR-1379 ( https://jar-cowi.atlassian.net/browse/JAR-1379|smart-link ) kræver, i tilfælde af valideringsfejl, at sagsbehandler tegner geometrien i stedet for at genbruge (kopiere) geometrien ifm. f.eks. "Opret fase".

En mulig løsning på denne situationen kunne være at slette areal < 0,1 m2 fra oprettet fase så sagsbehandler slet ikke skal informeres og dermed ikke skal tage stilling til at valideringen fejlet.

For valideringer fra både https://jar-cowi.atlassian.net/browse/JAR-1379 ( https://jar-cowi.atlassian.net/browse/JAR-1379|smart-link ) og https://jar-cowi.atlassian.net/browse/JAR-1309 ( https://jar-cowi.atlassian.net/browse/JAR-1309|smart-link ) kan sagsbehandler vælge at gemme geometri med kendte valideringsfejl for at tage hånd om disse på et senere tidspunkt og dermed ikke gå i stå midt i sagsbehandlingen.

I stedet for sagsbehandler forstyrres og forhindres i deres arbejde, kunne man implementere samme løsningsmodel som ovenfor hvor areal < 0,1 m2 automatisk slettes. Hvad tænker du om dette?

sync-by-unito[bot] commented 2 months ago

➤ Peter Riis commented:

Ok, men kan det så løses, så små arealer valideres/slettes på måder?

  1. Hvis fladen tegnes, så adviseres brugerne om at de små geometrier ikke er tilladte., og skal slettes.
  2. Ved kopiering af flader, så slettes geometrier som er mindre end det tilladte areal.

Vedhæftet SQL bør også valideres, så regionerne selv kan slette eksisterende små arealer. Dermed kan regionerne undgå at mulighed 2 opstår.

Kim Jacobsen , vil du sætte dette issue på næste PG møde, så vil lige kan tale om de løsningsforslag som Cowi beskriver.

sync-by-unito[bot] commented 2 months ago

➤ Kim Jacobsen commented:

Hej Line Boel

Vil du være sød at undersøge, hvor store “jeres” små / mindste statusflader er?

Blot så vi kan få et mål at gå ud fra.

mvh

Projektgruppen

😀

NB: En mulig løsning på denne situationen kunne være at slette areal < 0,1 m2 fra

sync-by-unito[bot] commented 2 months ago

➤ Line Boel commented:

Hans Christian Ries : kunne du hjælpe med at finde ud af, hvor lille det mindste af vores statusflader er…. ?

sync-by-unito[bot] commented 2 months ago

➤ Line Boel commented:

Hej Kim Jacobsen

Det er OK med os at slette alle areal < 0,1 m2

line

sync-by-unito[bot] commented 2 months ago

➤ Kim Jacobsen commented:

Det er nu afklaret. at alle areal under < 0,1 m2 kan slettes!

Løsning fra COWI ønskes beskrevet og estimeret. Vi ønsker både at få beskrevet et script til at rydde op samt et forslag til håndtering i brugerflade og forretningslogik

mvh

Kim

sync-by-unito[bot] commented 2 months ago

➤ Filip Bruman commented:

Peter Riis

Vil du læse BDD igennem og bekræfte at opgavens forståelse og omfang er korrekt.

sync-by-unito[bot] commented 2 months ago

➤ Peter Riis commented:

Filip Bruman , fin BDD og den er godkendt.

sync-by-unito[bot] commented 2 months ago

➤ Anders Rosengreen commented:

Kim Jacobsen

Hej Kim,

Kan du godkende vores overslag for analyse inkl. tid forbrugt på BDD?

Analysen kræver nedenstående arbejde og afklaring.

JAR.web-validering:

Tjek, at følgende:

ikke lader fundne små områder passere for at blive gemt (kopisituation, viser ikke DMP-valideringsfejl).

Database script:

Andet:

Overslag analyse inkl. BDD timer: Seniorkonsulent: 50 timer Projektleder: 5 timer Pris: 48460 DKK

Mvh 

Anders

sync-by-unito[bot] commented 2 months ago

➤ Kim Jacobsen commented:

Peter Riis

Kan vi gøre noget for at få denne analyse sat i gang.

Jeg har talt med COWI ang. pris. Prisen afspejler, at der er brugt en del tid i forbindelse med forespørgsler og undersøgelser hen ad vejen.

mvh

Kim

sync-by-unito[bot] commented 2 months ago

➤ Peter Riis commented:

Kim Jacobsen , er der noget som jeg skal gøre? Nu går issuet jo ikke på en analyse, så vi skal være opmærksomme på at det også skal implementeres i dette JAR.

sync-by-unito[bot] commented 2 months ago

➤ Peter Riis commented:

Da dette kræver yderligere en analyse fra Cowi fjernes dette fra 4.10

sync-by-unito[bot] commented 2 months ago

➤ Peter Riis commented:

Region Sjælland har slettet alle arealer mindre end 0,2 m2 i vores database.