Document jQuery.isEmptyObject doesn't take symbols into account
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Documentazione
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- jquery
- Ambito
- documentation
Direzione di ricerca
Inizia individuando la documentazione dell’API jQuery.isEmptyObject e confronta la descrizione attuale con il comportamento descritto nell’issue. Verifica se la documentazione debba spiegare le proprietà Symbol e le proprietà della catena dei prototipi, quindi aggiorna la documentazione pertinente e verifica che la voce renderizzata rifletta il comportamento concordato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- HTML
- Stelle
- 324
- Fork
- 259
- Merge medio
- 1g 30m
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di jquery/api.jquery.com
-
Mention of ‘timestamp’ in documentation for ‘cache’ parameter of ‘ajax’ is misleadingForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 62/100
jquery/api.jquery.com#1241 · 2 commenti ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 68/100
jquery/api.jquery.com#1176 ·
-
The documentation of CSS methods should be further classifiedForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 35/100
jquery/api.jquery.com#1281 · 2 commenti ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 35/100
jquery/api.jquery.com#1271 · 2 commenti ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
jquery/api.jquery.com#1259 ·
Tutte le issue di jquery/api.jquery.com
Issue simili
-
5-minutes algorithm all-levels any-language beginner-friendly community documentation easy first-timers-only good first issue hacktoberfest help wanted learning no-code-needed open-source python students
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 88/100
Huerte/algo-solutions-python#253 ·
I maintainer di solito rispondono entro 1 giorno
-
size/s
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
sipyourdrink-ltd/bernstein#7179 ·
I maintainer di solito rispondono entro 1 giorno
-
area/docs
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 86/100
meshery/meshery#23375 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Synthetic
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 66/100
samtoth/agda-synthetic-categories#373 ·
I maintainer di solito rispondono entro 1 giorno
-
agent-system-test-1319336938-43312257375-1- Agent System assignment-policy flow 43312257375-1Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 62/100
tanaabased/big-test-bucket#19952 ·
I maintainer di solito rispondono entro 1 giorno