Skip to content

Commit 906aaa2

Browse files
authored
Merge pull request #3231 from soyuka/patch/readable-names
More Descriptive Input/Output class names
2 parents d22dda9 + ddf170b commit 906aaa2

File tree

2 files changed

+11
-9
lines changed

2 files changed

+11
-9
lines changed

src/JsonSchema/SchemaFactory.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,9 @@ private function buildDefinitionName(string $resourceClass, string $format = 'js
206206

207207
$prefix = $resourceMetadata->getShortName();
208208
if (null !== $inputOrOutputClass && $resourceClass !== $inputOrOutputClass) {
209-
$prefix .= ':'.md5($inputOrOutputClass);
209+
$parts = explode('\\', $inputOrOutputClass);
210+
$shortName = end($parts);
211+
$prefix .= ':'.$shortName;
210212
}
211213

212214
if (isset($this->distinctFormats[$format])) {

tests/Swagger/Serializer/DocumentationNormalizerV2Test.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2780,7 +2780,7 @@ private function doTestNormalizeWithInputAndOutputClass(): void
27802780
'schema' => [
27812781
'type' => 'array',
27822782
'items' => [
2783-
'$ref' => '#/definitions/Dummy:300dcd476cef011532fb0ca7683395d7',
2783+
'$ref' => '#/definitions/Dummy:OutputDto',
27842784
],
27852785
],
27862786
],
@@ -2805,7 +2805,7 @@ private function doTestNormalizeWithInputAndOutputClass(): void
28052805
201 => [
28062806
'description' => 'Dummy resource created',
28072807
'schema' => [
2808-
'$ref' => '#/definitions/Dummy:300dcd476cef011532fb0ca7683395d7',
2808+
'$ref' => '#/definitions/Dummy:OutputDto',
28092809
],
28102810
],
28112811
400 => [
@@ -2821,7 +2821,7 @@ private function doTestNormalizeWithInputAndOutputClass(): void
28212821
'in' => 'body',
28222822
'description' => 'The new Dummy resource',
28232823
'schema' => [
2824-
'$ref' => '#/definitions/Dummy:b4f76c1a44965bd401aa23bb37618acc',
2824+
'$ref' => '#/definitions/Dummy:InputDto',
28252825
],
28262826
],
28272827
],
@@ -2845,7 +2845,7 @@ private function doTestNormalizeWithInputAndOutputClass(): void
28452845
200 => [
28462846
'description' => 'Dummy resource response',
28472847
'schema' => [
2848-
'$ref' => '#/definitions/Dummy:300dcd476cef011532fb0ca7683395d7',
2848+
'$ref' => '#/definitions/Dummy:OutputDto',
28492849
],
28502850
],
28512851
404 => [
@@ -2871,15 +2871,15 @@ private function doTestNormalizeWithInputAndOutputClass(): void
28712871
'in' => 'body',
28722872
'description' => 'The updated Dummy resource',
28732873
'schema' => [
2874-
'$ref' => '#/definitions/Dummy:b4f76c1a44965bd401aa23bb37618acc',
2874+
'$ref' => '#/definitions/Dummy:InputDto',
28752875
],
28762876
],
28772877
],
28782878
'responses' => [
28792879
200 => [
28802880
'description' => 'Dummy resource updated',
28812881
'schema' => [
2882-
'$ref' => '#/definitions/Dummy:300dcd476cef011532fb0ca7683395d7',
2882+
'$ref' => '#/definitions/Dummy:OutputDto',
28832883
],
28842884
],
28852885
400 => [
@@ -2893,7 +2893,7 @@ private function doTestNormalizeWithInputAndOutputClass(): void
28932893
],
28942894
]),
28952895
'definitions' => new \ArrayObject([
2896-
'Dummy:300dcd476cef011532fb0ca7683395d7' => new \ArrayObject([
2896+
'Dummy:OutputDto' => new \ArrayObject([
28972897
'type' => 'object',
28982898
'description' => 'This is a dummy.',
28992899
'externalDocs' => [
@@ -2911,7 +2911,7 @@ private function doTestNormalizeWithInputAndOutputClass(): void
29112911
]),
29122912
],
29132913
]),
2914-
'Dummy:b4f76c1a44965bd401aa23bb37618acc' => new \ArrayObject([
2914+
'Dummy:InputDto' => new \ArrayObject([
29152915
'type' => 'object',
29162916
'description' => 'This is a dummy.',
29172917
'externalDocs' => [

0 commit comments

Comments
 (0)