-
Notifications
You must be signed in to change notification settings - Fork 83
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
Fixed searching for groups in the sharing sideview #1395
Conversation
The result of FolderManager::getGroups and ::searchGroups are indexed arrays. Thus, the data returned by the .../search?... route are JSON Objects and not array. Signed-off-by: Jan Niklas Grieb (home) <jan.grieb@gmail.com>
…xtcloud#633 nextcloud#1096 Filter for any non-existing group reference before returning GID and displayname. This enables to continue editing the ACLs despite outdated group references in the groupfolder's permissions. Signed-off-by: Jan Niklas Grieb (home) <jan.grieb@gmail.com>
I added a second commit that avoids calling getGID() on null when searching for available groups, referring to issues #633 and #1096 . This other fix is in the backend and filters for any non-existing group reference before returning GID and displayname. This enables to continue editing the ACLs despite outdated group references in the groupfolder's permissions. If desirable, I could also split the frontend and backend fixes into two PRs. |
/backport to stable21 |
/backport to stable20 |
/backport to stable19 |
The result of FolderManager::getGroups and ::searchGroups are indexed
arrays. Thus, the data returned by the .../search?... route
are JSON Objects and not arrays.