coderedcorp / coderedcorp/coderedcms
Use correct site when loading settings
- Langage dominant
- Python
- Étoiles
- 765
- Forks
- 154
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
When loading wagtail settings, such as `GoogleApiSettings` or `MailchimpApiSettings` we have a tendency to do these in a place where no `Site` is available and explicitly load the default site. This is a bad practice as it totally breaks the concept of multi-site.
This practice needs to be refactored to get the site from the page or request, and if those are not available, use dependency injection techniques.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Piste de recherche
Commencez par localiser les utilisations de GoogleApiSettings et MailchimpApiSettings qui chargent explicitement la Site par défaut, puis vérifiez si chaque appelant dispose d’un contexte de page ou de requête. Identifiez les appelants qui nécessitent l’injection de dépendances ; le travail est terminé lorsque les paramètres résolvent la Site pertinente dans les contextes multisite sans revenir à la Site par défaut lorsqu’un contexte est disponible.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- django, python
- Domaine
- backend
- Type d'issue
- Refactorisation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 35/100