{"action":"opened","issue":{"url":"https://api.github.com/repos/FluidTYPO3/flux/issues/824","labels_url":"https://api.github.com/repos/FluidTYPO3/flux/issues/824/labels{/name}","comments_url":"https://api.github.com/repos/FluidTYPO3/flux/issues/824/comments","events_url":"https://api.github.com/repos/FluidTYPO3/flux/issues/824/events","html_url":"https://github.com/FluidTYPO3/flux/issues/824","id":75138229,"number":824,"title":"No protection against inserting a content element as its own child","user":{"login":"JostBaron","id":2916594,"avatar_url":"https://avatars.githubusercontent.com/u/2916594?v=3","gravatar_id":"","url":"https://api.github.com/users/JostBaron","html_url":"https://github.com/JostBaron","followers_url":"https://api.github.com/users/JostBaron/followers","following_url":"https://api.github.com/users/JostBaron/following{/other_user}","gists_url":"https://api.github.com/users/JostBaron/gists{/gist_id}","starred_url":"https://api.github.com/users/JostBaron/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JostBaron/subscriptions","organizations_url":"https://api.github.com/users/JostBaron/orgs","repos_url":"https://api.github.com/users/JostBaron/repos","events_url":"https://api.github.com/users/JostBaron/events{/privacy}","received_events_url":"https://api.github.com/users/JostBaron/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-05-11T08:38:03Z","updated_at":"2015-05-11T08:38:03Z","closed_at":null,"body":"It is somehow possible (don't know why, and can't reproduce it, but an editor did it) to insert a content element as its own child. Doing this has the effect that the page module can't be opened any longer for that page because of an infinite recursion. \r\n\r\nThis could be mitigated by adding two checks:\r\n\r\n 1. If a content element has children, and a child has the same ID than the element itself, don't render it (in previews and in the frontend). Maybe print a warning instead, or silently ignore it.\r\n 2. If a content element is pasted into another element, and both uids are the same, don't do it and show a warning."}}