Generate alternate accessor for fields with presence
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- javascript
- Domaine
- backend
Piste de recherche
Reproduisez le cas d’un champ optionnel en générant le message OptTest indiqué et en appelant toObject. Commencez par le code toObject généré et l’appel à getFieldWithDefault ; déterminez comment la présence devrait affecter la valeur par défaut. Le travail est terminé lorsqu’un champ optionnel non défini produit undefined plutôt que 0 ou une chaîne vide, avec une couverture pour le proto présenté.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
When I executed the toObject method on a message with a field marked optional, that field became the default value like 0 or "". I expect it would be undefined.
I guess it is because the third argument of getFieldWithDefault is not undefined.
Since proto3 currently support optional keyword, when optional is specified, I think it should be undefined if it is not given.
proto
syntax = "proto3";
package com.book;
message OptTest {
optional string name = 1;
}
generated toObject code
proto.com.book.OptTest.toObject = function(includeInstance, msg) {
var f, obj = {
name: jspb.Message.getFieldWithDefault(msg, 1, "")
};
if (includeInstance) {
obj.$jspbMessageInstance = msg;
}
return obj;
};
versions
$ npm list --depth=0 -g
/usr/local/lib
+-- [email protected]
+-- [email protected]
`-- [email protected]
- Langage dominant
- JavaScript
- Étoiles
- 470
- Forks
- 91
- Merge moyen
- 1 j 12 h
- PR mergées (30 j)
- 6
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Aucun guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de protocolbuffers/protobuf-javascript
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
protocolbuffers/protobuf-javascript#248 · 1 commentaire · 13 réactions ·
-
question
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
protocolbuffers/protobuf-javascript#222 · 9 commentaires ·
-
Why map.js sort keys?Ouverteenhancement port-fix triaged
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
protocolbuffers/protobuf-javascript#185 · 1 commentaire ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
Toutes les issues de protocolbuffers/protobuf-javascript
Issues similaires
-
🐞 bug pending triage
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
CircuitVerse/CircuitVerse#8025 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
schmayterling/hibi#245 ·
Les mainteneurs répondent en général sous 1 jour
-
factory-active task-identify-harness-labels-done task-identify-issue-type-done
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
vercel/ai#23304 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
bug package:vitest
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
cloudflare/workers-sdk#16255 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Proxy and MCP URLs with an empty authority parse as a different hostPeut-être pris @ericcaiwx-star l’a pris aujourd’hui. Ouverteclawsweeper:bulk-filed clawsweeper:linked-pr-open clawsweeper:needs-product-decision clawsweeper:no-new-fix-pr clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster P2
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
openclaw/openclaw-enterprise#2384 · 1 commentaire · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour