auto-fix "Call to function in_array() requires parameter #3 to be set/true."
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 25/100
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- PHP
- Stelle
- 714
- Fork
- 63
- Merge medio
- 1g 11h
- PR unite (30g)
- 8
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di phpstan/phpstan-strict-rules
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
phpstan/phpstan-strict-rules#317 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
phpstan/phpstan-strict-rules#316 · 4 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 52/100
phpstan/phpstan-strict-rules#298 · 2 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
phpstan/phpstan-strict-rules#283 · 6 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Rule about bitwise operatorAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
phpstan/phpstan-strict-rules#276 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di phpstan/phpstan-strict-rules
Issue simili
-
claude
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
sprain/sprainocchio#26 ·
-
sync-en
Difficoltà 1/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno
-
sync-en
Difficoltà 1/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 83/100
GSA/digital-gov-drupal#569 ·
I maintainer di solito rispondono entro 2 giorni
-
feature-request-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
Codeinwp/wp-menu-icons#360 ·