NodeList.forEach drops an explicit null callback context
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 68/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- javascript, react-native
- Domaine
- mobile-dev
Piste de recherche
Commencez par l’implémentation de NodeList.forEach et suivez la manière dont les valeurs null et undefined de thisArg sont gérées. Ajoutez ou mettez à jour la couverture pour un contexte de callback explicitement défini à null, puis vérifiez que les callbacks strict reçoivent null, tandis que les contextes omis ou définis à undefined conservent le comportement existant.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Description
NodeList.forEach treats both null and undefined as the omitted thisArg fast path. An explicitly supplied null is therefore discarded, and a strict callback receives undefined instead of null, unlike standard Array and NodeList behavior.
Expected behavior
Only an omitted/undefined thisArg should use the unbound callback path. Explicit null should be forwarded as the callback receiver.
React Native Version
0.87.1 and current main
Affected Platforms
JavaScript runtime - All
- Langage dominant
- C++
- Étoiles
- 127k
- Forks
- 25.3k
- Merge moyen
- 1 j 23 h
- PR mergées (30 j)
- 4
Guide de contribution
Ouvrir le 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 react/react-native
-
Needs: Triage :mag:
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
react/react-native#58565 · 1 commentaire · 2 réactions ·
-
Needs: Author Feedback Needs: Repro
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
react/react-native#58555 · 4 commentaires · 1 réaction ·
-
Needs: Attention Needs: Repro
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
react/react-native#58526 · 2 commentaires ·
-
Needs: Author Feedback Needs: Repro
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
react/react-native#58448 · 1 commentaire ·
-
Needs: Attention Needs: Repro
Difficulté 2/5 Une demi-journée Accessibilité débutants 82/100
react/react-native#58441 · 2 commentaires ·
Toutes les issues de react/react-native
Issues similaires
-
Difficulté 1/5 1-3 heures Accessibilité débutants 92/100
autowarefoundation/autoware_universe#13413 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
-
automated-analysis bug memory-safety
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
-
Sensor initialization takes very long when `--initial-sim-time` is set to current UNIX timestamp Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
gazebosim/gz-sensors#662 · 1 commentaire ·