Skip to content

Commit 5634c3c

Browse files
committed
Drop useless parameter
It was here to improve DX on 3.x
1 parent a93d7d2 commit 5634c3c

File tree

3 files changed

+2
-29
lines changed

3 files changed

+2
-29
lines changed

phpstan.neon

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,3 @@ parameters:
1111
-
1212
message: '~^Match expression does not handle remaining values:~'
1313
path: src/Persisters/Entity/BasicEntityPersister.php
14-
15-
# To be removed in 4.0
16-
17-
-
18-
message: '#Negated boolean expression is always false\.#'
19-
paths:
20-
- src/Mapping/Driver/AttributeDriver.php

src/Mapping/Driver/AttributeDriver.php

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -12,15 +12,13 @@
1212
use Doctrine\Persistence\Mapping\ClassMetadata as PersistenceClassMetadata;
1313
use Doctrine\Persistence\Mapping\Driver\ColocatedMappingDriver;
1414
use Doctrine\Persistence\Mapping\Driver\MappingDriver;
15-
use InvalidArgumentException;
1615
use ReflectionClass;
1716
use ReflectionMethod;
1817

1918
use function assert;
2019
use function class_exists;
2120
use function constant;
2221
use function defined;
23-
use function sprintf;
2422

2523
class AttributeDriver implements MappingDriver
2624
{
@@ -34,19 +32,9 @@ class AttributeDriver implements MappingDriver
3432

3533
private readonly AttributeReader $reader;
3634

37-
/**
38-
* @param array<string> $paths
39-
* @param true $reportFieldsWhereDeclared no-op, to be removed in 4.0
40-
*/
41-
public function __construct(array $paths, bool $reportFieldsWhereDeclared = true)
35+
/** @param array<string> $paths */
36+
public function __construct(array $paths)
4237
{
43-
if (! $reportFieldsWhereDeclared) {
44-
throw new InvalidArgumentException(sprintf(
45-
'The $reportFieldsWhereDeclared argument is no longer supported, make sure to omit it when calling %s.',
46-
__METHOD__,
47-
));
48-
}
49-
5038
$this->reader = new AttributeReader();
5139
$this->addPaths($paths);
5240
}

tests/Tests/ORM/Mapping/AttributeDriverTest.php

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
use Doctrine\ORM\Mapping\MappingAttribute;
1313
use Doctrine\Persistence\Mapping\Driver\MappingDriver;
1414
use Doctrine\Tests\ORM\Mapping\Fixtures\AttributeEntityWithNestedJoinColumns;
15-
use InvalidArgumentException;
1615
use stdClass;
1716

1817
class AttributeDriverTest extends MappingDriverTestCase
@@ -88,13 +87,6 @@ public function testManyToManyAssociationWithNestedJoinColumns(): void
8887
$metadata->associationMappings['assoc']->joinTable->inverseJoinColumns,
8988
);
9089
}
91-
92-
public function testItThrowsWhenSettingReportFieldsWhereDeclaredToFalse(): void
93-
{
94-
$this->expectException(InvalidArgumentException::class);
95-
96-
new AttributeDriver([], false);
97-
}
9890
}
9991

10092
#[ORM\Entity]

0 commit comments

Comments
 (0)