Update PéK admin requirements (#168)

Author: albi005

StartSch/Components/Pages/AdminDashboardPage.razor

@@ -29,12 +29,17 @@
     }
 
     <p style="margin-top: 16px">
-        Azokat a köröket adminisztálhatod, amelyiknél
-        <a href="https://pek.sch.bme.hu">PéK</a>-en
-        <code>körvezető</code>,
-        <code>admin</code> vagy
-        <code>PR</code>
-        címed van. Jogosultságok frissítéséhez jelentkezz ki, majd be.
+        Azokat a köröket adminisztrálhatod, amelyiknél
+        <a href="https://pek.sch.bme.hu">PéK-en </a>
+        @foreach (var title in Constants.TrustedPekTitles.SkipLast(2))
+        {
+            <b><code>@title</code></b>@:,
+        }
+        <b><code>@(Constants.TrustedPekTitles[^2])</code></b> vagy
+        <b><code>@(Constants.TrustedPekTitles[^1])</code></b>
+        címed van. Ha ez nem megfelelő,
+        <a href="https://github.com/kir-dev/StartSCH/issues/new">nyiss egy issue-t GitHub-on</a>.
+        Jogosultságok frissítéséhez jelentkezz ki, majd be.
     </p>
 </main>
 

StartSch/Constants.cs

@@ -1,3 +1,5 @@
+using System.Collections.ObjectModel;
+
 namespace StartSch;
 
 public static class Constants
@@ -6,11 +8,17 @@ public static class Constants
     public const string CookieAuthenticationScheme = nameof(CookieAuthenticationScheme);
     public const string StartSchUserIdClaim = "id";
     public const string StartSchPageAdminClaim = "startsch/page-admin";
+    
+    public static readonly ReadOnlyCollection<string> TrustedPekTitles =
+    [
+        "Adminisztrátor",
+        "körvezető",
+        "körvezető helyettes",
+        "PR menedzser",
+    ];
 
-    public static bool IsPrivilegedPekTitle(string pekTitle)
+    public static bool IsTrustedPekTitle(string pekTitle)
     {
-        return pekTitle.RoughlyMatches("korvez")
-               || pekTitle.RoughlyMatches("admin")
-               || pekTitle.Contains("PR");
+        return TrustedPekTitles.Contains(pekTitle, StringComparer.InvariantCultureIgnoreCase);
     }
 }

StartSch/PekMemberships.cs

@@ -20,7 +20,7 @@ public IEnumerable<GroupMembership> GetAdminMemberships()
         {
             return claimsPrincipal
                 .GetGroupMemberships()
-                .Where(g => g.Titles.Any(Constants.IsPrivilegedPekTitle));
+                .Where(g => g.Titles.Any(Constants.IsTrustedPekTitle));
         }
     }
 }

StartSch/Services/UserInfoService.cs

@@ -42,7 +42,7 @@ public async Task OnUserInformationReceived(UserInformationReceivedContext conte
                 .Select(m => m.PekId)
                 .ToList();
             var administeredPekGroupIds = memberships
-                .Where(m => m.Titles.Any(Constants.IsPrivilegedPekTitle))
+                .Where(m => m.Titles.Any(Constants.IsTrustedPekTitle))
                 .Select(m => m.PekId)
                 .ToHashSet();
             Dictionary<int, Page> pekGroupIdToPage = await db.Pages