cs fix

Author: dbu

lib/Doctrine/ODM/PHPCR/Document/AbstractFile.php

@@ -10,7 +10,7 @@
  * This class represents an abstract "file".
  */
 #[PHPCR\MappedSuperclass(mixins: ['mix:created'])]
-abstract class AbstractFile implements HierarchyInterface
+abstract class AbstractFile implements \Stringable, HierarchyInterface
 {
     #[PHPCR\Id(strategy: 'parent')]
     protected string $id;

lib/Doctrine/ODM/PHPCR/Document/Generic.php

@@ -15,7 +15,7 @@
  * You can not use this to create nodes as it has no type mapping.
  */
 #[PHPCR\Document]
-class Generic
+class Generic implements \Stringable
 {
     /**
      * Id (path) of this document.

lib/Doctrine/ODM/PHPCR/Document/Resource.php

@@ -12,7 +12,7 @@
  * @see http://wiki.apache.org/jackrabbit/nt:resource
  */
 #[PHPCR\Document(nodeType: 'nt:resource')]
-class Resource
+class Resource implements \Stringable
 {
     #[PHPCR\Id]
     protected string $id;

lib/Doctrine/ODM/PHPCR/PersistentCollection.php

@@ -8,7 +8,7 @@
 /**
  * @author Benjamin Eberlei <kontakt@beberlei.de>
  */
-abstract class PersistentCollection implements Collection
+abstract class PersistentCollection implements \Stringable, Collection
 {
     protected const INITIALIZED_NONE = 'not initialized';
 

lib/Doctrine/ODM/PHPCR/Query/Builder/QueryBuilder.php

@@ -28,7 +28,7 @@
  *
  * @author Daniel Leech <daniel@dantleech.com>
  */
-class QueryBuilder extends QBConstants
+class QueryBuilder extends QBConstants implements \Stringable
 {
     private ?ConverterInterface $converter = null;
     private int $firstResult = 0;

tests/Doctrine/Tests/ODM/PHPCR/DocumentClassMapperTest.php

@@ -209,7 +209,6 @@ public function testExpandClassName(string $className, string $fqClassName, bool
 
         $refl = new \ReflectionClass($this->mapper);
         $method = $refl->getMethod('expandClassName');
-        $method->setAccessible(true);
         $res = $method->invoke($this->mapper, $this->dm, $className);
 
         $this->assertEquals($fqClassName, $res);

tests/Doctrine/Tests/ODM/PHPCR/DocumentManagerTest.php

@@ -82,7 +82,6 @@ public function testGetMetadataFactory(): void
         $factory = $dm->getMetadataFactory();
         $reflection = new \ReflectionClass($factory);
         $property = $reflection->getProperty('dm');
-        $property->setAccessible(true);
         $this->assertSame($dm, $property->getValue($factory));
     }
 
@@ -114,9 +113,7 @@ public function testContains(): void
         $uow = $dm->getUnitOfWork();
 
         $method = new \ReflectionMethod($uow, 'registerDocument');
-        $method->setAccessible(true);
         $method->invoke($uow, $obj, '/foo');
-        $method->setAccessible(false);
 
         $this->assertTrue($dm->contains($obj));
     }
@@ -185,9 +182,7 @@ public function testGetDocumentIdReturnsValueOfUnitOfWork(): void
         $uow = $dm->getUnitOfWork();
 
         $reflectionProperty = new \ReflectionProperty($uow, 'documentIds');
-        $reflectionProperty->setAccessible(true);
         $reflectionProperty->setValue($uow, [spl_object_hash($obj) => '/foo']);
-        $reflectionProperty->setAccessible(false);
 
         $this->assertSame('/foo', $dm->getDocumentId($obj));
     }

tests/Doctrine/Tests/ODM/PHPCR/Query/Builder/AbstractLeafNodeTest.php

@@ -46,7 +46,6 @@ public function testExplodeField($fieldSpec, $xpctdExceptionMessage, $xpctdRes =
         }
 
         $method = $this->refl->getMethod('explodeField');
-        $method->setAccessible(true);
         $res = $method->invoke($this->leafNode, $fieldSpec);
 
         if (false === $xpctdExceptionMessage) {

tests/Doctrine/Tests/ODM/PHPCR/Query/Builder/QueryBuilderTest.php

@@ -107,7 +107,6 @@ public function testCloningQueryBuilderUniqueConverterInstance(): void
     {
         $reflection = new \ReflectionClass(get_class($this->node));
         $property = $reflection->getProperty('converter');
-        $property->setAccessible(true);
 
         $this->node->setConverter($this->createMock(ConverterInterface::class));
 

tests/Doctrine/Tests/ODM/PHPCR/Translation/AttributeTranslationStrategyTest.php

@@ -30,7 +30,6 @@ public function setUp(): void
 
         $class = new \ReflectionClass(AttributeTranslationStrategy::class);
         $this->method = $class->getMethod('getTranslatedPropertyName');
-        $this->method->setAccessible(true);
     }
 
     public function testSetPrefixAndGetPropertyName(): void