[Doctrine][Messenger] Use common sequence name to get id from Oracle

symfony · fabpot · Oct 6, 2024 · Apr 11, 2024 · Oct 6, 2024 · 52c3a32ca96a627f681c00ee0cc8ff6ade8a1b0d
commit 52c3a32ca96a627f681c00ee0cc8ff6ade8a1b0d
diff --git a/src/Symfony/Component/Messenger/Bridge/Doctrine/Transport/Connection.php b/src/Symfony/Component/Messenger/Bridge/Doctrine/Transport/Connection.php
@@ -470,6 +470,18 @@ private function executeInsert(string $sql, array $parameters = [], array $types
                 if (!$id) {
                     throw new TransportException('no id was returned by PostgreSQL from RETURNING clause.');
                 }
+            } elseif ($this->driverConnection->getDatabasePlatform() instanceof OraclePlatform) {
+                $sequenceName = 'seq_'.$this->configuration['table_name'];
+
+                $this->driverConnection->executeStatement($sql, $parameters, $types);
+
+                $result = $this->driverConnection->fetchOne('SELECT '.$sequenceName.'.CURRVAL FROM DUAL');
+
+                $id = (int) $result;
+
+                if (!$id) {
+                    throw new TransportException('no id was returned by Oracle from sequence: '.$sequenceName);
+                }
             } else {
                 $this->driverConnection->executeStatement($sql, $parameters, $types);
 
@@ -507,7 +519,7 @@ private function addTableToSchema(Schema $schema): void
         $table = $schema->createTable($this->configuration['table_name']);
         // add an internal option to mark that we created this & the non-namespaced table name
         $table->addOption(self::TABLE_OPTION_NAME, $this->configuration['table_name']);
-        $table->addColumn('id', Types::BIGINT)
+        $idColumn = $table->addColumn('id', Types::BIGINT)
             ->setAutoincrement(true)
             ->setNotnull(true);
         $table->addColumn('body', Types::TEXT)
@@ -527,6 +539,13 @@ private function addTableToSchema(Schema $schema): void
         $table->addIndex(['queue_name']);
         $table->addIndex(['available_at']);
         $table->addIndex(['delivered_at']);
+
+        // We need to create a sequence for Oracle and set the id column to get the correct nextval
+        if ($this->driverConnection->getDatabasePlatform() instanceof OraclePlatform) {
+            $idColumn->setDefault('seq_'.$this->configuration['table_name'].'.nextval');
+
+            $schema->createSequence('seq_'.$this->configuration['table_name']);
+        }
     }
 
     private function decodeEnvelopeHeaders(array $doctrineEnvelope): array