{"action":"created","issue":{"url":"https://api.github.com/repos/theforeman/foreman/issues/2904","labels_url":"https://api.github.com/repos/theforeman/foreman/issues/2904/labels{/name}","comments_url":"https://api.github.com/repos/theforeman/foreman/issues/2904/comments","events_url":"https://api.github.com/repos/theforeman/foreman/issues/2904/events","html_url":"https://github.com/theforeman/foreman/pull/2904","id":116515960,"number":2904,"title":"fixes #12342 - unsaved (new or cloned) hostgrops should keep their puppetclasses","user":{"login":"unorthodoxgeek","id":706359,"avatar_url":"https://avatars.githubusercontent.com/u/706359?v=3","gravatar_id":"","url":"https://api.github.com/users/unorthodoxgeek","html_url":"https://github.com/unorthodoxgeek","followers_url":"https://api.github.com/users/unorthodoxgeek/followers","following_url":"https://api.github.com/users/unorthodoxgeek/following{/other_user}","gists_url":"https://api.github.com/users/unorthodoxgeek/gists{/gist_id}","starred_url":"https://api.github.com/users/unorthodoxgeek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/unorthodoxgeek/subscriptions","organizations_url":"https://api.github.com/users/unorthodoxgeek/orgs","repos_url":"https://api.github.com/users/unorthodoxgeek/repos","events_url":"https://api.github.com/users/unorthodoxgeek/events{/privacy}","received_events_url":"https://api.github.com/users/unorthodoxgeek/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/theforeman/foreman/labels/Ready%20to%20merge","name":"Ready to merge","color":"bfe5bf"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-11-12T09:58:11Z","updated_at":"2015-11-16T13:19:30Z","closed_at":"2015-11-16T13:12:00Z","pull_request":{"url":"https://api.github.com/repos/theforeman/foreman/pulls/2904","html_url":"https://github.com/theforeman/foreman/pull/2904","diff_url":"https://github.com/theforeman/foreman/pull/2904.diff","patch_url":"https://github.com/theforeman/foreman/pull/2904.patch"},"body":"Unsaved hostgroups didn't respond correctly to individual_puppetclasses\r\nbecause that method expects some objects to be saved in the DB. In case\r\nof selecting a new environment, that lead to puppetclasses disappearing.\r\nSolution is - when the object is not saved yet, fallback on the\r\npuppetclasses method."},"comment":{"url":"https://api.github.com/repos/theforeman/foreman/issues/comments/157025368","html_url":"https://github.com/theforeman/foreman/pull/2904#issuecomment-157025368","issue_url":"https://api.github.com/repos/theforeman/foreman/issues/2904","id":157025368,"user":{"login":"unorthodoxgeek","id":706359,"avatar_url":"https://avatars.githubusercontent.com/u/706359?v=3","gravatar_id":"","url":"https://api.github.com/users/unorthodoxgeek","html_url":"https://github.com/unorthodoxgeek","followers_url":"https://api.github.com/users/unorthodoxgeek/followers","following_url":"https://api.github.com/users/unorthodoxgeek/following{/other_user}","gists_url":"https://api.github.com/users/unorthodoxgeek/gists{/gist_id}","starred_url":"https://api.github.com/users/unorthodoxgeek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/unorthodoxgeek/subscriptions","organizations_url":"https://api.github.com/users/unorthodoxgeek/orgs","repos_url":"https://api.github.com/users/unorthodoxgeek/repos","events_url":"https://api.github.com/users/unorthodoxgeek/events{/privacy}","received_events_url":"https://api.github.com/users/unorthodoxgeek/received_events","type":"User","site_admin":false},"created_at":"2015-11-16T13:19:30Z","updated_at":"2015-11-16T13:19:30Z","body":"@domcleal - You, sir, have just made me smile broadly :smile:"}}