protocolbuffers / protocolbuffers/protobuf-javascript
Oneof toObject loses case
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 471
- Forks
- 91
- Ø Merge
- 3 Std. 57 Min.
- Gemergte PRs (30 T.)
- 2
Beschreibung
What version of protobuf and what language are you using?
Version: master/v3.6.0/v3.5.0 etc.
Language: Javascript
What operating system Linux, Windows and version?
What runtime / compiler are you using (e.g., python version or gcc version)
What did you do?
Steps to reproduce the behavior:
generated classes and methods via protoc-grpc-web result in oneof losing its case when converted to a plain object by toObject()
See https://github.com/grpc/grpc-web/issues/675 for details
What did you expect to see
The result object to only have one defined property for the oneof case
What did you see instead?
All of the oneof cases are generated in the result plain object with their type default values
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
Anything else we should know about your project / environment
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit den von protoc-grpc-web generierten Klassen und ihrer toObject()-Konvertierung und verwende das in Issue #675 beschriebene Verhalten als Reproduktionsreferenz. Erledigt ist die Aufgabe, wenn ein einfaches Objekt nur die definierte Eigenschaft für den aktiven oneof-Fall enthält, statt Eigenschaften mit Standardwerten für jeden Fall.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- api
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100