App store shows group gid instead of displayname when restricting access

[mkrecek234]

⚠️ This issue respects the following points: ⚠️

• This is a bug, not a question or a configuration/webserver/proxy issue.
• This issue is not already reported on Github (I've searched it).
• Nextcloud Server is up to date. See Maintenance and Release Schedule for supported versions.
• Nextcloud Server is running on 64bit capable CPU, PHP and OS.
• I agree to follow Nextcloud's Code of Conduct.

Bug description

Create a user group ˋtestˋ as an admin.
Rename the user group to ˋfoò
Go to NC Apps section and select any active app
Restrict app access to a group

The list of groups selectable are showing the ˋgidˋ unique name, here ˋtestˋ - renaming that group did not change the ˋgidˋ but only ˋdisplaynamè

Expected: The app store group selector should show displayname not gid

Steps to reproduce

See above

Expected behavior

See above

Installation method

Community Web installer on a VPS or web space

Operating system

Debian/Ubuntu

PHP engine version

PHP 8.0

Web server

Apache (supported)

Database engine version

MySQL

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

Encryption is Disabled

What user-backends are you using?

• Default user-backend (database)
• LDAP/ Active Directory
• SSO - SAML
• Other

Configuration report

N/A

List of activated Apps

N/A

Nextcloud Signing status

N/A

Nextcloud Logs

N/A

Additional info

N/A

[szaimen]

Hi, which nc version?

[mkrecek234]

NC 26

[joshtrichards]

Predates v26. Reproduced in v25.

[joshtrichards]

The issue may be here:

server/apps/settings/src/components/AppDetails.vue

Line 201 in 014a634

return this.app.groups.map(group => { return { id: group, name: group } })

Shouldn't this be something more like:

return this.app.groups

It doesn't seem the map method call serves a purpose here. And even if it does serve one that I'm not seeing, it does not seem like this could possibly being doing what was intended since every group object's id and name elements will both end up being populated with objects that each contain the entire (and same) id and name arrays themselves (!)