Mit 1.9.0 gibt _.first([], 10)
undefined
anstelle von []
.
Ich denke, das ist ein großes Problem :stuck_out_tongue:
Wow... Es sieht fast so aus, als wäre dies eine beabsichtigte Änderung... https://github.com/jashkenas/underscore/pull/2513
Ich persönlich denke, Sie würden erwarten, dass _.first([], 2)
[]
gefolgt von _.first([1,2,3], 2)
[1,2]
Das sieht für mich nach einer bösen kleinen Regression aus – wenn es mit der explizit angegebenen Anzahl von Elementen aufgerufen wird. Irgendwelche anderen Meinungen? @jridgewell?
Das heißt, es wirft die Frage auf, was _.first([], 1)
sein sollte.
Hoppla, schlechte Bewertung meinerseits. Die Übergabe eines n
-Arguments sollte immer ein Array zurückgeben.
Keine Sorge, wir können eine 1.9.1 mit allen anderen Bereinigungen durchführen - und vielleicht einige der anderen offenen PRs erledigen ...
grazien. dieser hat uns heute auch gebissen.
Dieser führt in meinem Fall zu sehr obskuren Fehlern in Randfällen an vielen verschiedenen Stellen in einer größeren App.
Sollte es nicht Priorität haben?
Dieser hat uns auch gebissen. Glücklicherweise wurde es von einigen Unit-Tests erfasst, bevor wir es live ausrollen. Für eine schnelle Lösung wäre ich dankbar.
In Ordung! Underscore 1.9.1 ist jetzt mit diesem Bugfix veröffentlicht.
Danke @jashkenas
Hilfreichster Kommentar
In Ordung! Underscore 1.9.1 ist jetzt mit diesem Bugfix veröffentlicht.