Opened 4 years ago

Closed 4 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:2 by Simon Charette, 4 years ago

Triage Stage: UnreviewedAccepted

comment:3 by GitHub <noreply@…>, 4 years ago

Resolution: fixed
Status: newclosed

In 5673d4b:

Fixed #31477 -- Removed "using" argument from DatabaseOperations.execute_sql_flush().

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