RicoSuter/NSwag

Bug - NSwag generates bad code for an enum

Open

#2.883 aberto em 9 de jun. de 2020

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C# (1.189 forks)batch import
help wantedproject: NJsonSchemaproject: NJsonSchema.CodeGeneration.CSharpproject: NJsonSchema.CodeGeneration.TypeScripttype: bug

Métricas do repositório

Stars
 (6.291 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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

Guia do colaborador

Direção de pesquisa
Investigue como o NSwag lida com a nomenclatura de enums quando o mesmo enum aparece em vários DTOs. Examine o código de geração do cliente TypeScript no repositório do NSwag, especificamente a parte que atribui nomes aos enums. O problema mostra que um enum recebe um sufixo '2', provavelmente devido a uma resolução de conflito de nomenclatura. Verifique se há um mecanismo de detecção de colisão e por que ele falha. Revise também os dois comentários na issue para obter insights adicionais. Considere reproduzir o problema com um exemplo mínimo para confirmar o comportamento.
Pilha de tecnologia
csharptypescript
Domain
toolingapi
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Understanding of NSwagBasic knowledge of C# and TypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
45