diff options
| author | Simon Charette <charettes@users.noreply.github.com> | 2019-10-31 15:17:04 -0400 |
|---|---|---|
| committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2019-10-31 20:17:04 +0100 |
| commit | a699595fce27d5d9c45ba83c656319593c366be9 (patch) | |
| tree | 9a7c31b8cf9502d04b8e68d4598596032fc814d6 | |
| parent | a9bd01d363376d2e88f96cbf81bc6b77b0ff2bce (diff) | |
Refs #13312 -- Removed unnecessary IF wrapping in nulls_last handling on MySQL.
ISNULL function already returns 0 and 1 on MySQL.
| -rw-r--r-- | django/db/models/expressions.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/django/db/models/expressions.py b/django/db/models/expressions.py index 2eae16541e..44a52c1dc9 100644 --- a/django/db/models/expressions.py +++ b/django/db/models/expressions.py @@ -1156,7 +1156,7 @@ class OrderBy(BaseExpression): def as_mysql(self, compiler, connection): template = None if self.nulls_last: - template = 'IF(ISNULL(%(expression)s),1,0), %(expression)s %(ordering)s ' + template = 'ISNULL(%(expression)s), %(expression)s %(ordering)s ' elif self.nulls_first: template = 'IF(ISNULL(%(expression)s),0,1), %(expression)s %(ordering)s ' return self.as_sql(compiler, connection, template=template) |