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

if no share were send/received "shared with you/others" must not show all files #2150

Closed
individual-it opened this issue Oct 4, 2019 · 7 comments
Labels
Estimation:L(5) Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug Something isn't working

Comments

@individual-it
Copy link
Member

Steps to reproduce

  1. have a user that did not receive or send ever a share
  2. navigate to "All Files"
  3. navigate to "Shared with me"
  4. navigate to "shared with others"

Expected behaviour

"shared with me" & "shared with others" should be empty

Actual behaviour

all files of the user are listed

shares-lists

@individual-it individual-it added the Type:Bug Something isn't working label Oct 4, 2019
@PVince81 PVince81 added Priority:p2-high Escalation, on top of current planning, release blocker feature:collaborators labels Oct 8, 2019
@PVince81 PVince81 added this to the Milestone 1: Phoenix for users milestone Oct 8, 2019
@PVince81
Copy link
Contributor

PVince81 commented Oct 8, 2019

From what I see, the logic is skipping "LOAD_FILES" action whenever there are no results from the OCS Share call for both views.

It should be modified to actually clear the current file list.

@PVince81
Copy link
Contributor

PVince81 commented Oct 8, 2019

Ok, it seems to use the first entry in the list for currentFolder as our current state expects a value for it. So it's not trivial to generate an empty result set there without virtualizing the currentFolder.

I don't think the current code is correct. We need to be able to support a flat list of results that do not have any current folder in place, like the list of favorites or shares. The view should be able to work with a null currentFolder then.

I'll need to talk with @LukasHirt about a proper solution.

@PVince81 PVince81 self-assigned this Oct 18, 2019
@PVince81 PVince81 changed the title if no share were send "shared with others" show all files if no shares were received "shared with me" show all files if no share were send/received "shared with you/others" must not show all files Oct 21, 2019
@PVince81
Copy link
Contributor

as discussed with @LukasHirt, the first attempt will be at removing the requirement that currentFolder is set. might require going through the whole source code...

@PVince81
Copy link
Contributor

PVince81 commented Oct 21, 2019

slightly related to #2077 in terms of reusable file list

@PVince81
Copy link
Contributor

@LukasHirt
Copy link
Collaborator

The base issue is solved in #2280 thanks to the reset of the files array when the folder is being loaded. Still, needs anyway all the mentioned fixes.

@PVince81
Copy link
Contributor

PVince81 commented Mar 2, 2020

this isn't happening any more. I extensively tested such scenario when testing the "empty indicator" for the lists, closing.

@PVince81 PVince81 closed this as completed Mar 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Estimation:L(5) Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants