{"action":"created","issue":{"url":"https://api.github.com/repos/joomla/joomla-cms/issues/7156","labels_url":"https://api.github.com/repos/joomla/joomla-cms/issues/7156/labels{/name}","comments_url":"https://api.github.com/repos/joomla/joomla-cms/issues/7156/comments","events_url":"https://api.github.com/repos/joomla/joomla-cms/issues/7156/events","html_url":"https://github.com/joomla/joomla-cms/pull/7156","id":87650046,"number":7156,"title":"Remove unnecessary conditional check in category and article views","user":{"login":"joomdonation","id":977664,"avatar_url":"https://avatars.githubusercontent.com/u/977664?v=3","gravatar_id":"","url":"https://api.github.com/users/joomdonation","html_url":"https://github.com/joomdonation","followers_url":"https://api.github.com/users/joomdonation/followers","following_url":"https://api.github.com/users/joomdonation/following{/other_user}","gists_url":"https://api.github.com/users/joomdonation/gists{/gist_id}","starred_url":"https://api.github.com/users/joomdonation/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joomdonation/subscriptions","organizations_url":"https://api.github.com/users/joomdonation/orgs","repos_url":"https://api.github.com/users/joomdonation/repos","events_url":"https://api.github.com/users/joomdonation/events{/privacy}","received_events_url":"https://api.github.com/users/joomdonation/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/joomla/joomla-cms/labels/PR-staging","name":"PR-staging","color":"c7def8"},{"url":"https://api.github.com/repos/joomla/joomla-cms/labels/RTC","name":"RTC","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-06-12T08:09:37Z","updated_at":"2015-06-12T10:46:33Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/joomla/joomla-cms/pulls/7156","html_url":"https://github.com/joomla/joomla-cms/pull/7156","diff_url":"https://github.com/joomla/joomla-cms/pull/7156.diff","patch_url":"https://github.com/joomla/joomla-cms/pull/7156.patch"},"body":"## PR Summary\r\nThis PR simply remove unnecessary condition check in elseif command. That check always return true (if it return false, the code in if block will be run), so it is not needed. \r\n\r\n## Testing\r\nThis can be seen easily from code review. So I think we just need to a maintainer to review and merge it.\r\n\r\nI will do the the same modification for other components (has this same unnecessary code) if this PR is approved. \r\n"},"comment":{"url":"https://api.github.com/repos/joomla/joomla-cms/issues/comments/111446026","html_url":"https://github.com/joomla/joomla-cms/pull/7156#issuecomment-111446026","issue_url":"https://api.github.com/repos/joomla/joomla-cms/issues/7156","id":111446026,"user":{"login":"Bakual","id":1018684,"avatar_url":"https://avatars.githubusercontent.com/u/1018684?v=3","gravatar_id":"","url":"https://api.github.com/users/Bakual","html_url":"https://github.com/Bakual","followers_url":"https://api.github.com/users/Bakual/followers","following_url":"https://api.github.com/users/Bakual/following{/other_user}","gists_url":"https://api.github.com/users/Bakual/gists{/gist_id}","starred_url":"https://api.github.com/users/Bakual/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Bakual/subscriptions","organizations_url":"https://api.github.com/users/Bakual/orgs","repos_url":"https://api.github.com/users/Bakual/repos","events_url":"https://api.github.com/users/Bakual/events{/privacy}","received_events_url":"https://api.github.com/users/Bakual/received_events","type":"User","site_admin":false},"created_at":"2015-06-12T10:46:33Z","updated_at":"2015-06-12T10:46:33Z","body":"RTC based on review. Needs to be squashed before merging."}}