Fixed #34558 -- Fixed QuerySet.bulk_create() crash with Now() on Oracle.

4 files changed, 32 insertions, 1 deletions

diff --git a/django/db/models/functions/datetime.py b/django/db/models/functions/datetime.py
index baed94227a..eb5332ecc0 100644
--- a/django/db/models/functions/datetime.py
+++ b/django/db/models/functions/datetime.py
@@ -236,6 +236,11 @@ class Now(Func):
             **extra_context,
         )
 
+    def as_oracle(self, compiler, connection, **extra_context):
+        return self.as_sql(
+            compiler, connection, template="LOCALTIMESTAMP", **extra_context
+        )
+
 
 class TruncBase(TimezoneMixin, Transform):
     kind = None
diff --git a/docs/ref/models/database-functions.txt b/docs/ref/models/database-functions.txt
index 13b70d76cd..f22a7ae386 100644
--- a/docs/ref/models/database-functions.txt
+++ b/docs/ref/models/database-functions.txt
@@ -562,11 +562,21 @@ Usage example:
     ``Now()`` uses ``STATEMENT_TIMESTAMP`` instead. If you need the transaction
     timestamp, use :class:`django.contrib.postgres.functions.TransactionNow`.
 
+.. admonition:: Oracle
+
+    On Oracle, the SQL ``LOCALTIMESTAMP`` is used to avoid issues with casting
+    ``CURRENT_TIMESTAMP`` to ``DateTimeField``.
+
 .. versionchanged:: 4.2
 
     Support for microsecond precision on MySQL and millisecond precision on
     SQLite were added.
 
+.. versionchanged:: 5.0
+
+    In older versions, the SQL ``CURRENT_TIMESTAMP`` was used on Oracle instead
+    of ``LOCALTIMESTAMP``.
+
 ``Trunc``
 ---------
 
diff --git a/docs/releases/5.0.txt b/docs/releases/5.0.txt
index b1e09dab46..f23f39b014 100644
--- a/docs/releases/5.0.txt
+++ b/docs/releases/5.0.txt
@@ -382,6 +382,9 @@ Miscellaneous
 * ``ImageField.update_dimension_fields()`` is no longer called on the
   ``post_init`` signal if ``width_field`` and ``height_field`` are not set.
 
+* :class:`~django.db.models.functions.Now` database function now uses
+  ``LOCALTIMESTAMP`` instead of ``CURRENT_TIMESTAMP`` on Oracle.
+
 .. _deprecated-features-5.0:
 
 Features deprecated in 5.0
diff --git a/tests/bulk_create/tests.py b/tests/bulk_create/tests.py
index a5050c9b0b..aee0cd9996 100644
--- a/tests/bulk_create/tests.py
+++ b/tests/bulk_create/tests.py
@@ -10,7 +10,7 @@ from django.db import (
     connection,
 )
 from django.db.models import FileField, Value
-from django.db.models.functions import Lower
+from django.db.models.functions import Lower, Now
 from django.test import (
     TestCase,
     override_settings,
@@ -301,6 +301,19 @@ class BulkCreateTests(TestCase):
         self.assertEqual(bbb.count(), 1)
 
     @skipUnlessDBFeature("has_bulk_insert")
+    def test_bulk_insert_now(self):
+        NullableFields.objects.bulk_create(
+            [
+                NullableFields(datetime_field=Now()),
+                NullableFields(datetime_field=Now()),
+            ]
+        )
+        self.assertEqual(
+            NullableFields.objects.filter(datetime_field__isnull=False).count(),
+            2,
+        )
+
+    @skipUnlessDBFeature("has_bulk_insert")
     def test_bulk_insert_nullable_fields(self):
         fk_to_auto_fields = {
             "auto_field": NoFields.objects.create(),