Skip to content

Commit aef81d4

Browse files
authored
Merge pull request #34 from mezzio/renovate/vimeo-psalm-5.x
Update dependency vimeo/psalm to v5
2 parents 1640365 + e3911ac commit aef81d4

6 files changed

+157
-162
lines changed

composer.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@
5151
"laminas/laminas-coding-standard": "~2.4.0",
5252
"phpunit/phpunit": "^9.5.26",
5353
"psalm/plugin-phpunit": "^0.18.0",
54-
"vimeo/psalm": "^4.29"
54+
"vimeo/psalm": "^5.0"
5555
},
5656
"autoload": {
5757
"psr-4": {

composer.lock

+92-84
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

psalm-baseline.xml

+11-28
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
<?xml version="1.0" encoding="UTF-8"?>
2-
<files psalm-version="4.29.0@7ec5ffbd5f68ae03782d7fd33fff0c45a69f95b3">
2+
<files psalm-version="5.0.0@4e177bf0c9f03c17d2fbfd83b7cc9c47605274d8">
33
<file src="src/ProblemDetailsMiddleware.php">
44
<MissingClosureParamType occurrences="1">
55
<code>$listener</code>
@@ -34,28 +34,27 @@
3434
<code>$payload['status']</code>
3535
<code>$payload['status']</code>
3636
</MixedArgument>
37-
<MixedAssignment occurrences="3">
37+
<MixedAssignment occurrences="4">
3838
<code>$content</code>
3939
<code>$return[$key]</code>
40+
<code>$root</code>
4041
<code>$value</code>
4142
</MixedAssignment>
42-
<MixedInferredReturnType occurrences="2">
43+
<MixedInferredReturnType occurrences="1">
4344
<code>ResponseInterface</code>
44-
<code>string</code>
4545
</MixedInferredReturnType>
46-
<MixedReturnStatement occurrences="2">
47-
<code>$this-&gt;defaultTypesMap[$status] ?? sprintf('https://httpstatus.es/%s', $status)</code>
46+
<MixedMethodCall occurrences="1">
47+
<code>setAttribute</code>
48+
</MixedMethodCall>
49+
<MixedReturnStatement occurrences="1">
4850
<code>$this-&gt;getResponseGenerator($request)($payload)</code>
4951
</MixedReturnStatement>
50-
<PossiblyNullReference occurrences="1">
51-
<code>setAttribute</code>
52-
</PossiblyNullReference>
5352
<UndefinedInterfaceMethod occurrences="1">
5453
<code>getValue</code>
5554
</UndefinedInterfaceMethod>
56-
<UndefinedMethod occurrences="1">
57-
<code>setAttribute</code>
58-
</UndefinedMethod>
55+
<UndefinedPropertyFetch occurrences="1">
56+
<code>$dom-&gt;firstChild</code>
57+
</UndefinedPropertyFetch>
5958
</file>
6059
<file src="src/ProblemDetailsResponseFactoryFactory.php">
6160
<MixedArgument occurrences="4">
@@ -116,9 +115,6 @@
116115
<code>$request</code>
117116
<code>$response</code>
118117
</MissingClosureParamType>
119-
<MissingClosureReturnType occurrences="1">
120-
<code>function ($error, $request, $response) use ($exception, $expected) {</code>
121-
</MissingClosureReturnType>
122118
<MixedInferredReturnType occurrences="1">
123119
<code>array</code>
124120
</MixedInferredReturnType>
@@ -144,12 +140,6 @@
144140
</MixedOperand>
145141
</file>
146142
<file src="test/ProblemDetailsResponseFactoryTest.php">
147-
<MissingReturnType occurrences="4">
148-
<code>testCustomDetailMessageShouldBeVisible</code>
149-
<code>testExceptionCodeShouldBeIgnoredAnd500ServedInResponseBodyInNonDebugMode</code>
150-
<code>testFragileDataInExceptionMessageShouldBeHiddenInResponseBodyInNoDebugMode</code>
151-
<code>testFragileDataInExceptionMessageShouldBeVisibleInResponseBodyInNonDebugModeWhenAllowToShowByFlag</code>
152-
</MissingReturnType>
153143
<MixedArgument occurrences="2">
154144
<code>$payload['exception']</code>
155145
<code>$payload['foo']</code>
@@ -165,13 +155,6 @@
165155
<MixedAssignment occurrences="1">
166156
<code>$payload</code>
167157
</MixedAssignment>
168-
<MixedInferredReturnType occurrences="2">
169-
<code>array</code>
170-
<code>array</code>
171-
</MixedInferredReturnType>
172-
<TooManyArguments occurrences="1">
173-
<code>Assert::arrayHasKey('malformed-utf8', $payload)</code>
174-
</TooManyArguments>
175158
</file>
176159
<file src="test/TestAsset/RuntimeException.php">
177160
<MixedAssignment occurrences="1">

src/ProblemDetailsResponseFactory.php

+1
Original file line numberDiff line numberDiff line change
@@ -170,6 +170,7 @@ class ProblemDetailsResponseFactory
170170

171171
/**
172172
* @param (callable():ResponseInterface)|ResponseFactoryInterface $responseFactory
173+
* @param array<int, string> $defaultTypesMap
173174
*/
174175
public function __construct(
175176
$responseFactory,

0 commit comments

Comments
 (0)