KnpLabs / KnpLabs/php-github-api
ResultPager::get() can return string
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 2.2k
- Forks
- 585
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Because ResultPager::get() here returns the return value of ResponseMediator::getContent(), it can occasionally return a string, which causes an unhelpful TypeError since it should only ever return an array. Other usages of ResponseMediator::getContent() allow for this, but the ResultPager one does not.
This has been occurring in practice to us so figured I'd raise it here - let me know any other information you need on this issue, it is my first one!
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie in lib/Github/ResultPager.php bei ResultPager::get() und vergleichen Sie dessen Verwendung von ResponseMediator::getContent() mit dem in lib/Github/HttpClient/Message/ResponseMediator.php beschriebenen Verhalten. Bestätigen Sie, dass die Methode konsistent ein Array zurückgibt, ergänzen Sie eine Abdeckung für den Fall von String-Inhalten und verifizieren Sie, dass der daraus resultierende TypeError nicht mehr auftritt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- api
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 45/100