reactjs / reactjs/react.dev

Change Missing From Changelog: forEach() Loop's `Return` returns to forEach()'s invocation point, not to the calling function.

Offen
#2,732 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
JavaScript
Sterne
11.8k
Forks
7.9k
Ø Merge
1 T. 11 Std.
Gemergte PRs (30 T.)
11

Beschreibung

Hey all,

I'm noticing that in upgrading to ReactJS Version 16.12.0 from 16.7.0, the behavior of foreach was changed, so that return returns to where foreach is called, and not the function calling the function containing the foreach. But I don't see this change here:

https://github.com/facebook/react/blob/master/CHANGELOG.md

Basically,

function1() {
	function2();
	console.log("function 1");
}

function2() {
	const arr = ['cool'];
	arr.forEach((var) => {
		return var;
	});
	console.log("function 2");
}

Before 16.12.0, console output was: "function 1", but now it is "function 2; function 1." This change appears to be against MDN documentation and internet consensus: https://stackoverflow.com/questions/34653612/what-does-return-keyword-mean-inside-foreach-function

But it should have been included in the changelog, though I am not seeing it. Let me know if I'm misunderstanding.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit CHANGELOG.md und dem im Issue referenzierten Changelog-Eintrag für React 16.12.0. Überprüfe das gemeldete Verhalten von forEach() anhand der verlinkten MDN- und Stack-Overflow-Referenzen und ermittle anschließend, ob im Changelog ein Eintrag fehlt; abgeschlossen ist die Aufgabe, wenn der Bericht validiert ist und der Changelog die Veröffentlichung gegebenenfalls korrekt widerspiegelt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, react
Bereich
documentation
Issue-Typ
Dokumentation
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
30/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.