Feltene står som valgfrie, men det kreves likevel utfylling

[hanneline]

Describe the bug
Feltene står som valgfrie, men det kreves likevel utfylling.
Hva styrer om et felt er valgfri eller ikke?

To Reproduce
Steps to reproduce the behavior:

1. Gå til tjeneste NyTjeneste (org. hanneline)
2. Velg Test->Manuell
3. Velg Id 1398 i meldingsboksen
4. Alle feltene er valgfri jf ledetekstene
5. Klikk på feltet organisasjonsnummer
6. Gå ut av feltet uten å skrive noe i det
7. Feilmelding Field is required

Expected behavior
For valgfrie felt, kommer det ikke melding om at feltet må fylles ut.
For påkrevde felt står det ikke valgfri ved ledeteksten.

Screenshots
If applicable, add screenshots or animated gif to help explain your problem.

Additional info
Windows 10 Pro, Chrome

Tasks

• Set default value to required
• Change the rule that sets fields with minOccurs=0 to optional (and can't be changed)
• Check if xsd is converted properly. If not fix it.
• Write tests if necessary
• Test
  • Test that minOccurs=1 sets the field to required

[GGunnar]

#1103 is related, it improves this functionality but does not solve it

[malmil]

Pull Request #1415 solves this issue.
Should be tested when the PR has been merged. @trymen

[trymen]

Retested in dev. Optional field no longer throws an error when it is not filled (tested with input component). Closing this issue.

[hanneline]

@trymen Når kommer denne rettingen i altinn.studio?

[hannekot]

@hanneline kan oppdatere altinn.studio i morgen tidlig med den endringen

[hanneline]

Flott, blir det bare denne, eller får vi alle andre også? Skulle hatt i allefall 1103 også. :)

[hannekot]

@hanneline 1103 er med i oppgraderinga. Altinn.studio er nå oppdatert

[hanneline]

@hannekot Fungerer ikke i altinn.studio

[hannekot]

@hanneline prøv å slette elementene og legge de inn på nytt igjen

[hanneline]

@hannekot Det blir litt arbeid det, blir det slik hver gang dere legger til nye egenskaper? Må fikse logikken også på nytt da...

[hanneline]

@hannekot Egenskapen er ikke der, selv om jeg legger til et nytt felt.

[hannekot]

@hanneline legg inn en ny komponent, tykk på den komponenten som er lagt til (komponenten markert med hvit) og trykk på f(x) ikonet på siden så skal du få opp:

[hannekot]

Det er fortsatt problemer med denne
Funker greit hvis du ikke skriver inn noe og trykker lagre
Men hvis du så skriver inn noe, fjerner og så fjerner det igjen så vil du få beskjed om at feltet er required. Dette er ikke ønsket oppførsel

[hannekot]

@hanneline vi lager en issue på at du må slette komponentene og så legge de inn igjen for å få denne funksjonaliteten til å fungere (@trymen kan du tagge issuet her?)
I teorien så burde dette fungert uten at du måtte lage skjemaet på nytt igjen

[hanneline]

@hannekot Det var ikke der jeg forventet å finne egenskaper til komponenten.

Mitt problem med valgfri er at SERES XSDene er satt veldig åpne, lite som er må-felter i XSD. Så jeg trenger å sette at de er må-felter i skjema.

[hanneline]

@hannekot Får ikke endret Valgfri på komponent i skjema.

Ny komponent, kan endre Valgfri (default er valgfri):

Legger inn kobling til datamodell, kan ikke endre Valgfri (hvor systemet har fjernet avkrysning for Valgfri):

[hannekot]

@hanneline det skjer nok fordi feltet i datamodellen er required. Man kan ikke overstyre datamodellen i UI-editoren

[Hannott]

@hannekot Det er sant med modifikasjoner. I ORXSD er elementer merket minOccurs=0, så de burde være valgfrie, men fordi de inneholder attributter som er required, blir hele feltet required. Dette er en feiltolkning av XSD til JSON, noe man lett kan teste ved å validere en nesten tom XML mot en XSD som altinn.studio mener har alle felter påkrevd.

Eksempel nedenfor, hele gruppen og navn er valgfritt, men hvis det er med, så er orid/gruppeId attributtet påkrevd.

   <xs:sequence>
      <xs:element minOccurs="0" ref="soknad-grp-5663"/>
   </xs:sequence>

    <xs:element name="soknad-grp-5663">
        <xs:complexType>
            <xs:sequence>
                <xs:element minOccurs="0" ref="Kontaktperson-grp-5665"/>
            </xs:sequence>
            <xs:attribute fixed="5663" name="gruppeid" type="xs:positiveInteger" use="required"/>
        </xs:complexType>
    </xs:element>

    <xs:element name="Kontaktperson-grp-5665">
        <xs:complexType>
            <xs:sequence>
                <xs:element minOccurs="0" ref="KontaktpersonNavn-datadef-2">
                </xs:element>
            </xs:sequence>
            <xs:attribute fixed="5665" name="gruppeid" type="xs:positiveInteger" use="required"/>
        </xs:complexType>
    </xs:element>

    <xs:element name="KontaktpersonNavn-datadef-2">
        <xs:complexType>
            <xs:simpleContent>
                <xs:extension base="Tekst150-repformat-13">
                    <xs:attribute fixed="2" name="orid" type="xs:positiveInteger" use="required"/>
                </xs:extension>
            </xs:simpleContent>
        </xs:complexType>
    </xs:element>

    <xs:simpleType name="Tekst150-repformat-13">
        <xs:restriction base="xs:string">
            <xs:minLength value="1"/>
            <xs:maxLength value="150"/>
        </xs:restriction>
    </xs:simpleType>

[hannekot]

@Hannott parseren som gjør om fra XSD til JSON er ikke fedigutviklet og inneholder feil. Det er ikke prioritert for øyeblikket at dette skal fikses. Men det du kan gjøre er å opprette en bug på det du beskriver her (med mindre du har alt gjort det) så er det nok større sansynlighet at dette blir tatt tak i.

[gudrunvigerustfuru]

@trymen gå over å se om alle bugs nevnt i tråden er opprettet

[lvbachmann]

@helenekri @Febakke Den mest fornuftige måten å løse dette på er å si at felter i utgangspunktet er obligatoriske - dvs. standardverdien for "Valgfri" er at det ikke er avhuket. Er dette noe dere fra funksjonell side vil protestere på at er lurt?

[Febakke]

@lvbachmann Ja, det er mer naturlig at default verdien er obligatorisk. (Eller ikke valgfri)

[GGunnar]

#1943 må ses i sammenheng når man ser på denne feilen.

[malmil]

Nå er dette issuet løst ved at:

• Om datamodellfeltet har minOccurs sjekkes et nivå opp.
• Hvis ikke minOccurs er satt er defaultverdien obligatiorisk.
• Man kan overskrive valgfri/obligatorisk satt i datamodellen i tjenestelogikkmenyen

[jeevananthank]

@malmil
I have tested this with https://dev.altinn.studio/designer/newtestuser/newservice1007#/uieditor

1. An input field has been set as 'valgfri' in the Tjenestelogikk menu
2. A new instance is created and the same field is shown as 'required' field

[malmil]

@jeevananthank
Have you created a new service?
I just tested it in dev.altinn.studio and it works as expected. Fields set to "valgfri" get the "valgfri"-text, and the required once does not.

[malmil]

@malmil
I have tested this with https://dev.altinn.studio/designer/newtestuser/newservice1007#/uieditor

1. An input field has been set as 'valgfri' in the Tjenestelogikk menu
2. A new instance is created and the same field is shown as 'required' field

Found the error. Fixing it now 👍

[jeevananthank]

The fix is tested OK.