Reverted "Fixed #26434 -- Removed faulty clearing of ordering field when missing from explicit grouping."

This reverts commit ea3a71c2d0.

The implementation was flawed, as self.group_by contains Cols, not aliases.
This commit is contained in:
Jacob Walls
2025-10-28 10:43:54 -04:00
parent 9ba3f74a46
commit 43933a1dca
2 changed files with 1 additions and 26 deletions

View File

@@ -2346,13 +2346,7 @@ class Query(BaseExpression):
query (not even the model's default).
"""
if not force and (
self.is_sliced
or self.distinct_fields
or self.select_for_update
or (
isinstance(self.group_by, tuple)
and not {*self.order_by, *self.extra_order_by}.issubset(self.group_by)
)
self.is_sliced or self.distinct_fields or self.select_for_update
):
return
self.order_by = ()

View File

@@ -171,25 +171,6 @@ class AggregationTests(TestCase):
for attr, value in kwargs.items():
self.assertEqual(getattr(obj, attr), value)
def test_count_preserve_group_by(self):
# new release of the same book
Book.objects.create(
isbn="113235613",
name=self.b4.name,
pages=self.b4.pages,
rating=4.0,
price=Decimal("39.69"),
contact=self.a5,
publisher=self.p3,
pubdate=datetime.date(2018, 11, 3),
)
qs = Book.objects.values("contact__name", "publisher__name").annotate(
publications=Count("id")
)
self.assertEqual(qs.count(), Book.objects.count() - 1)
self.assertEqual(qs.order_by("id").count(), Book.objects.count())
self.assertEqual(qs.extra(order_by=["id"]).count(), Book.objects.count())
def test_annotation_with_value(self):
values = (
Book.objects.filter(