RicoSuter/NSwag

Bug - NSwag generates bad code for an enum

Open

#2.883 geöffnet am 9. Juni 2020

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (1.189 Forks)batch import
help wantedproject: NJsonSchemaproject: NJsonSchema.CodeGeneration.CSharpproject: NJsonSchema.CodeGeneration.TypeScripttype: bug

Repository-Metriken

Stars
 (6.291 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche, wie NSwag die Benennung von Enums handhabt, wenn dasselbe Enum in mehreren DTOs vorkommt. Sieh dir den TypeScript Client Generierungscode im NSwag Repository an, insbesondere den Teil, der Enums Namen zuweist. Das Problem zeigt, dass ein Enum ein '2' Suffix erhält, wahrscheinlich aufgrund einer Namenskonfliktlösung. Prüfe, ob es einen Kollisionserkennungsmechanismus gibt und warum er fehlschlägt. Überprüfe auch die beiden Kommentare zum Issue für weitere Erkenntnisse. Erwäge, das Problem mit einem minimalen Beispiel zu reproduzieren, um das Verhalten zu bestätigen.
Tech Stack
csharptypescript
Domain
toolingapi
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Understanding of NSwagBasic knowledge of C# and TypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45