Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(files_sharing): Sort by correct share attribute ("share with displayname") #47365

Merged
merged 2 commits into from
Aug 21, 2024

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Aug 20, 2024

Summary

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.

Checklist

@susnux
Copy link
Contributor Author

susnux commented Aug 20, 2024

/backport to stable30

…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>
@susnux susnux force-pushed the fix/share-sorting branch from d6a853c to dc153f2 Compare August 21, 2024 00:32
@susnux
Copy link
Contributor Author

susnux commented Aug 21, 2024

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@AndyScherzinger AndyScherzinger merged commit cf56874 into master Aug 21, 2024
113 checks passed
@AndyScherzinger AndyScherzinger deleted the fix/share-sorting branch August 21, 2024 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Error loadings shares list TypeError: t.title is undefined
4 participants