RicoSuter/NSwag

Bug - NSwag generates bad code for an enum

Open

#2883 aperta il 9 giu 2020

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C# (1189 fork)batch import
help wantedproject: NJsonSchemaproject: NJsonSchema.CodeGeneration.CSharpproject: NJsonSchema.CodeGeneration.TypeScripttype: bug

Metriche repository

Star
 (6291 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

I have 2 DTOs, both contain same Enum image

I have 2 controller methods using those DTO as URL query args image

I have 2 swagger definitions with same Enum image

And I get bad code from NSwag with "2" added to Enum name image

If this is not a bug then what am I doing wrong? Also this seem to be wrong only on method signature of the generated interface and the class, once I remove "2" the code compiles and I think it's generally ok code

image

Guida contributor

Direzione di ricerca
Indaga come NSwag gestisce la denominazione degli enum quando lo stesso enum appare in più DTO. Esamina il codice di generazione del client TypeScript nel repository NSwag, in particolare la parte che assegna nomi agli enum. Il problema mostra che a un enum viene assegnato un suffisso '2', probabilmente a causa di una risoluzione di conflitti di denominazione. Verifica se esiste un meccanismo di rilevamento delle collisioni e perché fallisce. Rivedi anche i due commenti sull'issue per eventuali approfondimenti. Considera di riprodurre il problema con un esempio minimale per confermare il comportamento.
Tech stack
csharptypescript
Dominio
toolingapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Understanding of NSwagBasic knowledge of C# and TypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
45