AutoRouterImproved extendability
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
Piste de recherche
Commencez par lire AutoRouterImproved et les méthodes du routeur parent qu’il utilise actuellement pour trouver le contrôleur et le nom de la méthode. Comparez les deux approches proposées : réutiliser le comportement du routeur parent ou rendre les auto-routeurs remplaçables et extensibles via des services spécifiques à l’application. Le travail est considéré comme terminé lorsqu’une conception a été choisie et qu’une implémentation permet de modifier le contrôleur et le nom de la méthode sans dupliquer le routeur.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
In the current state, the AutoRouterImproved is not thought to be replaced by an App-Specific service and to be extended on.
kenjis: Because it is better to implement the interface, not extending the class.
In my case, I need to change the controller & method name which is already implemented in the router.
Re-Implementing the AutoRouterImproved to do that seems kinda ridiculous.
Two approaches come to my mind:
- The specific solution: Use the "parent" routers methods for finding the method and controller name, this prevents duplicate code and allows for a little app-specific behavior.
- The generic solution: Make the auto routers replaceable by app-specific services
- Optional: make both extendable, so we don't have to write ridiculous amounts of code for small changes.
- Langage dominant
- PHP
- Étoiles
- 6k
- Forks
- 2k
- Merge moyen
- 1 j 22 h
- PR mergées (30 j)
- 59
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire 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.
Autres issues de codeigniter4/CodeIgniter4
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
codeigniter4/CodeIgniter4#10616 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
codeigniter4/CodeIgniter4#10618 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 62/100
codeigniter4/CodeIgniter4#10617 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
codeigniter4/CodeIgniter4#10615 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
codeigniter4/CodeIgniter4#10589 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de codeigniter4/CodeIgniter4
Issues similaires
-
sync-en
Difficulté 1/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 4 jours
-
Перевод устарел
Difficulté 2/5 Une demi-journée Accessibilité débutants 62/100
-
feature request
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
home php
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
vincent-peugnet/wcms#670 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
freescout-help-desk/freescout#5731 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour