vimeo/psalm

The result of __toString is mistakenly assumed to be always lower-cased

Open

#7536 aperta il 30 gen 2022

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)PHP (668 fork)batch import
Help wantedbugtype reconciliation

Metriche repository

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

Descrizione

https://psalm.dev/r/dafc6637b5

ERROR: RedundantFunctionCall - 11:6 - The call to strtolower is unnecessary

The above error is unexpected.

Guida contributor

Direzione di ricerca
Esamina il checker RedundantFunctionCall di Psalm, probabilmente in src/Psalm/Internal/Analyzer/Statements/Expression/Call/FunctionCallAnalyzer.php. Il falso positivo si verifica perché Psalm presume che toString restituisca lettere minuscole, ma può restituire qualsiasi combinazione di maiuscole/minuscole. La correzione consisterebbe nell'adeguare l'inferenza del tipo per toString in modo che non implichi minuscole, o nel modificare il rilevamento delle chiamate ridondanti. Il problema ha commenti ma nessuna soluzione chiara proposta, quindi è necessario indagare la gestione di toString e strtolower nel sistema di tipi.
Tech stack
php
Dominio
tooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
php
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75