Skip to content

Commit 265c77e

Browse files
blizzzenjeck
authored andcommitted
fix: treat owners like other users
After frontend changes we do not need the intermediate RECIPIENT-ONLY mode any more. And so this fixes setting the override for owners. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
1 parent 4a690c7 commit 265c77e

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

lib/Db/ContextMapper.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -177,14 +177,12 @@ public function findForNavBar(string $userId): array {
177177
$qb->expr()->isNull('n2.display_mode'),
178178
// requires a display mode also depending on the role…
179179
$qb->expr()->orX(
180-
// not an owner: requires RECIPIENT or ALL
180+
// not an owner: requires (RECIPIENT or) ALL
181181
$qb->expr()->andX(
182182
// groups are not considered, yet
183183
$qb->expr()->neq('c.owner_id', $qb->createNamedParameter($userId)),
184184
$qb->expr()->gt('n.display_mode', $qb->createNamedParameter(Application::NAV_ENTRY_MODE_HIDDEN, IQueryBuilder::PARAM_INT)),
185185
),
186-
// an owner (no explicit check necessary): requires ALL
187-
$qb->expr()->eq('n.display_mode', $qb->createNamedParameter(Application::NAV_ENTRY_MODE_ALL, IQueryBuilder::PARAM_INT)),
188186
),
189187
),
190188
// user override

0 commit comments

Comments
 (0)