Underscore: 1.9.0 regressão. Chamar o primeiro ou o último em uma matriz vazia retorna indefinido.

Criado em 19 abr. 2018  ·  9Comentários  ·  Fonte: jashkenas/underscore

Com 1.9.0 _.first([], 10) retorna undefined vez de [] .

Acho que este é um grande problema: stick_out_tongue:

bug fixed

Comentários muito úteis

Bem! Underscore 1.9.1 foi publicado, com esta correção de bug nele.

Todos 9 comentários

Uau ... Parece que essa foi uma mudança intencional ... https://github.com/jashkenas/underscore/pull/2513

Pessoalmente, acho que você esperaria que _.first([], 2) fosse [] seguindo de _.first([1,2,3], 2) sendo [1,2]

Isso parece uma pequena regressão desagradável para mim - quando chamado com o número de itens especificados explicitamente. Alguma outra opinião? @jridgewell?

Dito isso, levanta a questão de o que _.first([], 1) deveria ser.

Opa, crítica ruim da minha parte. Passar um argumento n sempre deve retornar um array.

Não se preocupe, podemos fazer um 1.9.1 com quaisquer outras limpezas que surgirem - além de talvez resolver alguns dos outros PRs abertos ...

gracias. este também nos mordeu hoje.

Este leva a bugs muito obscuros em casos extremos em vários lugares em um aplicativo maior no meu caso.

Não deveria ser uma prioridade?

Este também nos mordeu. Felizmente, ele foi detectado por alguns testes de unidade antes de implementá-lo ao vivo. Eu ficaria grato por uma solução rápida.

Bem! Underscore 1.9.1 foi publicado, com esta correção de bug nele.

Obrigado @jashkenas

Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

jdalton picture jdalton  ·  6Comentários

chikamichi picture chikamichi  ·  8Comentários

githublyp picture githublyp  ·  3Comentários

sky0014 picture sky0014  ·  8Comentários

haggholm picture haggholm  ·  8Comentários