vimeo/psalm

`str_replace()` with more search then replace values

Open

#4253 aperta il 28 set 2020

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)PHP (668 fork)batch import
Help wantedenhancementtype providers

Metriche repository

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

Descrizione

even if it is supported by php-src, I think it should be an error (or at least warning) when using str_replace with less replace args, then search args. its most likely a typo and the default behaviour of php-src assuming we want to replace with an empty string is not obvious.

I expect an error, telling me that the number of replace args is not equal with the number of search args in https://psalm.dev/r/7385fe7881

Guida contributor

Direzione di ricerca
Ricerca come Psalm rileva i problemi nelle chiamate di funzione. Esamina le implementazioni esistenti per funzioni integrate simili (es. funzioni array) e crea un nuovo controllo per str replace che emette un avviso quando il numero di parametri di ricerca supera il numero di parametri di sostituzione.
Tech stack
php
Dominio
devtools
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
PHPGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65