Limit received share interactions
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 36.9k
- Forks
- 5.2k
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 713
Description
Currently we allow all users to move files out of shares. However this leads to some scalibility and usability problems.
Assume:
userAhas a folderfoowith 3 filesbar1,bar2,bar3.- Now
userAsharesfoowithgroupB userX,userYanduserZare all part ofgroupB.userXmovesbar1out of the share to its own data
Now we have logic in place here that the file gets also moved to the trashbin of userA. But it is still weird behavior wise. Because all of sudden the file is gone for userY and userZ.
If we'd forbid moving files out of shared folder to your own folders (of course if you own the files you can still move whatever you want since they are your files). We'd could improve the handling here. We'd could come up with things like shared trashbins for shared files maybe.
I tried it out on gdrive with @skjnldsv yesterday and they have similar limitations. You can't move files out of a folder shared with you. There is a button 'copy to my drive'. But that is of course different.
This would of course be a big change. But it makes a lot of things simpler. And I guess also easier to explain to users.
Feedback welcome: @skjnldsv @MorrisJobke @nickvergessen @icewind1991 @jancborchardt @schiessle @blizzz @danxuliu
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No files, tests, or entry points are named. Begin by tracing how moving files from received shares is handled and how ownership affects the operation; done means users cannot move files they do not own out of shared folders, while owners retain that ability.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- authorization, backend
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100