vimeo/psalm

Different results between `DateTime` and `DateTimeImmutable`

Open

#5.220 aberto em 14 de fev. de 2021

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)PHP (668 forks)batch import
Help wantedbuginternal stubs/callmap

Métricas do repositório

Stars
 (5.369 stars)
Métricas de merge de PR
 (Mesclagem média 3d 12h) (5 fundiu PRs em 30d)

Description

I noticed that DateTime and DateTimeImmutable works different way, but those should work as the same.

I first noticed this with PHP 8 named parameters and DateTimeImmutable usage, and I made fix for that part - https://github.com/vimeo/psalm/pull/5218

But now I'm still seeing different handling on DateTime class - and I cannot find the code where that is done.

https://psalm.dev/r/129b9fce2b https://psalm.dev/r/ee31f26136

Guia do colaborador

Direção de pesquisa
Investigue o caminho do código para DateTime vs DateTimeImmutable no sistema de tipos do Psalm. Veja o tratamento de classes embutidas e parâmetros nomeados. O problema é que DateTime tem comportamento diferente de DateTimeImmutable, mas eles devem ser consistentes. Consulte o PR #5218 como referência para corrigir problemas semelhantes.
Pilha de tecnologia
php
Domain
backendtooling
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.
3
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.
Principalmente claro
Prerequisites
PHP
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50