{"action":"opened","issue":{"url":"https://api.github.com/repos/luismisanchez/wpimap-login/issues/6","repository_url":"https://api.github.com/repos/luismisanchez/wpimap-login","labels_url":"https://api.github.com/repos/luismisanchez/wpimap-login/issues/6/labels{/name}","comments_url":"https://api.github.com/repos/luismisanchez/wpimap-login/issues/6/comments","events_url":"https://api.github.com/repos/luismisanchez/wpimap-login/issues/6/events","html_url":"https://github.com/luismisanchez/wpimap-login/issues/6","id":169359963,"number":6,"title":"[Insight] exit() and die() functions should be avoided - in wpimap-login/inc/test-imap.php, line 34","user":{"login":"luismisanchez","id":6223023,"avatar_url":"https://avatars.githubusercontent.com/u/6223023?v=3","gravatar_id":"","url":"https://api.github.com/users/luismisanchez","html_url":"https://github.com/luismisanchez","followers_url":"https://api.github.com/users/luismisanchez/followers","following_url":"https://api.github.com/users/luismisanchez/following{/other_user}","gists_url":"https://api.github.com/users/luismisanchez/gists{/gist_id}","starred_url":"https://api.github.com/users/luismisanchez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/luismisanchez/subscriptions","organizations_url":"https://api.github.com/users/luismisanchez/orgs","repos_url":"https://api.github.com/users/luismisanchez/repos","events_url":"https://api.github.com/users/luismisanchez/events{/privacy}","received_events_url":"https://api.github.com/users/luismisanchez/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-08-04T12:14:05Z","updated_at":"2016-08-04T12:14:05Z","closed_at":null,"body":"**[in wpimap-login/inc/test-imap.php, line 34](https://github.com/luismisanchez/wpimap-login/blob/f31fed7b4e740f64721b293c6432e53f39152623/wpimap-login/inc/test-imap.php#L34)**\r\n\r\n> This line stops the execution flow, without explanation. If this is for debug, you should remove it. If this is to deal with an error, use exceptions instead.\r\n\r\n```php\r\n\r\n        }\r\n\r\n        if ($error != \"0\") {\r\n\r\n            die($error);\r\n\r\n        } else {\r\n\r\n            if ( !$server || !$port || !$email || !$password ) {\r\n\r\n```\r\n\r\n---------------------\r\n\r\n_Posted from [SensioLabsInsight](https://insight.sensiolabs.com/projects/d30a7988-ea50-4056-9586-7af37e4f52ce/analyses/1)_"}}
{"actor":{"display_login":"luismisanchez"}}