Opened 6 years ago
Closed 6 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 , 6 years ago
| Has patch: | set |
|---|
comment:2 by , 6 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
Note:
See TracTickets
for help on using tickets.
https://github.com/django/django/pull/12741