Skip to content

Commit eeb0cba

Browse files
committed
Remove unused function
1 parent a20cbfc commit eeb0cba

File tree

1 file changed

+7
-14
lines changed

1 file changed

+7
-14
lines changed

frontend/src/routes/(root)/(logged)/tutorials/+page.svelte

Lines changed: 7 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -58,19 +58,6 @@
5858
return { user, usePreview, previewRole: selectedPreviewRole }
5959
})
6060
61-
/**
62-
* Check if the current user (or preview role) has access to a roles array.
63-
* Handles both normal access and admin preview mode.
64-
*/
65-
function checkAccess(roles?: Role[]): boolean {
66-
const context = accessCheckContext
67-
// Use preview function if admin has selected a different role to preview
68-
if (context.usePreview) {
69-
return hasRoleAccessForPreview(context.previewRole, roles)
70-
}
71-
return hasRoleAccess(context.user, roles)
72-
}
73-
7461
// Get active tabs only (filtered by active and roles)
7562
// Optimized: $derived.by() automatically memoizes - only recalculates when dependencies change
7663
const activeTabs = $derived.by(() => {
@@ -227,12 +214,18 @@
227214
// Calculate progress for each tab
228215
function getTabProgress(tabId: TabId) {
229216
const tabConfig = TUTORIALS_CONFIG[tabId]
217+
const context = accessCheckContext
230218
231219
// Get all tutorial indexes for this tab (filtered by role)
232220
const indexes: number[] = []
233221
for (const tutorial of tabConfig.tutorials) {
234222
if (tutorial.active === false || tutorial.index === undefined) continue
235-
if (!checkAccess(tutorial.roles)) continue
223+
// Use context directly to check access
224+
if (context.usePreview) {
225+
if (!hasRoleAccessForPreview(context.previewRole, tutorial.roles)) continue
226+
} else {
227+
if (!hasRoleAccess(context.user, tutorial.roles)) continue
228+
}
236229
indexes.push(tutorial.index)
237230
}
238231

0 commit comments

Comments
 (0)