coderedcorp / coderedcorp/coderedcms
Use correct site when loading settings
- Vorherrschende Sprache
- Python
- Sterne
- 765
- Forks
- 154
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Beginnen Sie damit, Verwendungen von GoogleApiSettings und MailchimpApiSettings zu lokalisieren, die explizit die standardmäßige Site laden, und verfolgen Sie anschließend, ob jeder Aufrufer über einen Seiten- oder Request-Kontext verfügt. Identifizieren Sie Aufrufer, die Dependency Injection benötigen; abgeschlossen ist die Aufgabe, wenn die Einstellungen in Multisite-Kontexten die relevante Site auflösen, ohne auf die standardmäßige Site zurückzufallen, wenn ein Kontext verfügbar ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- django, python
- Bereich
- backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 35/100