Only expose storage location to admins

[PVince81]

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes) => unsure

[PVince81]

/backport to stable25

[nickvergessen]

Now it's also for group admins, is that good enough, or should we wrap with an additional $this->groupManager->isAdmin($currentLoggedInUser->getUID() ?

[PVince81]

@nickvergessen it's for admins and group admins

I've retested with admins and can confirm

[PVince81]

ah, got it. should we exclude group admins. not sure...

[nickvergessen]

I would say they also don't need it. Without system access there is no use of the value 🤔

[come-nc]

I agree groupadmin are not sysadmin, they should not see the value.

[PVince81]

here we go, fixed, rebased, squashed

now only admins see the location

[PVince81]

ok, I've added an assert, squashed, rebased

[PVince81]

@come-nc @nickvergessen can you approve or are there further concerns ? I've made it admin-only to see the storage location, not subadmins any more

[PVince81]

/backport to stable24

[PVince81]

/backport to stable23