geocoder-php / geocoder-php/Geocoder
No interface for formatters?
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 4k
- Forks
- 525
- Ø Merge
- 8 Min.
- Gemergte PRs (30 T.)
- 1
Beschreibung
I'm building a wrapper for this library and a service class for my app and when I wanted to make the formatter exchangeable I realized that I can't typehint the setter against an interface.
https://github.com/geocoder-php/Geocoder/blob/master/src/Common/Formatter/StringFormatter.php
If you agree I'll create a PR and add a Formatter interface that implements the format() method.
Would \Geocoder\Formatter\FormatterInterface.php work for you?
Beitragsleitfaden
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
Beginne mit dem Lesen von src/Common/Formatter/StringFormatter.php und untersuche dessen format()-Methode sowie den Setter, der einen Typehint benötigt. Bestätige den vorgeschlagenen Speicherort von FormatterInterface.php und den format()-Vertrag und berücksichtige anschließend die bestehende Verwendung des Formatters. Als erledigt gilt die Aufgabe, wenn das Formatter-Interface konsistent definiert ist und der austauschbare Setter darauf typehinten kann.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- backend
- Issue-Typ
- Feature
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100