Avec 1.9.0 _.first([], 10)
renvoie undefined
au lieu de []
.
Je pense que c'est un gros problème :stuck_out_tongue:
Wow... On dirait presque que c'était un changement intentionnel... https://github.com/jashkenas/underscore/pull/2513
Personnellement, je pense que vous vous attendriez _.first([], 2)
ce que []
après que _.first([1,2,3], 2)
soit [1,2]
Cela ressemble à une petite régression désagréable pour moi - lorsqu'il est appelé avec le nombre d'éléments spécifiés explicitement. D'autres avis ? @jridgewell ?
Cela dit, cela soulève la question de savoir ce que devrait être _.first([], 1)
.
Oups, mauvaise critique de ma part. Passer un argument n
doit toujours renvoyer un tableau.
Pas de soucis, nous pouvons faire une version 1.9.1 avec tous les autres nettoyages qui surviennent - et peut-être régler certains des autres PR ouverts...
gracias. celui-ci nous a mordus aussi aujourd'hui.
Celui-ci conduit à des bogues très obscurs dans les cas extrêmes à de nombreux endroits différents dans une application plus grande dans mon cas.
Cela ne devrait-il pas être une priorité?
Celui-ci nous a aussi mordu. Heureusement, il a été détecté par des tests unitaires avant de le déployer en direct. Je serais reconnaissant pour une solution rapide.
Bien! Underscore 1.9.1 est maintenant publié, avec cette correction de bogue.
Merci @jashkenas
Commentaire le plus utile
Bien! Underscore 1.9.1 est maintenant publié, avec cette correction de bogue.