Fix incorrect dissociation array destructuring (#92)

* Fix incorrect dissociation array destructuring Fixes #91 * Add missing Entity ID to a tracked dissociation event Co-authored-by: Damien Harper <[email protected]>
DamienHarper · Mar 8, 2022 · 657164b · 657164b
1 parent 94d4d4b
commit 657164b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/Provider/Doctrine/Auditing/Transaction/TransactionHydrator.php b/src/Provider/Doctrine/Auditing/Transaction/TransactionHydrator.php
@@ -102,6 +102,7 @@ private function hydrateWithScheduledCollectionUpdates(Transaction $transaction,
                         $transaction->trackAuditEvent(Transaction::DISSOCIATE, [
                             $collection->getOwner(),
                             $entity,
+                            $this->id($entityManager, $entity),
                             $mapping,
                         ]);
                     }

diff --git a/src/Provider/Doctrine/Auditing/Transaction/TransactionProcessor.php b/src/Provider/Doctrine/Auditing/Transaction/TransactionProcessor.php
@@ -159,7 +159,7 @@ private function processAssociations(Transaction $transaction, EntityManagerInte
 
     private function processDissociations(Transaction $transaction, EntityManagerInterface $entityManager): void
     {
-        foreach ($transaction->getDissociated() as [$source, $target, $mapping]) {
+        foreach ($transaction->getDissociated() as [$source, $target, $id, $mapping]) {
             $this->dissociate($entityManager, $source, $target, $mapping, $transaction->getTransactionHash());
         }
     }