vimeo/psalm

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

Open

#7.055 geöffnet am 4. Dez. 2021

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)PHP (668 Forks)batch import
Help wantedeasy problemsenhancementtype providers

Repository-Metriken

Stars
 (5.369 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 3T 12h) (5 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die Typinferenz für implode() im Quellcode von Psalm, wobei der Schwerpunkt darauf liegt, wie der Rückgabetyp aus dem Trennzeichen und den Array Argumenten bestimmt wird. Sieh dir die vorhandenen Testfälle für implode an und füge einen Test hinzu, der das erwartete Ergebnis einer nicht falschen Zeichenkette zeigt, wenn das Trennzeichen eine nicht falsche Zeichenkette ist und das Array mindestens zwei Elemente enthält.
Tech Stack
php
Domain
developer experiencetooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PHPGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75