vimeo/psalm

implode(<non-false-string>, array) should produce non-falsy-string

Open

#7 055 ouverte le 4 déc. 2021

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)PHP (668 forks)batch import
Help wantedeasy problemsenhancementtype providers

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

When the first argument is a non-falsy-string and the second argument has at least two elements, then the result is guaranteed to be at least non-falsy-string.

<?php

if(implode(' ', ['',''])) echo 1;

https://psalm.dev/r/19514cf5db

Guide contributeur

Direction de recherche
Inspectez l'inférence de type pour implode() dans le code source de Psalm, en vous concentrant sur la façon dont le type de retour est déterminé à partir du séparateur et des arguments du tableau. Consultez les cas de test existants pour implode et ajoutez un test qui montre le résultat attendu de non falsy string lorsque le séparateur est une chaîne non fausse et que le tableau a au moins deux éléments.
Stack technique
php
Domaine
developer experiencetooling
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
PHPGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75