Underscore: 1.9.0 régression. L'appel en premier ou en dernier sur un tableau vide renvoie undefined.

Créé le 19 avr. 2018  ·  9Commentaires  ·  Source: jashkenas/underscore

Avec 1.9.0 _.first([], 10) renvoie undefined au lieu de [] .

Je pense que c'est un gros problème :stuck_out_tongue:

bug fixed

Commentaire le plus utile

Bien! Underscore 1.9.1 est maintenant publié, avec cette correction de bogue.

Tous les 9 commentaires

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

Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

arieljake picture arieljake  ·  4Commentaires

sky0014 picture sky0014  ·  8Commentaires

chikamichi picture chikamichi  ·  8Commentaires

clouddueling picture clouddueling  ·  3Commentaires

githublyp picture githublyp  ·  3Commentaires