codeigniter4/CodeIgniter4

Dev: replace type `mixed` with more specific types

Chiusa

#6310 aperta il 29 lug 2022

Vedi su GitHubGuida contributor
 (11 commenti) (0 reazioni) (0 assegnatari)PHP (1707 fork)batch import
devgood first issuehelp wanted

Metriche repository

Star
 (4492 stelle)
Metriche merge PR
 (Merge medio 4g 15h) (68 PR mergiate in 30 g)

Descrizione

Type mixed means object|resource|array|string|int|float|bool|null. https://www.php.net/manual/en/language.types.declarations.php#language.types.declarations.mixed

The current code base has a lot of type mixed in PHPDocs, But I think all of them are not correct.

And mixed prevents PHPStan level from increasing.

PHPStan’s rule level 6 isn’t satisfied with implicit mixed https://phpstan.org/writing-php-code/phpdoc-types#mixed

We need to replace type mixed with more specific types.

Guida contributor

Direzione di ricerca
Cerca tutti i suggerimenti di tipo `mixed` nei PHPDoc in tutto il codice e determina il tipo specifico corretto per ciascuno in base al contesto (es. utilizzo dei parametri, valori di ritorno).
Tech stack
php
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
PHPPHPDoc
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70