Skip to content

Commit

Permalink
Fix duplicate global roles for FILTER_ALL
Browse files Browse the repository at this point in the history
and eliminate duplicate declare strict_types
  • Loading branch information
ingoj authored and kergomard committed Jan 15, 2025
1 parent 0cb2dd4 commit 811cb48
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions Services/AccessControl/classes/class.ilRbacReview.php
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,6 @@

declare(strict_types=1);

declare(strict_types=1);

/**
* class ilRbacReview
* Contains Review functions of core Rbac.
Expand Down Expand Up @@ -857,7 +855,7 @@ public function getRolesByFilter(int $a_filter = 0, int $a_user_id = 0, string $
// all (assignable) roles
case self::FILTER_ALL:
yield from $this->getAssignableRolesGenerator(true, true, $title_filter);

return;
// all (assignable) global roles
// no break
case self::FILTER_ALL_GLOBAL:
Expand Down

0 comments on commit 811cb48

Please sign in to comment.