Merge pull request #8 from humhub/fix/6-redirection-to-registration-f…

…ails Fix #6: Redirection to registration form fails (e.g. when a profile f…
humhub · Oct 11, 2024 · 7449fc9 · 7449fc9
2 parents 02c8688 + 6ffa398
commit 7449fc9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 10 deletions.
diff --git a/authclient/LinkedinAuthV2.php b/authclient/LinkedinAuthV2.php
@@ -3,7 +3,6 @@
 namespace humhubContrib\auth\linkedin\authclient;
 
 use yii\authclient\clients\LinkedIn;
-use humhub\helpers\ArrayHelper;
 
 /**
  * LinkedIn allows authentication via LinkedIn OAuth.
@@ -45,15 +44,9 @@ protected function defaultNormalizeUserAttributeMap()
         return [
             'id' => 'sub',
             'username' => 'displayName',
-            'firstname' => function ($attributes) {
-                return ArrayHelper::getValue($attributes, 'given_name', '');
-            },
-            'lastname' => function ($attributes) {
-                return ArrayHelper::getValue($attributes, 'family_name', '');
-            },
-            'email' => function ($attributes) {
-                return ArrayHelper::getValue($attributes, 'email', '');
-            },
+            'firstname' => 'given_name',
+            'lastname' => 'family_name',
+            'email' => 'email',
         ];
     }
 }
diff --git a/docs/CHANGELOG.md b/docs/CHANGELOG.md
@@ -4,6 +4,7 @@ Changelog
 1.1.1 (Unreleased)
 --------------------
 - Enh #7: Use PHP CS Fixer
+- Fix #6: Redirection to registration form fails (e.g. when a profile field is required) with version 2
 
 1.1.0 (June 7, 2024)
 --------------------