Use correct type

Author: franmomu

CHANGELOG.md

@@ -24,7 +24,9 @@ a release.
 - [NestedSet] childrenQueryBuilder() to allow specifying sort order separately for each field
 - [NestedSet] Added option to reorder only direct children in reorder() method
 
-### Sortable
+## Changed
+- Tree: In `ClosureTreeRepository::removeFromTree()` and `NestedTreeRepository::removeFromTree()` when something fails in the transaction, it uses the `code` from the original exception to construct the `\Gedmo\Exception\RuntimeException` instance instead of `null`.
+
 #### Fixed
 - [SortableGroup] Fix sorting date columns in SQLite (#2462).
 

src/Tree/Entity/Repository/ClosureTreeRepository.php

@@ -292,7 +292,7 @@ public function removeFromTree($node)
             $this->_em->close();
             $this->_em->getConnection()->rollback();
 
-            throw new \Gedmo\Exception\RuntimeException('Transaction failed: '.$e->getMessage(), null, $e);
+            throw new \Gedmo\Exception\RuntimeException('Transaction failed: '.$e->getMessage(), $e->getCode(), $e);
         }
         // remove from identity map
         $this->_em->getUnitOfWork()->removeFromIdentityMap($node);

src/Tree/Entity/Repository/NestedTreeRepository.php

@@ -749,7 +749,7 @@ public function removeFromTree($node)
                 $this->_em->close();
                 $this->_em->getConnection()->rollback();
 
-                throw new \Gedmo\Exception\RuntimeException('Transaction failed', null, $e);
+                throw new \Gedmo\Exception\RuntimeException('Transaction failed', $e->getCode(), $e);
             }
         } else {
             throw new InvalidArgumentException('Node is not related to this repository');

tests/Gedmo/IpTraceable/IpTraceableTest.php

@@ -12,9 +12,11 @@
 namespace Gedmo\Tests\IpTraceable;
 
 use Doctrine\Common\EventManager;
+use Doctrine\Persistence\Mapping\ClassMetadata;
 use Gedmo\Exception\InvalidArgumentException;
 use Gedmo\IpTraceable\IpTraceable;
 use Gedmo\IpTraceable\IpTraceableListener;
+use Gedmo\Mapping\Event\AdapterInterface;
 use Gedmo\Tests\IpTraceable\Fixture\Article;
 use Gedmo\Tests\IpTraceable\Fixture\Comment;
 use Gedmo\Tests\IpTraceable\Fixture\Type;
@@ -59,14 +61,22 @@ public function testIpV4(): void
     {
         $listener = new IpTraceableListener();
         $listener->setIpValue('123.218.45.39');
-        static::assertSame('123.218.45.39', $listener->getFieldValue(null, null, null));
+        static::assertSame('123.218.45.39', $listener->getFieldValue(
+            $this->createStub(ClassMetadata::class),
+            'ip',
+            $this->createStub(AdapterInterface::class)
+        ));
     }
 
     public function testIpV6(): void
     {
         $listener = new IpTraceableListener();
         $listener->setIpValue('2001:0db8:0000:85a3:0000:0000:ac1f:8001');
-        static::assertSame('2001:0db8:0000:85a3:0000:0000:ac1f:8001', $listener->getFieldValue(null, null, null));
+        static::assertSame('2001:0db8:0000:85a3:0000:0000:ac1f:8001', $listener->getFieldValue(
+            $this->createStub(ClassMetadata::class),
+            'ip',
+            $this->createStub(AdapterInterface::class)
+        ));
     }
 
     public function testIpTraceable(): void