#25681 closed Bug (fixed)
Possible minor contradiction in DATABASES setting depicted in multiple database/routing docs example.
Reported by: | aleks | Owned by: | |
---|---|---|---|
Component: | Documentation | Version: | 1.8 |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | yes | UI/UX: | no |
Description
Whilst perusing the (excellent) documentation, I noticed that in the 'Multiple databases' docs (https://docs.djangoproject.com/en/1.8/topics/db/multi-db/), the section 'Defining your databases' states: "Django requires that a default database entry be defined, but the parameters dictionary can be left blank if it will not be used"...
However in the example later in this section (https://docs.djangoproject.com/en/1.8/topics/db/multi-db/#an-example), the default value is not defined.
I fully appreciate this is an example, which as stated skips some complex considerations, however from what I understood, the default value is a *requirement* not a consideration as such?
Or have I misunderstood/misread the requirement for the 'default' value when using custom routers? (in which case my apologies!)
Change History (7)
comment:1 by , 9 years ago
Easy pickings: | set |
---|---|
Triage Stage: | Unreviewed → Accepted |
Type: | Uncategorized → Bug |
comment:2 by , 9 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:4 by , 9 years ago
Owner: | removed |
---|---|
Status: | assigned → new |
Triage Stage: | Accepted → Ready for checkin |
Setting DATABASES
as stated in current documentation returns an error.
Linked PR fixes the issue
I believe the correct solution is to add
'default': {}
to those examples, but we should double check.