swagger-api/swagger-codegen

Deserialization failed due to change of property name

Open

#4857 aperta il 26 feb 2017

Vedi su GitHub
 (4 commenti) (5 reazioni) (0 assegnatari)HTML (5474 fork)batch import
Client: TypeScriptIssue: Bughelp wanted

Metriche repository

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

Descrizione

Description

The issue is originally reported via https://github.com/swagger-api/swagger-codegen/issues/4805#issuecomment-280615622

For typescript-angular2:

"Request": {
                "device_type": {
                    "type": "string"
                },
            }
        },

generates:

export interface Request { 
    deviceType?: string;
}

The deserialization does not work as there's no mapping between deviceType and device_type (original attribute name in JSON)

Swagger-codegen version

latest master and 2.3.0

cc @Ondra09, @ataraxus, @Tifu

Guida contributor

Direzione di ricerca
Indaga i file modello TypeScript Angular2 nel repository swagger codegen per capire come i nomi delle proprietà vengono convertiti da snake case a camelCase. Cerca meccanismi di mappatura esistenti o opzioni di configurazione. Rivedi issue o PR simili per altri linguaggi che potrebbero aver affrontato questo problema. La correzione probabilmente richiede l'aggiunta di una mappatura di deserializzazione o l'uso di un decoratore come @JsonProperty. Controlla la logica di generazione del codice nella classe Codegen predefinita per la gestione dei nomi delle proprietà.
Tech stack
typescriptangular
Dominio
apideveloper experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
TypeScriptAngularGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70