Skip to content

Commit 376a23f

Browse files
author
MateuszKolankowski
committed
Merge branch '4.6'
# Conflicts: # phpstan-baseline-7.4.neon # phpstan-baseline-gte-8.0.neon # phpstan-baseline.neon # src/bundle/EventListener/CsrfListener.php # src/contracts/Security/AuthorizationHeaderRESTRequestMatcher.php # src/lib/Input/Handler/Xml.php # src/lib/Server/Controller/SessionController.php # src/lib/Server/Security/CsrfTokenManager.php # src/lib/Server/Security/RestAuthenticator.php # src/lib/Server/Values/RestContent.php # src/lib/Server/Values/RestContentType.php # src/lib/Server/Values/RoleAssignment.php
2 parents fe5e635 + c2e1a2e commit 376a23f

File tree

8 files changed

+10
-8
lines changed

8 files changed

+10
-8
lines changed

phpstan-baseline-7.4.neon

Whitespace-only changes.

phpstan-baseline-gte-8.0.neon

Whitespace-only changes.

src/contracts/Security/AuthorizationHeaderRESTRequestMatcher.php

Whitespace-only changes.

src/lib/Input/Handler/Xml.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
use DOMNodeList;
1313
use Ibexa\Contracts\Rest\Exceptions;
1414
use Ibexa\Contracts\Rest\Input\Handler;
15+
use InvalidArgumentException;
1516

1617
/**
1718
* Input format handler base class.
@@ -155,6 +156,9 @@ protected function convertDom(DOMNode $node): array|string|int|bool|float|null
155156
];
156157
$isArray = true;
157158
} elseif (is_array($current[$tagName])) {
159+
if (!is_array($current[$tagName])) {
160+
throw new InvalidArgumentException('Current tag name is not an array as expected.');
161+
}
158162
$current[$tagName][] = $this->convertDom($childNode);
159163
}
160164

src/lib/Server/Controller/SessionController.php

Whitespace-only changes.

src/lib/Server/Security/CsrfTokenManager.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,9 @@ final class CsrfTokenManager extends BaseCsrfTokenManager
2121
private string $namespace;
2222

2323
public function __construct(
24-
TokenGeneratorInterface $generator = null,
25-
TokenStorageInterface $storage = null,
26-
RequestStack $requestStack = null
24+
?TokenGeneratorInterface $generator = null,
25+
?TokenStorageInterface $storage = null,
26+
?RequestStack $requestStack = null
2727
) {
2828
$this->storage = $storage ?: new NativeSessionTokenStorage();
2929
$this->namespace = $this->resolveNamespace($requestStack);
@@ -36,7 +36,7 @@ public function hasToken(string $tokenId): bool
3636
return $this->storage->hasToken($this->namespace . $tokenId);
3737
}
3838

39-
private function resolveNamespace(RequestStack $requestStack = null): string
39+
private function resolveNamespace(?RequestStack $requestStack = null): string
4040
{
4141
if ($requestStack !== null && ($request = $requestStack->getMainRequest())) {
4242
return $request->isSecure() ? 'https-' : '';

src/lib/Server/Security/RestAuthenticator.php

Whitespace-only changes.

src/lib/Server/Values/RestContentType.php

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,13 +19,11 @@ class RestContentType extends RestValue
1919
{
2020
public ContentType $contentType;
2121

22-
/**
23-
* @var \Ibexa\Contracts\Core\Repository\Values\ContentType\FieldDefinition[]
24-
*/
22+
/** @var \Ibexa\Contracts\Core\Repository\Values\ContentType\FieldDefinition[]|null */
2523
public ?array $fieldDefinitions;
2624

2725
/**
28-
* @param \Ibexa\Contracts\Core\Repository\Values\ContentType\FieldDefinition[] $fieldDefinitions
26+
* @param \Ibexa\Contracts\Core\Repository\Values\ContentType\FieldDefinition[]|null $fieldDefinitions
2927
*/
3028
public function __construct(ContentType $contentType, ?array $fieldDefinitions = null)
3129
{

0 commit comments

Comments
 (0)