{"action":"created","issue":{"url":"https://api.github.com/repos/owncloud/core/issues/15253","labels_url":"https://api.github.com/repos/owncloud/core/issues/15253/labels{/name}","comments_url":"https://api.github.com/repos/owncloud/core/issues/15253/comments","events_url":"https://api.github.com/repos/owncloud/core/issues/15253/events","html_url":"https://github.com/owncloud/core/issues/15253","id":64647925,"number":15253,"title":"group & members permission mismatch","user":{"login":"mhaggel","id":5560970,"avatar_url":"https://avatars.githubusercontent.com/u/5560970?v=3","gravatar_id":"","url":"https://api.github.com/users/mhaggel","html_url":"https://github.com/mhaggel","followers_url":"https://api.github.com/users/mhaggel/followers","following_url":"https://api.github.com/users/mhaggel/following{/other_user}","gists_url":"https://api.github.com/users/mhaggel/gists{/gist_id}","starred_url":"https://api.github.com/users/mhaggel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mhaggel/subscriptions","organizations_url":"https://api.github.com/users/mhaggel/orgs","repos_url":"https://api.github.com/users/mhaggel/repos","events_url":"https://api.github.com/users/mhaggel/events{/privacy}","received_events_url":"https://api.github.com/users/mhaggel/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/owncloud/core/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/owncloud/core/labels/sharing","name":"sharing","color":"e102d8"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-03-26T22:29:26Z","updated_at":"2015-10-09T06:54:20Z","closed_at":null,"body":"Hi,\r\nI need some help: Either I have a problem understanding how the permissions of a share work or there is a bug in the code (owncloud 8.0.2).\r\n\r\nsituation: User A shares a folder X with a group M. All the group members (the users) have initially the same permissions ('15' by looking into the DB). \r\nBy revoking the 'delete' one from the group then the group & members get permissions '7'. (so far so good)\r\nGiving back the removed one the group has again the '15', however the individuals (users) got stuck with the '7'. \r\nEventually if i continue play around with the group permissions i will end up having a share with group permission 15 and individual group members with 1!! \r\nSince i am relatively new to ownCloud, i would like to ask whether this is an expected behavior or not. \r\n\r\nOne more clue is that i am getting users & groups from ldap.\r\n\r\nMany thanks in advance.\r\nM.\r\n"},"comment":{"url":"https://api.github.com/repos/owncloud/core/issues/comments/146772326","html_url":"https://github.com/owncloud/core/issues/15253#issuecomment-146772326","issue_url":"https://api.github.com/repos/owncloud/core/issues/15253","id":146772326,"user":{"login":"rullzer","id":45821,"avatar_url":"https://avatars.githubusercontent.com/u/45821?v=3","gravatar_id":"","url":"https://api.github.com/users/rullzer","html_url":"https://github.com/rullzer","followers_url":"https://api.github.com/users/rullzer/followers","following_url":"https://api.github.com/users/rullzer/following{/other_user}","gists_url":"https://api.github.com/users/rullzer/gists{/gist_id}","starred_url":"https://api.github.com/users/rullzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rullzer/subscriptions","organizations_url":"https://api.github.com/users/rullzer/orgs","repos_url":"https://api.github.com/users/rullzer/repos","events_url":"https://api.github.com/users/rullzer/events{/privacy}","received_events_url":"https://api.github.com/users/rullzer/received_events","type":"User","site_admin":false},"created_at":"2015-10-09T06:54:20Z","updated_at":"2015-10-09T06:54:20Z","body":"So yeah we simply do not check for the added permissions case: https://github.com/owncloud/core/blob/master/lib/private/share/share.php#L1105-L1106\r\n\r\nThis is weird since all children should be updated as well. Or well the children of the same share.. e.g. the user moved the mountpoint so there is an extra entry in the shares table.\r\n\r\nLets write a ~~unit~~ test for this. And then see if we can fix it without breaking all of sharing."}}