Fixed #30463 -- Fixed crash of deprecation message when Meta.ordering contains expressions.

django/db/models/sql/compiler.py

@@ -553,9 +553,9 @@ def as_sql(self, with_limits=True, with_col_aliases=False):
                         # order_by = None
                         warnings.warn(
                             "%s QuerySet won't use Meta.ordering in Django 3.1. "
-                            "Add .order_by('%s') to retain the current query." % (
+                            "Add .order_by(%s) to retain the current query." % (
                                 self.query.model.__name__,
-                                "', '".join(self._meta_ordering)
+                                ', '.join(repr(f) for f in self._meta_ordering),
                             ),
                             RemovedInDjango31Warning,
                             stacklevel=4,

docs/releases/2.2.2.txt

@@ -11,3 +11,6 @@ Bugfixes
 
 * Fixed a regression in Django 2.2 that stopped Show/Hide toggles working on
   dynamically added admin inlines (:ticket:`30459`).
+
+* Fixed a regression in Django 2.2 where deprecation message crashes if
+  ``Meta.ordering`` contains an expression (:ticket:`30463`).

tests/ordering/models.py

@@ -14,6 +14,7 @@
 """
 
 from django.db import models
+from django.db.models.expressions import OrderBy
 
 
 class Author(models.Model):
@@ -30,7 +31,12 @@ class Article(models.Model):
     pub_date = models.DateTimeField()
 
     class Meta:
-        ordering = ('-pub_date', 'headline')
+        ordering = (
+            '-pub_date',
+            'headline',
+            models.F('author__name').asc(),
+            OrderBy(models.F('second_author__name')),
+        )
 
     def __str__(self):
         return self.headline

tests/ordering/tests.py

@@ -408,7 +408,9 @@ def test_default_ordering_by_f_expression(self):
     def test_deprecated_values_annotate(self):
         msg = (
             "Article QuerySet won't use Meta.ordering in Django 3.1. Add "
-            ".order_by('-pub_date', 'headline') to retain the current query."
+            ".order_by('-pub_date', 'headline', OrderBy(F(author__name), "
+            "descending=False), OrderBy(F(second_author__name), "
+            "descending=False)) to retain the current query."
         )
         with self.assertRaisesMessage(RemovedInDjango31Warning, msg):
             list(Article.objects.values('author').annotate(Count('headline')))