vimeo/psalm

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

Open

#7 536 ouverte le 30 janv. 2022

Voir sur GitHub
 (6 commentaires) (0 réactions) (0 assignés)PHP (668 forks)batch import
Help wantedbugtype reconciliation

Métriques du dépôt

Stars
 (5 369 stars)
Métriques de merge PR
 (Merge moyen 3j 12h) (5 PRs mergées en 30 j)

Description

https://psalm.dev/r/dafc6637b5

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

The above error is unexpected.

Guide contributeur

Direction de recherche
Examinez le vérificateur RedundantFunctionCall de Psalm, probablement dans src/Psalm/Internal/Analyzer/Statements/Expression/Call/FunctionCallAnalyzer.php. Le faux positif se produit parce que Psalm suppose que toString renvoie des minuscules, mais il peut renvoyer n'importe quelle casse. La correction consisterait à ajuster l'inférence de type pour toString afin qu'elle n'implique pas de minuscules, ou à modifier la détection d'appel redondant. Le problème a des commentaires mais aucune solution claire proposée, donc il faut étudier la gestion de toString et strtolower par le système de types.
Stack technique
php
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
php
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75