Fix: Voorkom dataverlies in Erkenningsinstantie bij pagina refresh

[Maxim-Peeters]

We merkten bij het indienen van de fiscale attesten dat de gegevens van de erkenningsinstantie (naam, adres, KBO) soms verdwenen na het refreshen van de Groep-pagina.

De oorzaak was het gebruik van de onUpdated() lifecycle hook in het Erkenningsinstantie-component om modelValue te synchroniseren met de lokale state. Omdat deze hook bij elke update loopt, ook tijdens het laden wanneer modelValue tijdelijk undefined kan zijn, werden de gegevens soms overschreven met lege waarden.

Oplossing:

• onUpdated() vervangen door watch() zodat alleen gereageerd wordt op echte wijzigingen van modelValue
• Guard toegevoegd om enkel te updaten wanneer er geldige data is
• Redundante props (:groep en :instantie) verwijderd uit Groep.vue, aangezien v-model volstaat

Ik kan dit momenteel niet testen zonder backend, maar hetzelfde patroon wordt al elders in de repo gebruikt, dus ik verwacht dat dit correct werkt.

Stevige Linker,
Maxim Peeters
Groepsleiding Scouts en Gidsen Retie

[KoenDeVoegt]

Bedankt voor je PR, ik krijg echter volgende error als ik dit test.

Mvg,
Koen De Voegt

[Maxim-Peeters]

Dag Koen,

Bedankt voor het testen van mijn changes, tijdens het coderen had ik iets over het hoofd gezien.

De watch functie kan niet geïmporteerd worden vanuit @vue/reactivity, maar moet komen vanuit "vue" zelf.

De fix is nu gepusht naar dezelfde branch. Kan je opnieuw testen?

Mvg,
Maxim

[KoenDeVoegt]

Dag Maxim,

Het schijnt nog steeds niet te werken. Kan je mss het probleem wat gedetailleerder beschrijven? We hebben het ook nog niet kunnen reproduceren. Misschien kan je een har file produceren? https://toolbox.googleapps.com/apps/har_analyzer/

Mvg,
Koen

[Maxim-Peeters]

Dag Koen,

Ik kan dit zelf momenteel niet volledig testen zonder backend, waardoor ik eigenlijk twee “foute” commits heb doorgestuurd in een poging het probleem te fixen.

Ik krijg bij mij geen error in de console. Wat ik zie is dat de erkenningsinstantie gewoon niet opgeslagen blijft na refresh, zonder foutmelding.

Aangezien er geen network error of console error is, lijkt een HAR file mij hier ook weinig extra info te geven.

Mvg,
Maxim