{"action":"created","issue":{"url":"https://api.github.com/repos/codeborne/selenide/issues/830","repository_url":"https://api.github.com/repos/codeborne/selenide","labels_url":"https://api.github.com/repos/codeborne/selenide/issues/830/labels{/name}","comments_url":"https://api.github.com/repos/codeborne/selenide/issues/830/comments","events_url":"https://api.github.com/repos/codeborne/selenide/issues/830/events","html_url":"https://github.com/codeborne/selenide/pull/830","id":366530359,"node_id":"MDExOlB1bGxSZXF1ZXN0MjIwMTgxMTE2","number":830,"title":"Fix minor code smell issues reported by Sonar","user":{"login":"plagov","id":16601852,"node_id":"MDQ6VXNlcjE2NjAxODUy","avatar_url":"https://avatars1.githubusercontent.com/u/16601852?v=4","gravatar_id":"","url":"https://api.github.com/users/plagov","html_url":"https://github.com/plagov","followers_url":"https://api.github.com/users/plagov/followers","following_url":"https://api.github.com/users/plagov/following{/other_user}","gists_url":"https://api.github.com/users/plagov/gists{/gist_id}","starred_url":"https://api.github.com/users/plagov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/plagov/subscriptions","organizations_url":"https://api.github.com/users/plagov/orgs","repos_url":"https://api.github.com/users/plagov/repos","events_url":"https://api.github.com/users/plagov/events{/privacy}","received_events_url":"https://api.github.com/users/plagov/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-10-03T21:09:57Z","updated_at":"2018-10-03T21:12:30Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/codeborne/selenide/pulls/830","html_url":"https://github.com/codeborne/selenide/pull/830","diff_url":"https://github.com/codeborne/selenide/pull/830.diff","patch_url":"https://github.com/codeborne/selenide/pull/830.patch"},"body":"* remove catch for `NoSuchFrameException` because it is a subclass of\r\n`WebDriverException`\r\n* make `isFirefox62Bug()` method static since it's a private method"},"comment":{"url":"https://api.github.com/repos/codeborne/selenide/issues/comments/426805456","html_url":"https://github.com/codeborne/selenide/pull/830#issuecomment-426805456","issue_url":"https://api.github.com/repos/codeborne/selenide/issues/830","id":426805456,"node_id":"MDEyOklzc3VlQ29tbWVudDQyNjgwNTQ1Ng==","user":{"login":"plagov","id":16601852,"node_id":"MDQ6VXNlcjE2NjAxODUy","avatar_url":"https://avatars1.githubusercontent.com/u/16601852?v=4","gravatar_id":"","url":"https://api.github.com/users/plagov","html_url":"https://github.com/plagov","followers_url":"https://api.github.com/users/plagov/followers","following_url":"https://api.github.com/users/plagov/following{/other_user}","gists_url":"https://api.github.com/users/plagov/gists{/gist_id}","starred_url":"https://api.github.com/users/plagov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/plagov/subscriptions","organizations_url":"https://api.github.com/users/plagov/orgs","repos_url":"https://api.github.com/users/plagov/repos","events_url":"https://api.github.com/users/plagov/events{/privacy}","received_events_url":"https://api.github.com/users/plagov/received_events","type":"User","site_admin":false},"created_at":"2018-10-03T21:12:30Z","updated_at":"2018-10-03T21:12:30Z","author_association":"NONE","body":"Sonar also reports three code smells with major status to replace with an if/else structure. So was it done on purpose and these warnings from Sonar can be ignored? Or it would be better to replace these ternary operators with if/else?"}}
{"actor":{"display_login":"plagov"}}