From 2ca407505b8c9d1cf0d3aefebf708def834625cf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jaros=C5=82aw=20Sadzi=C5=84ski?= Date: Wed, 23 Nov 2022 16:26:40 +0100 Subject: [PATCH] (core) Allow doc owners to view document usage regardless of access rules Summary: Document usage is now available for owners regardless ACL rules Test Plan: Updated Reviewers: georgegevoian Reviewed By: georgegevoian Differential Revision: https://phab.getgrist.com/D3717 --- app/server/lib/GranularAccess.ts | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/app/server/lib/GranularAccess.ts b/app/server/lib/GranularAccess.ts index 3fa0fcb2..7f8a39d6 100644 --- a/app/server/lib/GranularAccess.ts +++ b/app/server/lib/GranularAccess.ts @@ -628,6 +628,10 @@ export class GranularAccess implements GranularAccessForBundle { options: {role?: Role | null} = {} ): Promise { const result: FilteredDocUsageSummary = { ...docUsage }; + // Owners can see everything all the time. + if (await this.isOwner(docSession)) { + return result; + } const role = options.role ?? await this.getNominalAccess(docSession); const hasEditRole = canEdit(role); if (!hasEditRole) { result.dataLimitStatus = null; }