toObject method of my message class uses an undeclared variable
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 42/100
- Tipo de issue
- Bug
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- javascript
- Domínio
- tooling
Direção de pesquisa
Comece com minimal.proto e execute o comando protoc --js_out fornecido para reproduzir a saída gerada para M2.toObject. Compare o método gerado com a saída esperada e verifique se a variável f não declarada não é mais emitida.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
What version of protobuf and what language are you using?
Version: 3.7.0
Language: Javascript
What operating system (Linux, Windows, ...) and version?
Ubuntu 16.04.5 LTS (Bitnami LAMP)
MacOS Mojave 10.14.2
What runtime / compiler are you using (e.g., python version or gcc version)
node js v11.10.1
What did you do?
Steps to reproduce the behavior:
file minimal.proto:
syntax = "proto3";
message M1 {
uint64 x = 1;
}
message M2 {
M1 m = 1;
}
run protoc --js_out="import_style=commonjs,binary:." minimal.proto
What did you expect to see
proto.M2.toObject = function(includeInstance, msg) {
var obj, f = {
m: (f = msg.getM()) && proto.M1.toObject(includeInstance, f)
};
if (includeInstance) {
obj.$jspbMessageInstance = msg;
}
return obj;
};
(or at least that's what it was doing when it worked with version 3.5)
What did you see instead?
proto.M2.toObject = function(includeInstance, msg) {
var obj = {
m: (f = msg.getM()) && proto.M1.toObject(includeInstance, f)
};
if (includeInstance) {
obj.$jspbMessageInstance = msg;
}
return obj;
};
note that the declaration of f is missing in this code
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
- Linguagem predominante
- JavaScript
- Estrelas
- 470
- Forks
- 92
- Merge médio
- 1d 12h
- PRs com merge (30d)
- 6
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Sem guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de protocolbuffers/protobuf-javascript
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 30/100
protocolbuffers/protobuf-javascript#248 · 1 comentário · 13 reações ·
-
question
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 45/100
protocolbuffers/protobuf-javascript#222 · 9 comentários ·
-
Why map.js sort keys?Abertaenhancement port-fix triaged
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
protocolbuffers/protobuf-javascript#185 · 1 comentário ·
-
enhancement port-fix triaged
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
protocolbuffers/protobuf-javascript#182 · 3 comentários · 1 reação ·
Todas as issues de protocolbuffers/protobuf-javascript
Issues semelhantes
-
app_installations reports an error on every repo or suborg change when the plugin is not configuredAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 64/100
github-community-projects/safe-settings#1111 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Mantenedores costumam responder em até 1 dia
-
needs triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
pbakaus/impeccable#1055 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 82/100
guaguastandup/zotero-pdf2zh#368 ·
-
Add: EUROTV NEWSAbertachannels:add check:passed
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100