Document jQuery.isEmptyObject doesn't take symbols into account
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 35/100
- Tipo de issue
- Documentação
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- jquery
- Domínio
- documentation
Direção de pesquisa
Comece localizando a documentação da API jQuery.isEmptyObject e compare sua descrição atual com o comportamento descrito na issue. Confirme se a documentação deve explicar as propriedades Symbol e as propriedades da cadeia de protótipos; em seguida, atualize a documentação relevante e verifique se a entrada renderizada reflete o comportamento acordado.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Description
On version 3.6.0, I found a vulnerability，I find that the method of detecting empty objects is not perfect.
isEmptyObject: function( obj ) {
var name;
for ( name in obj ) {
return false;
}
return true;
},
The downside of writing this is that the properties of the prototype chain and the Symbol type are not taken into account，I have a better way to implement it as follows.
isEmptyObject: function( obj ) {
var keys = Object.keys(obj);
// 看浏览器是否支持 Symbol
if (typeof Symbol !== "undefined") {
keys = keys.concat(Object.getOwnPropertySymbols(obj));
}
return keys.length === 0;
}
Link to test case
This direct modification of the source code test is good, I do not have a service to run this test. I'm looking forward to hearing from you.
- Linguagem predominante
- HTML
- Estrelas
- 324
- Forks
- 259
- Merge médio
- 1d 30min
- PRs com merge (30d)
- 1
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o 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 jquery/api.jquery.com
-
Mention of ‘timestamp’ in documentation for ‘cache’ parameter of ‘ajax’ is misleadingTalvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Aberta
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 62/100
jquery/api.jquery.com#1241 · 2 comentários ·
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 68/100
jquery/api.jquery.com#1176 ·
-
The documentation of CSS methods should be further classifiedTalvez livre de novo Um pull request para esta issue foi fechado sem ser mesclado. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 35/100
jquery/api.jquery.com#1281 · 2 comentários ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 35/100
jquery/api.jquery.com#1271 · 2 comentários ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 45/100
jquery/api.jquery.com#1259 ·
Todas as issues de jquery/api.jquery.com
Issues semelhantes
-
README says the land/sea mask must match the XS grid; the demo mask does not, and Step 4 worksAbertadocumentation
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
buckai-observatory/Automated-Cloud-Removal-and-Seamless-Satellite-Image-Mosaicking-Pipeline#52 ·
Mantenedores costumam responder em até 1 dia
-
documentation
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 84/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 82/100
run-llama/llama_index#24163 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 82/100
btclib-org/btclib#3090 ·
Mantenedores costumam responder em até 1 dia
-
Translate the READMEAbertadocumentation good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
asasemahmed/openboard#68 ·
Mantenedores costumam responder em até 1 dia