else if-statements bevatten dezelfde conditie

Sudwest-Fryslan / OpenZaakBrug

Een vertaler om de bestaande applicaties, die het oude zds-standaard praten, toch met een zgw-zaaksysteem te laten werken.

European Union Public License 1.2

5 stars 7 forks source link

else if-statements bevatten dezelfde conditie #351

Closed joostvanderborg closed 1 year ago

joostvanderborg commented 1 year ago

De conditie op regel 636 kan nooit waar zijn, omdat in regel 626 dezelfde conditie-check bestaat. Daardoor zal zgwRol.betrokkeneIdentificatie.innRechtsvorm nooit de waarde naamloze_vennootschap kunnen krijgen.

https://github.com/Sudwest-Fryslan/OpenZaakBrug/blob/6011ceb14b7e1311185ed39b70d3bf92e50f8143/src/main/java/nl/haarlem/translations/zdstozgw/translation/zds/services/ZaakService.java#LL636C11-L636C11

jacodg commented 1 year ago

Goed punt. Ik weet zelf niet direct welke string er in de if gebruikt zou moeten worden om onderscheid te maken tussen beide of dat er eentje weg kan maar ik zal het "in de groep gooien"

haarlem-dvh commented 1 year ago

Dit is inderdaad een bug. Er zijn meerdere mogelijke rechtsvormen die 'vennootschap' bevatten en dus altijd binnen 626 zullen vallen.

https://www.gemmaonline.nl/index.php/Rsgb_3.0/doc/enumeration/soortrechtsvorm