Skip to content

Commit

Permalink
Temporarily disable unsupported Enrollment Authz (CFM-31)
Browse files Browse the repository at this point in the history
  • Loading branch information
Benn Oshrin committed Apr 14, 2025
1 parent 60eedb6 commit 4f1f7f3
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 10 deletions.
14 changes: 9 additions & 5 deletions app/src/Controller/EnrollmentFlowsController.php
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,9 @@ public function calculatePermission(): bool {
case EnrollmentAuthzEnum::CoAdmin:
$authorized = $this->RegistryAuth->isCoAdmin($flow->co_id);
break;
// CFM-31 implement these (and enable in EnrollmentAuthzEnum, and possibly start() below
// and fields.inc)
/*
case EnrollmentAuthzEnum::CoOrCouAdmin:
// XXX
break;
Expand All @@ -93,7 +96,7 @@ public function calculatePermission(): bool {
case EnrollmentAuthzEnum::None:
// XXX willHandleAuth needs to check for this mode and then return 'open' if set
$authorized = true;
break;
break;*/
}
}

Expand Down Expand Up @@ -142,10 +145,11 @@ public function start(string $id) {

$isEnrollee = in_array($flow->authz_type, [
EnrollmentAuthzEnum::AuthUser,
EnrollmentAuthzEnum::CouPerson,
EnrollmentAuthzEnum::GroupMember,
EnrollmentAuthzEnum::Person,
EnrollmentAuthzEnum::None
// XXX not yet implemented
// EnrollmentAuthzEnum::CouPerson,
// EnrollmentAuthzEnum::GroupMember,
// EnrollmentAuthzEnum::Person,
// EnrollmentAuthzEnum::None
]);

$actor = $this->getCurrentActor();
Expand Down
3 changes: 2 additions & 1 deletion app/src/Lib/Enum/EnrollmentAuthzEnum.php
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,11 @@
class EnrollmentAuthzEnum extends StandardEnum {
const AuthUser = 'AU';
const CoAdmin = 'CA';
/* CFM-31 not yet implemented
const CoOrCouAdmin = 'A';
const CouAdmin = 'UA';
const CouPerson = 'UP';
const GroupMember = 'GM';
const Person = 'CP';
const None = 'N';
const None = 'N';*/
}
11 changes: 7 additions & 4 deletions app/templates/EnrollmentFlows/fields.inc
Original file line number Diff line number Diff line change
Expand Up @@ -36,16 +36,19 @@ declare(strict_types = 1);
var authz = document.getElementById('authz-type').value;

// Handle page interaction
if(authz == '<?= \App\Lib\Enum\EnrollmentAuthzEnum::CouAdmin ?>'
|| authz == '<?= \App\Lib\Enum\EnrollmentAuthzEnum::CouPerson ?>') {
/* CFM-31 not yet implemented
if(authz == '<?php // \App\Lib\Enum\EnrollmentAuthzEnum::CouAdmin ?>'
|| authz == '<?php // \App\Lib\Enum\EnrollmentAuthzEnum::CouPerson ?>') {
hideFields(['authz-group-id'], isPageLoad);
showFields(['authz-cou-id'], isPageLoad);
} else if(authz == '<?= \App\Lib\Enum\EnrollmentAuthzEnum::GroupMember ?>') {
} else if(authz == '<?php // \App\Lib\Enum\EnrollmentAuthzEnum::GroupMember ?>') {
hideFields(['authz-cou-id'], isPageLoad);
showFields(['authz-group-id'], isPageLoad);
} else {
hideFields(['authz-cou-id', 'authz-group-id'], isPageLoad);
}
}*/

hideFields(['authz-cou-id', 'authz-group-id'], isPageLoad);
}

function jsLocalOnLoad() {
Expand Down

0 comments on commit 4f1f7f3

Please sign in to comment.