Skip to content

Commit 2eabfb6

Browse files
committed
setAccessible() has no effect as of PHP 8.1
1 parent 3e6d1d7 commit 2eabfb6

File tree

2 files changed

+0
-6
lines changed

2 files changed

+0
-6
lines changed

src/Codeception/Module/Doctrine.php

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,6 @@ protected function clean(): void
299299
$reflectedEm = new ReflectionClass($em);
300300
if ($reflectedEm->hasProperty('repositories')) {
301301
$property = $reflectedEm->getProperty('repositories');
302-
$property->setAccessible(true);
303302
$property->setValue($em, []);
304303
}
305304
$this->em->clear();
@@ -401,13 +400,11 @@ public function haveFakeRepository(string $className, array $methods = []): void
401400
//Support doctrine versions before 2.4.0
402401

403402
$property = $reflectedEm->getProperty('repositories');
404-
$property->setAccessible(true);
405403
$property->setValue($em, array_merge($property->getValue($em), [$className => $mock]));
406404
} elseif ($reflectedEm->hasProperty('repositoryFactory')) {
407405
//For doctrine 2.4.0+ versions
408406

409407
$repositoryFactoryProperty = $reflectedEm->getProperty('repositoryFactory');
410-
$repositoryFactoryProperty->setAccessible(true);
411408
$repositoryFactory = $repositoryFactoryProperty->getValue($em);
412409

413410
$reflectedRepositoryFactory = new ReflectionClass($repositoryFactory);
@@ -419,7 +416,6 @@ public function haveFakeRepository(string $className, array $methods = []): void
419416
);
420417
} elseif ($reflectedRepositoryFactory->hasProperty('repositoryList')) {
421418
$repositoryListProperty = $reflectedRepositoryFactory->getProperty('repositoryList');
422-
$repositoryListProperty->setAccessible(true);
423419

424420
$repositoryHash = $em->getClassMetadata($className)->getName() . spl_object_id($em);
425421
$repositoryListProperty->setValue(

src/Codeception/Util/ReflectionPropertyAccessor.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ public function getProperty(object $obj, string $field): mixed
2525
$reflectedEntity = new ReflectionClass($class);
2626
if ($reflectedEntity->hasProperty($field)) {
2727
$property = $reflectedEntity->getProperty($field);
28-
$property->setAccessible(true);
2928
return $property->getValue($obj);
3029
}
3130
$class = get_parent_class($class);
@@ -62,7 +61,6 @@ private function setPropertiesForClass(?object $obj, string $class, array $data)
6261

6362
foreach ($reflectedEntity->getProperties() as $property) {
6463
if (isset($data[$property->name])) {
65-
$property->setAccessible(true);
6664
$property->setValue($obj, $data[$property->name]);
6765
}
6866
}

0 commit comments

Comments
 (0)