BRP-API / brp-api-gezag

Het gezag component van Haal Centraal
0 stars 0 forks source link

minderjarige geboren in buitenland krijgt toch gezag uitspraak #182

Open fsamwel opened 4 days ago

fsamwel commented 4 days ago

Zie https://github.com/BRP-API/brp-api-gezag/blob/feature-geboorteland-niet-geadopteerd/features/gezag-minderjarige/geboorte_land.feature

Deze minderjarige is geboren in het buitenland (heeft geboorteland anders dan 6030) en is niet geadopteerd (aktenummer is niet 1.Q....). Dan verwacht ik uit vraag 1.3a en 1.3b gezag niet te bepalen. In de vorige versie van de gezag api kreeg ik dat ook.

Wanneer ik deze feature uitvoer (op 1.7.0-20241024112908) krijg ik tweehoofdig ouderlijk gezag.

In de logging zie ik: v0103A='Nee', v0103B='null'

Waarom wordt bij vraag 1.3a het antwoord 'Nee' met geboorte_land=6043?

Patrick-4488 commented 3 days ago

@fsamwel het gaat weer om een ander veld dat gebruikt wordt om vestiging in nederland te bepalen. In de gezagmodule wordt gekeken naar het veld DatumVestigingInNederland van verblijfplaats

Pas wanneer dit veld niet null en niet leeg is wordt in de code gekeken of de landcode nederland is. Als je de waarde voor DatumVestigingNederland invuld, gaat het wel goed.

Patrick-4488 commented 3 days ago

PS: natuurlijk kunnen we ook concluderen dat de controlle beter kan, we kunnen ook de code aanpassen.

Patrick-4488 commented 3 days ago
        if (datumVestigingInNederland == null
            || datumVestigingInNederland.isEmpty()
            || geboorteland.equals(GEBOORTELAND_CODE_NEDERLAND)
        ) {
            answer = V1_3A_NEE;
        } else {
            answer = V1_3A_JA;
        }
fsamwel commented 3 days ago

ik heb in de HUP (handleiding uitvoering procedures) opgezocht. Ook bij eerste inschrijving uit buitenland worden datum en land vestiging gevuld (net als bij remigratie en inschrijving van niet-ingezetene uit RNI). Deze afleiding is dus terecht. Ik zal het testgeval aanpassen. En sluit het issue

fsamwel commented 3 days ago

@Patrick-4488 ik heb nu de immigratie velden in Gegeven toegevoegd, maar krijg vervolgens een 500 server error: Cannot invoke \"nl.rijksoverheid.mev.gezagsmodule.service.gezagmodule.GezagBepaling.getErrorTraceCode()\" because \"gezagBepaling\" is null

Kan je hier nog eens naar kijken?

Patrick-4488 commented 2 days ago

doe ik!

Patrick-4488 commented 2 days ago

Een insert op de database gaat mis:

text: 'INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geslachts_naam,geboorte_datum,geboorte_land_code,doc_gemeente_code) VALUES($1,$2,$3,$4,$5,$6,$7,$8,$9)',
  values: [
    '1',
    '0',
    '0',
    'K',
    '000000036',
    'Arendsen',
    '20151201',
    '6043',
    'verklaring van persoon'
  ],
  level: 'error',
  message: 'exception in geboren in buitenland'
}
{
  length: 161,
  name: 'error',
  severity: 'ERROR',
  code: '22P02',
  detail: undefined,
  hint: undefined,
  position: undefined,
  internalPosition: undefined,
  internalQuery: undefined,
  where: "unnamed portal parameter $9 = '...'",
  schema: undefined,
  table: undefined,
  column: undefined,
  dataType: undefined,
  constraint: undefined,
  file: 'numutils.c',
  line: '244',
  routine: 'pg_strtoint16',
  level: 'error'
}

Dit resulteerd in dat de gezagsmodule de persoon niet kon ophalen en door een foutje in de code onstond een 500 error. Ik heb de code aangepast dat de exceptie niet meer optreed.

Ik weet niet zeker hoe ik de insert moet fixen, ik verwacht als dit gefixed is het goed werkt.