Altinn / app-frontend-react

Altinn application React frontend
BSD 3-Clause "New" or "Revised" License
18 stars 31 forks source link

Betinget rendering utføres ikke ved instansiering hvis verdi settes i kalkulering #137

Open hanneline opened 5 years ago

hanneline commented 5 years ago

Describe the bug

Betinget rendering utføres ikke som forventet.

Hvis organisasjonsform er AS, skal en overskrift og 3 felt med radioknapper vises. Hvis jeg setter organisasjonsform AS i kalkuleringer for at feltet AS skal ha verdiene AS ved åpning av skjema, så vises ikke overskrifta og de tre feltene.

Endrer jeg til annen organisasjonsform feks DA, så er skjema likt. Endrer jeg tilbake til AS, så vises overskrifta og de 3 feltene.

Har annet som er er betinget rendering på, som er ok ved instansiering. Skjer kalkulering etter renderingen?

Expected behavior

Forventet at de feltene som skulle vises når organisasjonsformen er AS, skulle vises.

Screenshots

Ved åpning av skjema image

Endret til annen org.form image

Endret tilbake til AS image

Additional info

Windows 10 Pro, Chrome

hanneline commented 5 years ago

Overskrifta og feltene kommer fram hvis jeg endret et annet felt etter åpning, feks klikker ja eller nei på konsern.

gudrunvigerustfuru commented 5 years ago

@matsgm kan du svare på spørsmålet om rekkefølge/ trigging av kalkulering?

matsgm commented 5 years ago

@hanneline : Hvis du ved første utfylling skriver "AS" i det nevnte feltet, for deretter å klikke på (eller tabbe) til neste felt, vises fortsatt ikke de betingede elementene?

hanneline commented 5 years ago

@matsgm Jo, hvis jeg legger inn AS manuelt, og går ut av feltet, så vises de betingede feltene.

Men hvis jeg bruker kalkulering til å legge inn verdier i skjema, så må det visstnok endres noe i skjema "manuelt", før at endringen i kalkuleringen vises. Slik bør det vel ikke være?

matsgm commented 5 years ago

@hanneline Regler for dynamikk er implementert slik at det er når man "går ut av felt" (onBlur) at regler kjøres. Jeg har diskutert litt internt og har kommet frem til at det helt sikkert er flere tilfeller hvor regler burde kjøres dynamisk mens felter blir fyllt ut. F.eks. når de dynamiske feltene er de umiddelbart nærmeste feltene etter det man fyller ut.

Jeg skal ta opp dette internt så får du en tilbakemelding i denne tråden.

gudrunvigerustfuru commented 5 years ago

Hei @hanneline Vi ser at dette området må jobbes videre på, men det er ikke prioritert før MVP 3 (etter sommeren). Vi tar opp tråden igjen her.