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

Make groupfolders independent from existing folders owned by admin or ppl lose data! #26

Closed
jospoortvliet opened this issue Jun 30, 2017 · 3 comments

Comments

@jospoortvliet
Copy link
Member

jospoortvliet commented Jun 30, 2017

There's a weird overlap with existing folders. Take this scenario:

  • Go to the Files app and create a folder named superawesometest
  • Put a few files in it.
  • Go to the Group Folders admin and create a folder named superawesometest
  • Add the Admin group to the access rights for superawesometest
  • Go back to Files and observe the new Group Folder has overwritten the existing superawesometest and it is empty. Where are my files?!!?!
  • Go to the Group Folders admin and remove Admin group access to the folder superawesometest
  • Go back to Files and observe the new Group Folder has disappeared and the existing superawesometest is back with its original content!
  • Note that this overwriting of existing folders with (empty) Group Folders also happens with users - if you create a group folder with the same name as a folder one of your users has, they will lose access to that folder and all its content.

I think Group Folders should act like shares - if somebody shares a folder with the same name as an existing one with you, it should show up besides the existing one, not overwrite it.

@jospoortvliet
Copy link
Member Author

jospoortvliet commented Jun 30, 2017

  • This bug is also in master (1.10), just tested
  • deleting the Group Folder does NOT delete the users' folder and its data (thankfully). It merely obscures it while the Group Folder exists.
  • did not find any other problems with Group Folders 1.10 - nice improvements ;-)

@radiantwave
Copy link

#18

@jospoortvliet
Copy link
Member Author

Ah, now I understand what you reported ;-)

May I suggest to try and have a more extensive description - maybe @icewind1991 understood the problem but I didn't, at least... Now I'm sorry for creating a duplicate. Let me just make mine a comment on yours.

@wiswedel wiswedel changed the title BUG: Make groupfolders independent from existing folders owned by admin or ppl lose data! Make groupfolders independent from existing folders owned by admin or ppl lose data! Jul 3, 2019
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

2 participants