{"action":"created","issue":{"url":"https://api.github.com/repos/theforeman/foreman-xen/issues/36","labels_url":"https://api.github.com/repos/theforeman/foreman-xen/issues/36/labels{/name}","comments_url":"https://api.github.com/repos/theforeman/foreman-xen/issues/36/comments","events_url":"https://api.github.com/repos/theforeman/foreman-xen/issues/36/events","html_url":"https://github.com/theforeman/foreman-xen/pull/36","id":124841005,"number":36,"title":"filter duplicate results in storages","user":{"login":"mmoll","id":1434927,"avatar_url":"https://avatars.githubusercontent.com/u/1434927?v=3","gravatar_id":"","url":"https://api.github.com/users/mmoll","html_url":"https://github.com/mmoll","followers_url":"https://api.github.com/users/mmoll/followers","following_url":"https://api.github.com/users/mmoll/following{/other_user}","gists_url":"https://api.github.com/users/mmoll/gists{/gist_id}","starred_url":"https://api.github.com/users/mmoll/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmoll/subscriptions","organizations_url":"https://api.github.com/users/mmoll/orgs","repos_url":"https://api.github.com/users/mmoll/repos","events_url":"https://api.github.com/users/mmoll/events{/privacy}","received_events_url":"https://api.github.com/users/mmoll/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":12,"created_at":"2016-01-04T21:15:11Z","updated_at":"2016-01-07T16:03:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/theforeman/foreman-xen/pulls/36","html_url":"https://github.com/theforeman/foreman-xen/pull/36","diff_url":"https://github.com/theforeman/foreman-xen/pull/36.diff","patch_url":"https://github.com/theforeman/foreman-xen/pull/36.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/theforeman/foreman-xen/issues/comments/169707805","html_url":"https://github.com/theforeman/foreman-xen/pull/36#issuecomment-169707805","issue_url":"https://api.github.com/repos/theforeman/foreman-xen/issues/36","id":169707805,"user":{"login":"mmoll","id":1434927,"avatar_url":"https://avatars.githubusercontent.com/u/1434927?v=3","gravatar_id":"","url":"https://api.github.com/users/mmoll","html_url":"https://github.com/mmoll","followers_url":"https://api.github.com/users/mmoll/followers","following_url":"https://api.github.com/users/mmoll/following{/other_user}","gists_url":"https://api.github.com/users/mmoll/gists{/gist_id}","starred_url":"https://api.github.com/users/mmoll/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmoll/subscriptions","organizations_url":"https://api.github.com/users/mmoll/orgs","repos_url":"https://api.github.com/users/mmoll/repos","events_url":"https://api.github.com/users/mmoll/events{/privacy}","received_events_url":"https://api.github.com/users/mmoll/received_events","type":"User","site_admin":false},"created_at":"2016-01-07T16:03:51Z","updated_at":"2016-01-07T16:03:51Z","body":"It is. Please see https://github.com/theforeman/foreman-xen/commit/f04e314e1256878e530c5a05ecd739ad0e96f206#commitcomment-14397646 ff.\r\n\r\nEspecially the statement \"Yes, reverting to if statement fixes it. Also this patch seems to fix it.\" makes me want to have this tested by somebody who was hit by that back then.\r\n\r\nBecause of the sheer complexity of the code for doing things that are not really complex (mainly by choosing procedural over functional patterns to comb through data structures) it's easy to fix things for one situation but break it for others."}}