Skip to content
This repository was archived by the owner on Sep 19, 2022. It is now read-only.

Commit e90ddb9

Browse files
author
Pavel Vyskočil
committed
Fixed the checks in method getMemberStatusByUserAndVo() in AdapterLDAP
* Added check if $group_id is null
1 parent 6cf3098 commit e90ddb9

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ All notable changes to this project will be documented in this file.
66
- Added missing space before 'addInstitutionButton' or link
77
- Added missing import
88
- Fixed the style of changelog
9+
- Fixed the checks in method getMemberStatusByUserAndVo() in AdapterLDAP
910

1011
## [v3.0.3]
1112
#### Fixed

lib/AdapterLdap.php

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -325,10 +325,9 @@ public function getMemberStatusByUserAndVo($user, $vo)
325325
array("perunGroupid")
326326
);
327327

328-
if (empty($groupId)) {
328+
if ($groupId === null || empty($groupId)) {
329329
return Member::INVALID;
330-
} else {
331-
return Member::VALID;
332330
}
331+
return Member::VALID;
333332
}
334333
}

0 commit comments

Comments
 (0)