Skip to content

Commit 513d9a7

Browse files
committed
6871: Fixed psalm raised issues
1 parent 51ac0d0 commit 513d9a7

3 files changed

Lines changed: 4 additions & 3 deletions

File tree

src/Command/Checksum/RecalculateChecksumCommand.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
name: 'app:checksum:recalculate',
2525
description: 'Recalculate relation checksums for slides and media entities'
2626
)]
27-
class RecalculateChecksumCommand extends Command
27+
final class RecalculateChecksumCommand extends Command
2828
{
2929
private const string OPTION_TENANT = 'tenant';
3030
private const string OPTION_MODIFIED_AFTER = 'modified-after';

src/Security/AzureOidcAuthenticator.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,7 @@ public function authenticate(Request $request): Passport
9696
}
9797

9898
// Check if user exists already - if not create a user
99+
/* @var User|null $user */
99100
$user = $this->entityManager->getRepository(User::class)->findOneBy(['providerId' => $providerId]);
100101

101102
if (null === $user) {
@@ -114,7 +115,7 @@ public function authenticate(Request $request): Passport
114115

115116
if (self::OIDC_PROVIDER_INTERNAL === $provider) {
116117
// Set tenants from claims.
117-
$oidcGroups = $claims[$this->oidcInternalClaimGroups] ?? [];
118+
$oidcGroups = (array) ($claims[$this->oidcInternalClaimGroups] ?? []);
118119
$this->setTenantRoles($user, $oidcGroups);
119120
}
120121

src/Service/RelationsChecksumCalculator.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
* Extracted from RelationsChecksumListener to allow reuse in console commands
1717
* and other contexts outside Doctrine lifecycle events.
1818
*/
19-
class RelationsChecksumCalculator
19+
final class RelationsChecksumCalculator
2020
{
2121
public const array CHECKSUM_TABLES = ['feed_source', 'feed', 'slide', 'media', 'theme', 'template', 'playlist_slide',
2222
'playlist', 'screen_campaign', 'screen', 'screen_group_campaign', 'screen_group',

0 commit comments

Comments
 (0)