{"action":"created","issue":{"url":"https://api.github.com/repos/owncloud/core/issues/15564","labels_url":"https://api.github.com/repos/owncloud/core/issues/15564/labels{/name}","comments_url":"https://api.github.com/repos/owncloud/core/issues/15564/comments","events_url":"https://api.github.com/repos/owncloud/core/issues/15564/events","html_url":"https://github.com/owncloud/core/pull/15564","id":68042809,"number":15564,"title":"Do not try to make user subadmin of group twice","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},"labels":[{"url":"https://api.github.com/repos/owncloud/core/labels/5%20-%20To%20review","name":"5 - To review","color":"ededed"},{"url":"https://api.github.com/repos/owncloud/core/labels/backport-request","name":"backport-request","color":"006b75"},{"url":"https://api.github.com/repos/owncloud/core/labels/bug","name":"bug","color":"fc2929"}],"state":"closed","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/owncloud/core/milestones/31","html_url":"https://github.com/owncloud/core/milestones/8.1-current","labels_url":"https://api.github.com/repos/owncloud/core/milestones/31/labels","id":912160,"number":31,"title":"8.1-current","description":"","creator":{"login":"MorrisJobke","id":245432,"avatar_url":"https://avatars.githubusercontent.com/u/245432?v=3","gravatar_id":"","url":"https://api.github.com/users/MorrisJobke","html_url":"https://github.com/MorrisJobke","followers_url":"https://api.github.com/users/MorrisJobke/followers","following_url":"https://api.github.com/users/MorrisJobke/following{/other_user}","gists_url":"https://api.github.com/users/MorrisJobke/gists{/gist_id}","starred_url":"https://api.github.com/users/MorrisJobke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MorrisJobke/subscriptions","organizations_url":"https://api.github.com/users/MorrisJobke/orgs","repos_url":"https://api.github.com/users/MorrisJobke/repos","events_url":"https://api.github.com/users/MorrisJobke/events{/privacy}","received_events_url":"https://api.github.com/users/MorrisJobke/received_events","type":"User","site_admin":false},"open_issues":160,"closed_issues":761,"state":"open","created_at":"2014-12-22T12:04:01Z","updated_at":"2015-04-14T11:35:13Z","due_on":"2015-04-02T22:00:00Z","closed_at":null},"comments":6,"created_at":"2015-04-13T08:38:54Z","updated_at":"2015-04-14T12:32:52Z","closed_at":"2015-04-13T10:46:43Z","pull_request":{"url":"https://api.github.com/repos/owncloud/core/pulls/15564","html_url":"https://github.com/owncloud/core/pull/15564","diff_url":"https://github.com/owncloud/core/pull/15564.diff","patch_url":"https://github.com/owncloud/core/pull/15564.patch"},"body":"As stated in #13709\r\n\r\nIf the provisioning api is used to make a user subadmin of a group that user is already a subadmin just return success. In my opinion returning success is better than returning an error since the desired outcome is already in place. \r\n\r\nAdded unit test to ensure things keep working.\r\n\r\nProbably best to get this into 8.1\r\n\r\n@karlitschek probably a good idea to backport to 8.0 as well\r\n\r\nCC: @DeepDiver1975 @LukasReschke @MorrisJobke "},"comment":{"url":"https://api.github.com/repos/owncloud/core/issues/comments/92794691","html_url":"https://github.com/owncloud/core/pull/15564#issuecomment-92794691","issue_url":"https://api.github.com/repos/owncloud/core/issues/15564","id":92794691,"user":{"login":"karlitschek","id":867445,"avatar_url":"https://avatars.githubusercontent.com/u/867445?v=3","gravatar_id":"","url":"https://api.github.com/users/karlitschek","html_url":"https://github.com/karlitschek","followers_url":"https://api.github.com/users/karlitschek/followers","following_url":"https://api.github.com/users/karlitschek/following{/other_user}","gists_url":"https://api.github.com/users/karlitschek/gists{/gist_id}","starred_url":"https://api.github.com/users/karlitschek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/karlitschek/subscriptions","organizations_url":"https://api.github.com/users/karlitschek/orgs","repos_url":"https://api.github.com/users/karlitschek/repos","events_url":"https://api.github.com/users/karlitschek/events{/privacy}","received_events_url":"https://api.github.com/users/karlitschek/received_events","type":"User","site_admin":false},"created_at":"2015-04-14T12:32:52Z","updated_at":"2015-04-14T12:32:52Z","body":"yes. please backport "}}