Merge pull request #175 from dmitryuk/topic/174

DamienHarper · Sep 19, 2023 · a40416e · a40416e
2 parents a3e5492 + 6d65543
commit a40416e
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 1 deletion.
diff --git a/src/Provider/Doctrine/Persistence/Schema/SchemaManager.php b/src/Provider/Doctrine/Persistence/Schema/SchemaManager.php
@@ -270,7 +270,7 @@ public function resolveAuditTableName(string $entity, Configuration $configurati
     public function computeAuditTablename(string $entityTableName, Configuration $configuration): ?string
     {
         return preg_replace(
-            sprintf('#^([^\.]+\.)?(%s)$#', preg_quote($entityTableName, '#')),
+            '#^([^.]+\.)?(.+)$#',
             sprintf(
                 '$1%s$2%s',
                 preg_quote($configuration->getTablePrefix(), '#'),

diff --git a/tests/Provider/Doctrine/Issues/Issue174Test.php b/tests/Provider/Doctrine/Issues/Issue174Test.php
@@ -0,0 +1,30 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DH\Auditor\Tests\Provider\Doctrine\Issues;
+
+use DH\Auditor\Provider\Doctrine\Configuration;
+use DH\Auditor\Provider\Doctrine\Persistence\Schema\SchemaManager;
+use DH\Auditor\Tests\Provider\Doctrine\Traits\Schema\DefaultSchemaSetupTrait;
+use PHPUnit\Framework\TestCase;
+
+/**
+ * @internal
+ */
+#[\PHPUnit\Framework\Attributes\Small]
+/**
+ * @internal
+ */
+final class Issue174Test extends TestCase
+{
+    use DefaultSchemaSetupTrait;
+
+    public function testIssue174(): void
+    {
+        $configuration = new Configuration(['table_prefix' => 'audit_', 'table_suffix' => '_suffix']);
+        $manager = new SchemaManager($this->provider);
+        $result = $manager->computeAuditTablename('schema.entity', $configuration);
+        self::assertSame('schema.audit_entity_suffix', $result);
+    }
+}