WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Commit be3e2fe

Browse files
committed
fixing a problem with add/remove administrators
1 parent 94949b4 commit be3e2fe

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

Changes.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
Changes in 2.2.1.1-alpha
2+
- [BUGFIX #1] - fixed the problem with Add/Remove adminstrotors for institution admins
3+
- [BUGFIX #2] - introduce better type-safety for admin API (reported by: Nahit (Github: https://github.com/Dogru-Isim))
4+
15
Changes in 2.2.1
26
=====================
37
- [FEATURE #1] - added logging of last administrator authentication to help eliminating "dead" accounts

web/admin/inc/manageAdmins.inc.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,9 +52,10 @@
5252
// either the operation is done by federation operator himself
5353
$isFedAdmin = $user->isFederationAdmin($my_inst->federation);
5454
// or an admin of the IdP with federation admin blessings
55+
$is_admin_with_blessing = $my_inst->isPrimaryOwner($_SESSION['user']);
5556

5657
if (isset($_SESSION['entitledIdPs']) && in_array($my_inst->identifier, $_SESSION['entitledIdPs'])) {
57-
$is_admin_with_blessing = true;
58+
$is_admin_with_blessing = true;
5859
}
5960

6061
if (isset($_SESSION['resyncedIdPs']) && in_array($my_inst->identifier, $_SESSION['resyncedIdPs'])) {

0 commit comments

Comments
 (0)