Can't order_by annotated field if annotated field uses conditional expression

[Kris Fields]

Continuing with the Client example on ​https://docs.djangoproject.com/en/dev/ref/models/conditional-expressions/

from django.db import models

class Client(models.Model):
    REGULAR = 'R'
    GOLD = 'G'
    PLATINUM = 'P'
    ACCOUNT_TYPE_CHOICES = (
        (REGULAR, 'Regular'),
        (GOLD, 'Gold'),
        (PLATINUM, 'Platinum'),
    )
    name = models.CharField(max_length=50)
    registered_on = models.DateField()
    account_type = models.CharField(
        max_length=1,
        choices=ACCOUNT_TYPE_CHOICES,
        default=REGULAR,
    )

Client.objects.annotate(
    discount=Case(
        When(account_type=Client.GOLD, then=Value('5%')),
        When(account_type=Client.PLATINUM, then=Value('10%')),
        default=Value('0%'),
        output_field=CharField(),
        ),
    ).order_by('discount')

Error is:

<repr(<django.db.models.query.QuerySet at 0x10afaef10>) failed: AttributeError: 'WhereNode' object has no attribute 'resolve_expression'>

[Josh Smeaton]

See: ​https://github.com/django/django/pull/4239

Funnily enough, being explicit worked before this patch:

CaseTestModel.objects.filter(integerlte=2).annotate(test=Case(

When(integer=1, then=2),
When(integer=2, then=1),
default=3,
output_field=models.IntegerField(),

)).order_by(F('test').asc())

The problem is that the same instance of each When clause has resolve_expression called on it multiple times. Once for the annotation, and once again for the order_by. The problem is that When.resolve_expression converts the Q() into a WhereNode(), and overwrites the condition property. It's not idempotent.

[Tim Graham]

Seems like a candidate for backport.

[Josh Smeaton <josh.smeaton@…>]

In ceaf31adfff3801f1092a215f73704e15a70e90c:

Fixed #24420 -- Allowed ordering by case expressions

[Josh Smeaton <josh.smeaton@…>]

In 823f8cdbc91f85a8ab3cb1ccfec5659037b5c148:

[1.8.x] Fixed #24420 -- Allowed ordering by case expressions

Backport of ceaf31adfff3801f1092a215f73704e15a70e90c from master