vimeo/psalm

Different results between `DateTime` and `DateTimeImmutable`

Open

#5220 aperta il 14 feb 2021

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)PHP (668 fork)batch import
Help wantedbuginternal stubs/callmap

Metriche repository

Star
 (5369 star)
Metriche merge PR
 (Merge medio 3g 12h) (5 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Indaga il percorso del codice per DateTime vs DateTimeImmutable nel sistema di tipi di Psalm. Esamina la gestione delle classi integrate e dei parametri nominati. Il problema è che DateTime ha un comportamento diverso da DateTimeImmutable, ma dovrebbero essere coerenti. Esamina la PR #5218 come riferimento per risolvere problemi simili.
Tech stack
php
Dominio
backendtooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
PHP
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50