{"action":"opened","issue":{"url":"https://api.github.com/repos/ta4j/ta4j/issues/348","repository_url":"https://api.github.com/repos/ta4j/ta4j","labels_url":"https://api.github.com/repos/ta4j/ta4j/issues/348/labels{/name}","comments_url":"https://api.github.com/repos/ta4j/ta4j/issues/348/comments","events_url":"https://api.github.com/repos/ta4j/ta4j/issues/348/events","html_url":"https://github.com/ta4j/ta4j/issues/348","id":369632276,"node_id":"MDU6SXNzdWUzNjk2MzIyNzY=","number":348,"title":"Lots of NullPointerExceptions in package analysis.criteria","user":{"login":"milczarekIT","id":5555560,"node_id":"MDQ6VXNlcjU1NTU1NjA=","avatar_url":"https://avatars0.githubusercontent.com/u/5555560?v=4","gravatar_id":"","url":"https://api.github.com/users/milczarekIT","html_url":"https://github.com/milczarekIT","followers_url":"https://api.github.com/users/milczarekIT/followers","following_url":"https://api.github.com/users/milczarekIT/following{/other_user}","gists_url":"https://api.github.com/users/milczarekIT/gists{/gist_id}","starred_url":"https://api.github.com/users/milczarekIT/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/milczarekIT/subscriptions","organizations_url":"https://api.github.com/users/milczarekIT/orgs","repos_url":"https://api.github.com/users/milczarekIT/repos","events_url":"https://api.github.com/users/milczarekIT/events{/privacy}","received_events_url":"https://api.github.com/users/milczarekIT/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-10-12T16:58:19Z","updated_at":"2018-10-12T16:58:19Z","closed_at":null,"author_association":"NONE","body":"Some time ago I raised issue: https://github.com/ta4j/ta4j/issues/345 (TotalProfit2Criterion.calculate(TimeSeries, Trade) throws NPE for open trade) which is already fixed and merged to `develop` branch.\r\nUnfortunately I found more NullPointerExceptions in classes added in last time. Here is the list of classes affected too:\r\n\r\n- TotalLossCriterion\r\n- ProfitLossPercentageCriterion\r\n- ProfitLossCriterion\r\n- NumberOfWinningTradesCriterion\r\n- NumberOfLosingTradesCriterion\r\n- NumberOfBreakEvenTradesCriterion\r\n- NumberOfBarsCriterion\r\n- AverageProfitableTradesCriterion\r\n\r\nGood news is that bugs was detected before release.\r\n\r\nI am going to fix these NPEs"}}
{"actor":{"display_login":"milczarekIT"}}