RicoSuter/NJsonSchema

minlength is impacted by required attribute in json schema

Aperta

#768 aperta il 31 ago 2018

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)C# (549 fork)github user discovery
help wantedtype: enhancement

Metriche repository

Star
 (1580 stelle)
Metriche merge PR
 (Merge medio 39g 6h) (1 PR mergiata in 30 g)

Descrizione

I used below schema to generate C# code using CSharpGenerator. But there is no validation for the length if the field marked as required. The validation ( [System.ComponentModel.DataAnnotations.StringLength(int.MaxValue, MinimumLength = 1)]) is correctly generated if the field is not marked as required.

{ "$schema": "http://json-schema.org/draft-04/schema#", "type": "object", "properties": { "files": { "type": "string", "minlength": 1 } }, "required": [ "files" ] }

Is this intentional? I am using version: 9.10.67 on nuget.

Guida contributor

Direzione di ricerca
Esamina il codice sorgente di CSharpGenerator per capire come gestisce la combinazione degli attributi minlength e required. Controlla se esiste una logica condizionale che omette l'attributo StringLength quando una proprietà è required.
Tech stack
csharp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
.NETC#JSON Schema
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70