{"action":"created","issue":{"url":"https://api.github.com/repos/Elgg/Elgg/issues/10636","repository_url":"https://api.github.com/repos/Elgg/Elgg","labels_url":"https://api.github.com/repos/Elgg/Elgg/issues/10636/labels{/name}","comments_url":"https://api.github.com/repos/Elgg/Elgg/issues/10636/comments","events_url":"https://api.github.com/repos/Elgg/Elgg/issues/10636/events","html_url":"https://github.com/Elgg/Elgg/issues/10636","id":192067170,"number":10636,"title":"User delete causes owned metadata/annotation deletes","user":{"login":"mrclay","id":170687,"avatar_url":"https://avatars.githubusercontent.com/u/170687?v=3","gravatar_id":"","url":"https://api.github.com/users/mrclay","html_url":"https://github.com/mrclay","followers_url":"https://api.github.com/users/mrclay/followers","following_url":"https://api.github.com/users/mrclay/following{/other_user}","gists_url":"https://api.github.com/users/mrclay/gists{/gist_id}","starred_url":"https://api.github.com/users/mrclay/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrclay/subscriptions","organizations_url":"https://api.github.com/users/mrclay/orgs","repos_url":"https://api.github.com/users/mrclay/repos","events_url":"https://api.github.com/users/mrclay/events{/privacy}","received_events_url":"https://api.github.com/users/mrclay/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-11-28T17:29:32Z","updated_at":"2016-12-09T15:12:25Z","closed_at":null,"body":"While it's makes good sense to delete the user's content, and MD/annotations on that content, it's not clear that the default behavior should be to also delete MD/annotations on *other* content.\r\n\r\nWe [started deleting owned MD/annotations](https://github.com/Elgg/Elgg/commit/8b5fd806d6871aaeb179376328a8af215ed44f88) for 1.6, because comments (which were then annotations) were being left over and causing problems.\r\n\r\nNow that we'd like to move 2nd entity attributes to metadata (#10629), this poses some danger. A user can update an entity's attribute (now stored as MD) and if that user is deleted, the entity attribute will be deleted too.\r\n\r\nMy preferred solution would be to strip metadata specifically down to simply a key-value store, no access control (already a goal) but also no owner. With this change, an entity's attributes would not be in danger of deletion depending on who last touched them."},"comment":{"url":"https://api.github.com/repos/Elgg/Elgg/issues/comments/266036947","html_url":"https://github.com/Elgg/Elgg/issues/10636#issuecomment-266036947","issue_url":"https://api.github.com/repos/Elgg/Elgg/issues/10636","id":266036947,"user":{"login":"jdalsem","id":861833,"avatar_url":"https://avatars.githubusercontent.com/u/861833?v=3","gravatar_id":"","url":"https://api.github.com/users/jdalsem","html_url":"https://github.com/jdalsem","followers_url":"https://api.github.com/users/jdalsem/followers","following_url":"https://api.github.com/users/jdalsem/following{/other_user}","gists_url":"https://api.github.com/users/jdalsem/gists{/gist_id}","starred_url":"https://api.github.com/users/jdalsem/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jdalsem/subscriptions","organizations_url":"https://api.github.com/users/jdalsem/orgs","repos_url":"https://api.github.com/users/jdalsem/repos","events_url":"https://api.github.com/users/jdalsem/events{/privacy}","received_events_url":"https://api.github.com/users/jdalsem/received_events","type":"User","site_admin":false},"created_at":"2016-12-09T15:12:25Z","updated_at":"2016-12-09T15:12:25Z","body":"Making the choice for removing owner would make #10629 a whole lot easier... what do other @elgg/core members think of this?\r\n\r\nPersonally i would like to keep at least time_created on the metadata, but otherwise the rest can be removed. We do need to investigate some real life cases to see if metadata owners differ from metadata entity owners and why..."}}
{"actor":{"display_login":"jdalsem"}}