Skip to content

Commit e9c9c90

Browse files
committed
Require orisai/auth ^2.0.0
1 parent cebd613 commit e9c9c90

File tree

5 files changed

+8
-15
lines changed

5 files changed

+8
-15
lines changed

composer.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,14 +46,14 @@
4646
"nextras/dbal": "~4.0.5",
4747
"nextras/migrations": "~3.2.0",
4848
"nextras/orm": "~4.0.6",
49-
"orisai/auth": "^1.0.3",
49+
"orisai/auth": "^2.0.0",
5050
"orisai/clock": "^1.1.0",
5151
"orisai/data-sources": "^1.0.0",
5252
"orisai/exceptions": "^1.1.1",
5353
"orisai/installer": "^1.0.0",
5454
"orisai/localization": "^1.0.0",
5555
"orisai/nette-application": "^1.0.0",
56-
"orisai/nette-auth": "^1.0.0",
56+
"orisai/nette-auth": "^1.1.0",
5757
"orisai/nette-clock": "^1.0.0",
5858
"orisai/nette-console": "^1.2.0",
5959
"orisai/nette-data-sources": "^1.0.0",

src/Auth/UI/UserIdentity.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ final class UserIdentity extends StringIdentity
1010
{
1111

1212
/**
13-
* @param array<int, string> $roles
13+
* @param list<string> $roles
1414
*/
1515
public function __construct(string $id, array $roles, private UserIdentity|null $impersonator = null)
1616
{

src/Auth/UI/UserIdentityCreator.php

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,9 @@
22

33
namespace OriCMF\Auth\UI;
44

5-
use OriCMF\Role\DB\Role;
65
use OriCMF\User\DB\User;
76
use Orisai\Auth\Authorization\Authorizer;
87
use Orisai\Auth\Authorization\IdentityAuthorizationDataBuilder;
9-
use function array_map;
108

119
final class UserIdentityCreator
1210
{
@@ -17,10 +15,10 @@ public function __construct(private readonly Authorizer $authorizer)
1715

1816
public function create(User $user, UserIdentity|null $impersonator = null): UserIdentity
1917
{
20-
$roles = array_map(
21-
static fn (Role $role): string => $role->name,
22-
$user->roles->getIterator()->fetchAll(),
23-
);
18+
$roles = [];
19+
foreach ($user->roles as $role) {
20+
$roles[] = $role->name;
21+
}
2422

2523
$identity = new UserIdentity($user->id, $roles, $impersonator);
2624

src/wiring.neon

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ includes:
2323

2424
orisai.localization:
2525
directories:
26+
- %vendorDir%/orisai/auth/Locale/**
2627
- %modules.orisai_cmf.dir%/src/**/Locale/**
2728

2829
nextras.migrations:

tools/phpstan.neon

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,6 @@ parameters:
3232
path: ../src/UI/Presenter/BasePresenter.php
3333
count: 1
3434

35-
# array works, iterable does not - probably phpstan-nextras issue?
36-
-
37-
message: '#^Parameter \#1 \$callback of function array_map expects \(callable\(Nextras\\Orm\\Entity\\IEntity\)\: mixed\)\|null, Closure\(OriCMF\\Role\\DB\\Role\)\: string given\.$#'
38-
path: ../src/Auth/UI/UserIdentityCreator.php
39-
count: 1
40-
4135
# Weakly typed underlying lib
4236
-
4337
message: '#^(.+)does not accept array(.+)$#'

0 commit comments

Comments
 (0)