Skip to content

Commit 42b21af

Browse files
committed
Clean code
1 parent 1a7afe6 commit 42b21af

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

lib/Service/ProvisioningEventService.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ protected function dispatchUserAccountUpdate(string $uid, ?string $displayName,
137137
*/
138138
public function provisionUser(string $tokenUserId, int $providerId, object $idTokenPayload): ?IUser {
139139
try {
140-
// for multiple reasons, it is better to take the uid directly from a token field
140+
// for multiple reasons, it is better to take the uid directly from a token field
141141
//$uid = $this->mapDispatchUID($providerId, $idTokenPayload, $tokenUserId);
142142
$uid = $tokenUserId;
143143
$displayname = $this->mapDispatchDisplayname($providerId, $idTokenPayload);

tests/unit/MagentaCloud/ProvisioningEventServiceTest.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -349,15 +349,15 @@ public function testNoMap_AccessOk() {
349349

350350
/**
351351
* For multiple reasons, uid should com directly from a token
352-
* field, usually sub. Thus, uid is not remapped by event, even
353-
* if you try with a listener.
352+
* field, usually sub. Thus, uid is not remapped by event, even
353+
* if you try with a listener.
354354
*/
355355
public function testUidNoMapEvent_AccessOk() {
356356
$this->mockAssertLoginSuccess();
357357
$this->attributeListener = function (Event $event): void {
358358
if ($event instanceof AttributeMappedEvent &&
359359
$event->getAttribute() == ProviderService::SETTING_MAPPING_UID) {
360-
$this->fail('UID event mapping not supported');
360+
$this->fail('UID event mapping not supported');
361361
}
362362
};
363363
$this->accountListener = function (Event $event) :void {

0 commit comments

Comments
 (0)