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

Comment icon is not shown on shared folder #10877

Closed
tobiasKaminsky opened this issue Aug 27, 2018 · 5 comments
Closed

Comment icon is not shown on shared folder #10877

tobiasKaminsky opened this issue Aug 27, 2018 · 5 comments

Comments

@tobiasKaminsky
Copy link
Member

  • share a folder A from user 1 to user 2
  • add a comment on this folder A
  • log in as user 2 and see no comment icon on this folder

It works if user 1 is commenting something on a file b.txt within A (A/b.txt)

@blizzz (as suggested by Joas)

@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #4246 (Shared folder to a group not shown in shared by me), #7125 (Plugins not working on Shared Folders), #5408 (Files: not all shares are shown if folder is marked as favorite), #10308 (Shared folder icons and text is showed inconsistently), and #1072 (Not possible to share current folder (need to display share icon in the breadcrumbs)).

@blizzz
Copy link
Member

blizzz commented Sep 3, 2018

The WebDAV property returns an unread count of 0. Debugging why tomorrow. So far I failed to see that DAV Comments plugins where pulled in at all (but the property is coming somehwere from).

@blizzz
Copy link
Member

blizzz commented Sep 5, 2018

Introduced with 429f8ae

@blizzz blizzz added this to the Nextcloud 15 milestone Sep 5, 2018
@blizzz
Copy link
Member

blizzz commented Sep 5, 2018

I guess the crucial point is that the implementation of getNumberOfUnreadCommentsForFolder works against the filecache table and does not take mounts/shares into account. Thus, even if you get a number it is not necessarily precise.

@icewind1991 you've got an idea how to resolve this without losing too much of the performance benefits?

@danxuliu
Copy link
Member

Fixed by #12897

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants