{"action":"created","issue":{"url":"https://api.github.com/repos/phpbb-extensions/collapsible-categories/issues/53","repository_url":"https://api.github.com/repos/phpbb-extensions/collapsible-categories","labels_url":"https://api.github.com/repos/phpbb-extensions/collapsible-categories/issues/53/labels{/name}","comments_url":"https://api.github.com/repos/phpbb-extensions/collapsible-categories/issues/53/comments","events_url":"https://api.github.com/repos/phpbb-extensions/collapsible-categories/issues/53/events","html_url":"https://github.com/phpbb-extensions/collapsible-categories/pull/53","id":189467903,"number":53,"title":"Refactor Collapsible Cats","user":{"login":"VSEphpbb","id":303711,"avatar_url":"https://avatars.githubusercontent.com/u/303711?v=3","gravatar_id":"","url":"https://api.github.com/users/VSEphpbb","html_url":"https://github.com/VSEphpbb","followers_url":"https://api.github.com/users/VSEphpbb/followers","following_url":"https://api.github.com/users/VSEphpbb/following{/other_user}","gists_url":"https://api.github.com/users/VSEphpbb/gists{/gist_id}","starred_url":"https://api.github.com/users/VSEphpbb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/VSEphpbb/subscriptions","organizations_url":"https://api.github.com/users/VSEphpbb/orgs","repos_url":"https://api.github.com/users/VSEphpbb/repos","events_url":"https://api.github.com/users/VSEphpbb/events{/privacy}","received_events_url":"https://api.github.com/users/VSEphpbb/received_events","type":"User","site_admin":false},"labels":[{"id":195182811,"url":"https://api.github.com/repos/phpbb-extensions/collapsible-categories/labels/enhancement","name":"enhancement","color":"84b6eb","default":true},{"id":195187778,"url":"https://api.github.com/repos/phpbb-extensions/collapsible-categories/labels/Ready%20to%20Review","name":"Ready to Review","color":"c7def8","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/phpbb-extensions/collapsible-categories/milestones/3","html_url":"https://github.com/phpbb-extensions/collapsible-categories/milestones/1.0.1","labels_url":"https://api.github.com/repos/phpbb-extensions/collapsible-categories/milestones/3/labels","id":1292003,"number":3,"title":"1.0.1","description":"","creator":{"login":"VSEphpbb","id":303711,"avatar_url":"https://avatars.githubusercontent.com/u/303711?v=3","gravatar_id":"","url":"https://api.github.com/users/VSEphpbb","html_url":"https://github.com/VSEphpbb","followers_url":"https://api.github.com/users/VSEphpbb/followers","following_url":"https://api.github.com/users/VSEphpbb/following{/other_user}","gists_url":"https://api.github.com/users/VSEphpbb/gists{/gist_id}","starred_url":"https://api.github.com/users/VSEphpbb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/VSEphpbb/subscriptions","organizations_url":"https://api.github.com/users/VSEphpbb/orgs","repos_url":"https://api.github.com/users/VSEphpbb/repos","events_url":"https://api.github.com/users/VSEphpbb/events{/privacy}","received_events_url":"https://api.github.com/users/VSEphpbb/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":2,"state":"open","created_at":"2015-09-05T18:01:43Z","updated_at":"2016-11-15T21:35:34Z","due_on":null,"closed_at":null},"comments":1,"created_at":"2016-11-15T18:21:14Z","updated_at":"2016-11-16T03:10:19Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/phpbb-extensions/collapsible-categories/pulls/53","html_url":"https://github.com/phpbb-extensions/collapsible-categories/pull/53","diff_url":"https://github.com/phpbb-extensions/collapsible-categories/pull/53.diff","patch_url":"https://github.com/phpbb-extensions/collapsible-categories/pull/53.patch"},"body":"This PR extracts some logic out of the event listener, simplifying the event listener and giving our Operator object more useable methods.\r\n\r\nThe main reason for this change is to make it a little bt easier for other extensions that want to integrate collapsible cats into their exts."},"comment":{"url":"https://api.github.com/repos/phpbb-extensions/collapsible-categories/issues/comments/260841487","html_url":"https://github.com/phpbb-extensions/collapsible-categories/pull/53#issuecomment-260841487","issue_url":"https://api.github.com/repos/phpbb-extensions/collapsible-categories/issues/53","id":260841487,"user":{"login":"VSEphpbb","id":303711,"avatar_url":"https://avatars.githubusercontent.com/u/303711?v=3","gravatar_id":"","url":"https://api.github.com/users/VSEphpbb","html_url":"https://github.com/VSEphpbb","followers_url":"https://api.github.com/users/VSEphpbb/followers","following_url":"https://api.github.com/users/VSEphpbb/following{/other_user}","gists_url":"https://api.github.com/users/VSEphpbb/gists{/gist_id}","starred_url":"https://api.github.com/users/VSEphpbb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/VSEphpbb/subscriptions","organizations_url":"https://api.github.com/users/VSEphpbb/orgs","repos_url":"https://api.github.com/users/VSEphpbb/repos","events_url":"https://api.github.com/users/VSEphpbb/events{/privacy}","received_events_url":"https://api.github.com/users/VSEphpbb/received_events","type":"User","site_admin":false},"created_at":"2016-11-16T03:10:19Z","updated_at":"2016-11-16T03:10:19Z","body":"Just trying something different. \n\nSent from my iPhone\n\n> On Nov 15, 2016, at 4:49 PM, Michael Cullum <notifications@github.com> wrote:\n> \n> @michaelcullum commented on this pull request.\n> \n> In operator/operator.php:\n> \n> >  \t */\n>  \tprotected function set_collapsed_categories($forum_id)\n>  \t{\n> -\t\tif (!isset($this->collapsed_categories))\n> +\t\tif ($this->collapsed_categories === null)\n> Why?\n> \n> —\n> You are receiving this because you modified the open/close state.\n> Reply to this email directly, view it on GitHub, or mute the thread.\n> \n"}}
{"actor":{"display_login":"VSEphpbb"}}