Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Jul 22, 2025

Summary

The variable is initialized to {} which is truthy, so instead just check if there is a user assigned to the share info.
It fixes this:
Screenshot 2025-07-23 at 00-21-18 All files - Files - Nextcloud

Checklist

@susnux susnux requested a review from a team as a code owner July 22, 2025 22:49
@susnux susnux requested review from artonge, skjnldsv and szaimen and removed request for a team July 22, 2025 22:49
@susnux susnux requested review from leftybournes and nfebe and removed request for artonge July 22, 2025 22:50
@susnux susnux added this to the Nextcloud 32 milestone Jul 22, 2025
@susnux
Copy link
Contributor Author

susnux commented Jul 22, 2025

/backport to stable31

@susnux
Copy link
Contributor Author

susnux commented Aug 6, 2025

/compile rebase

susnux and others added 2 commits August 6, 2025 09:37
The variable is intialized to `{}` which is truthy, so instead just
check if there is a user assigned to the share info.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nextcloud-command nextcloud-command force-pushed the fix/only-show-reshare-if-there-is branch from 2a4c3c6 to 96011a1 Compare August 6, 2025 09:40
@nextcloud-command nextcloud-command requested a review from a team as a code owner August 6, 2025 09:40
@skjnldsv skjnldsv enabled auto-merge August 6, 2025 10:02
@skjnldsv skjnldsv merged commit 535bf3a into master Aug 6, 2025
154 of 160 checks passed
@skjnldsv skjnldsv deleted the fix/only-show-reshare-if-there-is branch August 6, 2025 10:40
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
@skjnldsv skjnldsv modified the milestones: Nextcloud 33, Nextcloud 32 Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants