Docs for CurrentSiteManager indicate an M2M field called "sites" will be auto-detected on a model when it won't
|Reported by:||Gabriel Hurley||Owned by:||Gabriel Hurley|
|Cc:||Triage Stage:||Ready for checkin|
|Has patch:||yes||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
Throughout the Sites Framework docs the example models use
sites = ManyToManyField(Site), and under the CurrentSiteManager section it indicates that CurrentSiteManager will be able to automatically discover either a ForeignKey called "site" or a ManyToManyField called "sites", when in fact the code only looks for a field called "site".
While it'd be an easy docs fix, it seems more valuable to fix the code for the manager to actually do what the docs say.
I'm happy to write up a patch and tests for it later.
Change History (7)
comment:2 Changed 6 years ago by
|Owner:||changed from nobody to Gabriel Hurley|
|Status:||new → assigned|