RicoSuter/NSwag

Bug - NSwag generates bad code for an enum

Open

#2 883 ouverte le 9 juin 2020

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)C# (1 189 forks)batch import
help wantedproject: NJsonSchemaproject: NJsonSchema.CodeGeneration.CSharpproject: NJsonSchema.CodeGeneration.TypeScripttype: bug

Métriques du dépôt

Stars
 (6 291 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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

Guide contributeur

Direction de recherche
Étudiez comment NSwag gère la dénomination des énumérations lorsque la même énumération apparaît dans plusieurs DTO. Examinez le code de génération du client TypeScript dans le dépôt NSwag, en particulier la partie qui attribue des noms aux énumérations. Le problème montre qu'une énumération reçoit un suffixe '2', probablement en raison d'une résolution de conflit de nommage. Vérifiez s'il existe un mécanisme de détection de collision et pourquoi il échoue. Consultez également les deux commentaires sur l'issue pour obtenir des informations supplémentaires. Envisagez de reproduire le problème avec un exemple minimal pour confirmer le comportement.
Stack technique
csharptypescript
Domaine
toolingapi
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Understanding of NSwagBasic knowledge of C# and TypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45