Skip to content

Commit ec0bf05

Browse files
authored
Use PHP attributes syntax in schema validator message (doctrine#12074)
1 parent 23b74e4 commit ec0bf05

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/Tools/SchemaValidator.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ public function validateClass(ClassMetadata $class): array
153153
$ce[] = 'The field ' . $class->name . '#' . $fieldName . ' is on the inverse side of a ' .
154154
'bi-directional relationship, but the specified mappedBy association on the target-entity ' .
155155
$assoc->targetEntity . '#' . $assoc->mappedBy . ' does not contain the required ' .
156-
"'inversedBy=\"" . $fieldName . "\"' attribute.";
156+
"'inversedBy: \"" . $fieldName . "\"' attribute.";
157157
} elseif ($targetMetadata->associationMappings[$assoc->mappedBy]->inversedBy !== $fieldName) {
158158
$ce[] = 'The mappings ' . $class->name . '#' . $fieldName . ' and ' .
159159
$assoc->targetEntity . '#' . $assoc->mappedBy . ' are ' .
@@ -174,7 +174,7 @@ public function validateClass(ClassMetadata $class): array
174174
$ce[] = 'The field ' . $class->name . '#' . $fieldName . ' is on the owning side of a ' .
175175
'bi-directional relationship, but the specified inversedBy association on the target-entity ' .
176176
$assoc->targetEntity . '#' . $assoc->inversedBy . ' does not contain the required ' .
177-
"'mappedBy=\"" . $fieldName . "\"' attribute.";
177+
"'mappedBy: \"" . $fieldName . "\"' attribute.";
178178
} elseif ($targetMetadata->associationMappings[$assoc->inversedBy]->mappedBy !== $fieldName) {
179179
$ce[] = 'The mappings ' . $class->name . '#' . $fieldName . ' and ' .
180180
$assoc->targetEntity . '#' . $assoc->inversedBy . ' are ' .

tests/Tests/ORM/Tools/SchemaValidatorTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ public function testInvalidBiDirectionalRelationMappingMissingInversedByAttribut
135135
[
136136
'The field Doctrine\Tests\ORM\Tools\DDC3274One#two is on the inverse side of a bi-directional ' .
137137
'relationship, but the specified mappedBy association on the target-entity ' .
138-
"Doctrine\Tests\ORM\Tools\DDC3274Two#one does not contain the required 'inversedBy=\"two\"' attribute.",
138+
"Doctrine\Tests\ORM\Tools\DDC3274Two#one does not contain the required 'inversedBy: \"two\"' attribute.",
139139
],
140140
$ce,
141141
);
@@ -151,7 +151,7 @@ public function testInvalidBiDirectionalRelationMappingMissingMappedByAttribute(
151151
[
152152
'The field Doctrine\Tests\ORM\Tools\Issue9536Owner#one is on the owning side of a bi-directional ' .
153153
'relationship, but the specified inversedBy association on the target-entity ' .
154-
"Doctrine\Tests\ORM\Tools\Issue9536Target#two does not contain the required 'mappedBy=\"one\"' " .
154+
"Doctrine\Tests\ORM\Tools\Issue9536Target#two does not contain the required 'mappedBy: \"one\"' " .
155155
'attribute.',
156156
],
157157
$ce,

0 commit comments

Comments
 (0)