geocoder-php / geocoder-php/Geocoder
No interface for formatters?
Personne n'a encore pris cette issue.
- Langage dominant
- PHP
- Étoiles
- 4k
- Forks
- 525
- Merge moyen
- 8 min
- PR mergées (30 j)
- 1
Description
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?
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par lire src/Common/Formatter/StringFormatter.php et examinez sa méthode format() ainsi que le setter qui a besoin d'un typehint. Confirmez l'emplacement proposé de FormatterInterface.php et le contrat de format(), puis examinez l'utilisation actuelle du formatter. Le travail est terminé lorsque l'interface du formatter est définie de manière cohérente et que le setter interchangeable peut utiliser cette interface comme typehint.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- php
- Domaine
- backend
- Type d'issue
- Fonctionnalité
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 38/100