vimeo/psalm

array_unique 2nd parameter may restrict the array items type

Aberta

#7.956 aberto em 13 de mai. de 2022

Ver no GitHubGuia do colaborador
 (11 comentários) (0 reação) (0 responsável)PHP (668 forks)batch import
Help wantedeasy problemsinternal stubs/callmap

Métricas do repositório

Stars
 (5.369 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Hi,

the array_unique() function has an optional 2nd parameter flag, which is one of the following value:

  • SORT_REGULAR
  • SORT_NUMERIC
  • SORT_STRING (default value)
  • SORT_LOCALE_STRING

if the flag is one of SORT_LOCALE or SORT_LOCALE_STRING, the array should only contain strings or values that can be casted to string. I'm not sure exactly what the definition should be, maybe: int|float|string|bool|\Stringable|null

Here we can see it fail : https://3v4l.org/lJsCp

and here we can see that Psalm does not find any error: https://psalm.dev/r/2135f5de25

edit: I realized that the SORT_NUMERIC also restrict the item type, probably something like int|float|string|null

Guia do colaborador

Direção de pesquisa
Explore o código base do Psalm para encontrar como array unique está atualmente tipado e, em seguida, adicione um tipo de retorno condicional com base no segundo parâmetro (SORT REGULAR, SORT NUMERIC, SORT STRING, SORT LOCALE STRING) usando o sistema de tipos do Psalm.
Pilha de tecnologia
php
Domain
tooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PHP
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60