Skip to content

[stable30] fix(files_sharing): Sort by correct share attribute ("share with displayname") - #47374

Merged
AndyScherzinger merged 2 commits into
stable30from
backport/47365/stable30
Aug 21, 2024
Merged

[stable30] fix(files_sharing): Sort by correct share attribute ("share with displayname")#47374
AndyScherzinger merged 2 commits into
stable30from
backport/47365/stable30

Conversation

@backportbot

@backportbotbackportbotBot commented Aug 21, 2024

Copy link
Copy Markdown

Backport of #47365

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

…layname")
There is no `title` attribute, so this causes an exception.
Instead sort by the "share with" displayname which will be the user or group the node is shared to.
Meaning this will also be the title of the share in the UI.
If this is not available or there are multiple for the same, then sort by the custom label.
If also this is not set sort by the creation time.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
[skip ci]
@AndyScherzinger

Copy link
Copy Markdown
Member

/compile /

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@AndyScherzinger
AndyScherzinger marked this pull request as ready for review August 21, 2024 10:23
@susnuxsusnux added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 21, 2024
@AndyScherzinger
AndyScherzinger merged commit 947a5e7 into stable30Aug 21, 2024
@AndyScherzinger
AndyScherzinger deleted the backport/47365/stable30 branch August 21, 2024 11:18
@skjnldsvskjnldsv mentioned this pull request Aug 22, 2024
44 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: sharingregression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@AndyScherzinger@susnux@nextcloud-command