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

Add a sharing option to restrict autocompletion for users in different groups #27849

Closed
pmaier1 opened this issue May 10, 2017 · 8 comments · Fixed by #27869
Closed

Add a sharing option to restrict autocompletion for users in different groups #27849

pmaier1 opened this issue May 10, 2017 · 8 comments · Fixed by #27869
Assignees
Labels
enhancement p1-urgent Critical issue, need to consider hotfix with just that issue status/STALE
Milestone

Comments

@pmaier1
Copy link
Contributor

pmaier1 commented May 10, 2017

As an admin I want to restrict my users to only find other users that share group membership of at least one group in the sharing dialog. Users should still be able to share with users that they don't share group membership with if they know the exact username / mail address.

Basically we need to copy the sharing restriction 'Restrict users to only share with users in their groups' and enable sharing.

Please make it a suboption of 'Allow username autocompletion in share dialog. If this is disabled the full username needs to be entered.'

  • Restrict username autocompletion to group membership
@pmaier1
Copy link
Contributor Author

pmaier1 commented May 10, 2017

@felixboehm @PVince81

@pmaier1 pmaier1 changed the title Add a sharing restriction to restrict autocompletion for users in different groups Add a sharing option to restrict autocompletion for users in different groups May 10, 2017
@PVince81
Copy link
Contributor

With a bit of hope the methods that @tomneedham added recently might be reusable for this. Might need to extend them to also have an option $restrictToGroupMemberships when seraching

@PVince81
Copy link
Contributor

The way I understand it we need to extend the ShareesController to check for the new option.

If the new option is set "Restrict autocomplete to group memberships", just use findUsersInGroup for autocomplete instead of find when searching, but still allow full input.

This + unit tests I think 1md might be enough.

@PVince81 PVince81 self-assigned this May 12, 2017
@PVince81
Copy link
Contributor

What about group name autocompletion ? I assume the new option will also only display autocomplete results for groups the user is a member of. And entering a full group name still allows sharing.

@PVince81
Copy link
Contributor

Implementation: 10 minutes. Writing tests, one full day ;-)

PR here #27869 (tests still to be written)

@felixboehm felixboehm added the p1-urgent Critical issue, need to consider hotfix with just that issue label May 12, 2017
@felixboehm
Copy link
Contributor

Let's do it

@PVince81
Copy link
Contributor

there are only 4 test cases, all added a unit tests and integration tests: #27869

however there might be other things to test, find out like what happens when you combine the new option with any of the old options...

@lock
Copy link

lock bot commented Aug 1, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement p1-urgent Critical issue, need to consider hotfix with just that issue status/STALE
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants