auto-fix "Call to function in_array() requires parameter #3 to be set/true."
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 25/100
Piste de recherche
Read the linked pull request #349 first; it already has this issue in progress. The issue describes the fixer behavior for in_array(), array_search() and eligible array_keys() calls, including named arguments, splat calls, and types inferred from PHPDoc. Done means only calls whose strict equivalence can be proven are changed, while base64_decode() and array_keys() calls without a search value remain unaffected.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
if we know native haystack and needle types for a in_array, array_keys or array_search call, it would be great we could fix add a true as 3rd parameter in case its missing or can always be true. we should only fix when haystacks' iterable type is a super type of needle (strict comparison). the fixer must not change calls where equivalence cannot be proven.
Example: in_array($needle, X::VALS) into its strict counterpart in_array($needle, X::VALS, true).
https://phpstan.org/r/e288cf2a-08e5-4c06-9255-1a98eefc99fe
named arguments should be supported and calls using the splat operator should be skipped.
we should not fix calls which get their iterable-value type from a phpdoc, e.g.:
/** @return int[] */
function getArray(): array {}
function doFoo(int $i):bool {
return in_array($i, getArray());
}
For array_keys(), only consider calls with at least two arguments, since the strictness flag matters only when a search value is supplied. Calls without a search value should remain unchanged.
The rule also reports calls to other functions, such as base64_decode(). Those calls should not be made fixable by this change.
- Langage dominant
- PHP
- Étoiles
- 714
- Forks
- 63
- Merge moyen
- 1 j 11 h
- PR mergées (30 j)
- 8
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de phpstan/phpstan-strict-rules
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
phpstan/phpstan-strict-rules#317 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
phpstan/phpstan-strict-rules#316 · 4 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 52/100
phpstan/phpstan-strict-rules#298 · 2 commentaires · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
phpstan/phpstan-strict-rules#283 · 6 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Rule about bitwise operatorOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
phpstan/phpstan-strict-rules#276 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de phpstan/phpstan-strict-rules
Issues similaires
-
claude
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
sprain/sprainocchio#26 ·
-
sync-en
Difficulté 1/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
-
sync-en
Difficulté 1/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 83/100
GSA/digital-gov-drupal#569 ·
Les mainteneurs répondent en général sous 2 jours
-
feature-request-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
Codeinwp/wp-menu-icons#360 ·