Fixed #28391 -- Fixed Cast() with CharField and max_length on MySQL.

Thanks Tim Graham for the review.

4 files changed, 12 insertions, 4 deletions

diff --git a/django/db/backends/base/features.py b/django/db/backends/base/features.py
index 41a456bfed..0cc60fd450 100644
--- a/django/db/backends/base/features.py
+++ b/django/db/backends/base/features.py
@@ -234,6 +234,9 @@ class BaseDatabaseFeatures:
     # Does the backend support indexing a TextField?
     supports_index_on_text_field = True
 
+    # Does the backend support CAST with precision?
+    supports_cast_with_precision = True
+
     def __init__(self, connection):
         self.connection = connection
 
diff --git a/django/db/backends/sqlite3/features.py b/django/db/backends/sqlite3/features.py
index 80af6068ff..c82d565b6e 100644
--- a/django/db/backends/sqlite3/features.py
+++ b/django/db/backends/sqlite3/features.py
@@ -30,6 +30,7 @@ class DatabaseFeatures(BaseDatabaseFeatures):
     can_clone_databases = True
     supports_temporal_subtraction = True
     ignores_table_name_case = True
+    supports_cast_with_precision = False
 
     @cached_property
     def uses_savepoints(self):
diff --git a/django/db/models/fields/__init__.py b/django/db/models/fields/__init__.py
index 260882bf5b..e3a8e85567 100644
--- a/django/db/models/fields/__init__.py
+++ b/django/db/models/fields/__init__.py
@@ -607,13 +607,16 @@ class Field(RegisterLookupMixin):
         self.run_validators(value)
         return value
 
+    def db_type_parameters(self, connection):
+        return DictWrapper(self.__dict__, connection.ops.quote_name, 'qn_')
+
     def db_check(self, connection):
         """
         Return the database column check constraint for this field, for the
         provided connection. Works the same way as db_type() for the case that
         get_internal_type() does not map to a preexisting model field.
         """
-        data = DictWrapper(self.__dict__, connection.ops.quote_name, "qn_")
+        data = self.db_type_parameters(connection)
         try:
             return connection.data_type_check_constraints[self.get_internal_type()] % data
         except KeyError:
@@ -639,7 +642,7 @@ class Field(RegisterLookupMixin):
         # mapped to one of the built-in Django field types. In this case, you
         # can implement db_type() instead of get_internal_type() to specify
         # exactly which wacky database column type you want to use.
-        data = DictWrapper(self.__dict__, connection.ops.quote_name, "qn_")
+        data = self.db_type_parameters(connection)
         try:
             return connection.data_types[self.get_internal_type()] % data
         except KeyError:
diff --git a/django/db/models/functions/base.py b/django/db/models/functions/base.py
index 82a6083c58..a965f4107c 100644
--- a/django/db/models/functions/base.py
+++ b/django/db/models/functions/base.py
@@ -10,7 +10,7 @@ class Cast(Func):
     template = '%(function)s(%(expressions)s AS %(db_type)s)'
 
     mysql_types = {
-        fields.CharField: 'char',
+        fields.CharField: 'char(%(max_length)s)',
         fields.IntegerField: 'signed integer',
         fields.BigIntegerField: 'signed integer',
         fields.SmallIntegerField: 'signed integer',
@@ -31,7 +31,8 @@ class Cast(Func):
         extra_context = {}
         output_field_class = type(self.output_field)
         if output_field_class in self.mysql_types:
-            extra_context['db_type'] = self.mysql_types[output_field_class]
+            data = self.output_field.db_type_parameters(connection)
+            extra_context['db_type'] = self.mysql_types[output_field_class] % data
         return self.as_sql(compiler, connection, **extra_context)
 
     def as_postgresql(self, compiler, connection):