Opened 6 years ago

Closed 5 years ago

#19657 closed Bug (fixed)

sql, sqlcustom, sqlindexes and sqlall do not take allow_syncdb into account

Reported by: manelclos@… Owned by: nobody
Component: Core (Management commands) Version: master
Severity: Normal Keywords: sql sqlall database router
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


When synchronizing the database, sql for models will appear in all databases, this is, not respecting allow_syncdb returning False.

Change History (9)

comment:1 Changed 6 years ago by manelclos@…

Has patch: set

comment:2 Changed 6 years ago by Claude Paroz

Needs tests: set
Patch needs improvement: set
Triage Stage: UnreviewedAccepted
Type: UncategorizedBug

I don't think you need to add a new parameter to the sql_* functions, as you should be able to get the db alias through connection.alias (to be confirmed).

I noticed that the sql_* were missing tests. This might need a new regressiontests/sql_commands test directory.

comment:3 Changed 6 years ago by Claude Paroz

I've just added some base tests for those commands: [718afcafc202940cd15799459178a2534b8217d5]

comment:4 Changed 6 years ago by Claude Paroz

Needs tests: unset
Patch needs improvement: unset

comment:5 Changed 6 years ago by Chris Wilson

Patch needs improvement: set

Please could you add tests that the correct value of include_auto_created is passed for each set of SQL statements, and that it's honoured correctly by the filtered_app_models function?

Actually, would it make more sense for filtered_app_models to be a method of Router, so that it's easier to override?

comment:6 Changed 6 years ago by Claude Paroz

Patch needs improvement: unset

I've just added a second commit to the PR to address your latest concern.

comment:7 Changed 5 years ago by Claude Paroz

Version: 1.4master

Pull request just updated.

comment:8 Changed 5 years ago by Tim Graham

Triage Stage: AcceptedReady for checkin

comment:9 Changed 5 years ago by Claude Paroz <claude@…>

Resolution: fixed
Status: newclosed

In 2992f428614349b0dfe0f4183905f492fd3f62c2:

Fixed #19657 -- Made sql commands honor allow_migrate

Thanks Manel Clos for the report and the initial patch, and
Marc Tamlyn and Tim Graham for the review.

Note: See TracTickets for help on using tickets.
Back to Top