vimeo/psalm

implode(<string>, non-empty-list<string>) doesn't always result in non-empty-string

Offen

#6.866 geöffnet am 08.11.2021

Auf GitHub ansehenContributor Guide
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)PHP (668 Forks)batch import
Help wantedbugeasy problemsgood first issueinternal stubs/callmap

Repository-Metriken

Stars
 (5.369 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

https://psalm.dev/r/9b7a45c338

The weird edge case of implode('asdf', ['']); can result in an empty string. Similarly if the first argument is an empty string then an array of any amount of empty strings will also produce an empty string

Contributor Guide

Research-Richtung
Untersuche, wie Psalm den Rückgabetyp von implode behandelt, insbesondere den Grenzfall, bei dem implode mit einem nicht leeren Trennzeichen und einem Array, das eine leere Zeichenkette enthält, eine leere Zeichenkette zurückgibt. Überprüfe die Typinferenzlogik und teste mit dem bereitgestellten Beispiel.
Tech Stack
php
Domain
tooling
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
PHPPsalm
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70