Skip to content

Commit be5c130

Browse files
committed
update
1 parent e2ba185 commit be5c130

File tree

3 files changed

+8
-7
lines changed

3 files changed

+8
-7
lines changed

docs/CHANGELOG.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
11
Changelog
22
=========
33

4-
Unreleased
4+
1.2.4 (February 25, 2023)
55
--------------------
66
- Fix: Don't add to group a user that is not active
7+
- Fix: PHP 8.1 warning in the module settings if empty values
78

89
1.2.3 (February 20, 2023)
910
--------------------

models/ConfigureForm.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -244,11 +244,11 @@ public function save()
244244
$module = Yii::$app->getModule('auth-keycloak');
245245

246246
$module->settings->set('enabled', $this->enabled);
247-
$module->settings->set('clientId', trim($this->clientId));
248-
$module->settings->set('clientSecret', trim($this->clientSecret));
249-
$module->settings->set('realm', trim($this->realm));
250-
$module->settings->set('baseUrl', rtrim(trim($this->baseUrl), '/'));
251-
$module->settings->set('usernameMapper', trim($this->usernameMapper));
247+
$module->settings->set('clientId', trim((string)$this->clientId));
248+
$module->settings->set('clientSecret', trim((string)$this->clientSecret));
249+
$module->settings->set('realm', trim((string)$this->realm));
250+
$module->settings->set('baseUrl', rtrim(trim((string)$this->baseUrl), '/'));
251+
$module->settings->set('usernameMapper', trim((string)$this->usernameMapper));
252252
if (!$this->title) {
253253
$this->title = static::DEFAULT_TITLE;
254254
}

module.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
"name": "Keycloak Sign-In",
44
"description": "Integrating Keycloak Sign-In (OAuth 2.0)",
55
"keywords": ["auth", "keycloak"],
6-
"version": "1.2.3",
6+
"version": "1.2.4",
77
"humhub": {
88
"minVersion": "1.12"
99
},

0 commit comments

Comments
 (0)