{"action":"created","issue":{"url":"https://api.github.com/repos/netz98/n98-magerun/issues/573","labels_url":"https://api.github.com/repos/netz98/n98-magerun/issues/573/labels{/name}","comments_url":"https://api.github.com/repos/netz98/n98-magerun/issues/573/comments","events_url":"https://api.github.com/repos/netz98/n98-magerun/issues/573/events","html_url":"https://github.com/netz98/n98-magerun/pull/573","id":82905315,"number":573,"title":"Improve observer list accuracy.","user":{"login":"pocallaghan","id":708624,"avatar_url":"https://avatars.githubusercontent.com/u/708624?v=3","gravatar_id":"","url":"https://api.github.com/users/pocallaghan","html_url":"https://github.com/pocallaghan","followers_url":"https://api.github.com/users/pocallaghan/followers","following_url":"https://api.github.com/users/pocallaghan/following{/other_user}","gists_url":"https://api.github.com/users/pocallaghan/gists{/gist_id}","starred_url":"https://api.github.com/users/pocallaghan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pocallaghan/subscriptions","organizations_url":"https://api.github.com/users/pocallaghan/orgs","repos_url":"https://api.github.com/users/pocallaghan/repos","events_url":"https://api.github.com/users/pocallaghan/events{/privacy}","received_events_url":"https://api.github.com/users/pocallaghan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-30T20:29:13Z","updated_at":"2015-06-01T06:53:10Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/netz98/n98-magerun/pulls/573","html_url":"https://github.com/netz98/n98-magerun/pull/573","diff_url":"https://github.com/netz98/n98-magerun/pull/573.diff","patch_url":"https://github.com/netz98/n98-magerun/pull/573.patch"},"body":"If the class node isn't set in the configuration for an observer,\r\nMagento will also check for the model node. Added support for this to\r\nthe list command."},"comment":{"url":"https://api.github.com/repos/netz98/n98-magerun/issues/comments/107328392","html_url":"https://github.com/netz98/n98-magerun/pull/573#issuecomment-107328392","issue_url":"https://api.github.com/repos/netz98/n98-magerun/issues/573","id":107328392,"user":{"login":"tkn98","id":10513307,"avatar_url":"https://avatars.githubusercontent.com/u/10513307?v=3","gravatar_id":"","url":"https://api.github.com/users/tkn98","html_url":"https://github.com/tkn98","followers_url":"https://api.github.com/users/tkn98/followers","following_url":"https://api.github.com/users/tkn98/following{/other_user}","gists_url":"https://api.github.com/users/tkn98/gists{/gist_id}","starred_url":"https://api.github.com/users/tkn98/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tkn98/subscriptions","organizations_url":"https://api.github.com/users/tkn98/orgs","repos_url":"https://api.github.com/users/tkn98/repos","events_url":"https://api.github.com/users/tkn98/events{/privacy}","received_events_url":"https://api.github.com/users/tkn98/received_events","type":"User","site_admin":false},"created_at":"2015-06-01T06:53:10Z","updated_at":"2015-06-01T06:53:10Z","body":"Thanks for reporting and the PR.\r\n\r\nJust a little question: What happens if none of the two are set? Are those required or optional?\r\n\r\nFor the patch itself, there is a little spacing issue and you could move the logic to create that string into a private function of it's own with two parameters: $observer and $type."}}