Opened 5 years ago
Closed 5 years ago
#31477 closed Cleanup/optimization (fixed)
Simplify signature of `DatabaseOperations.execute_sql_flush()`
Reported by: | Jon Dufresne | Owned by: | nobody |
---|---|---|---|
Component: | Uncategorized | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
The current signature is:
def execute_sql_flush(self, using, sql_list):
The using
argument can be dropped and inferred by the calling instance: self.connection.alias
.
def execute_sql_flush(self, sql_list):
Some internal ises of this method are already doing:
connection.ops.execute_sql_flush(connection.alias, sql_flush)
Change History (3)
comment:1 by , 5 years ago
Has patch: | set |
---|
comment:2 by , 5 years ago
Triage Stage: | Unreviewed → Accepted |
---|
Note:
See TracTickets
for help on using tickets.
https://github.com/django/django/pull/12741