{"comment":{"url":"https://api.github.com/repos/Fernando-Jack/openmrs-core/comments/10556028","html_url":"https://github.com/Fernando-Jack/openmrs-core/commit/60199c46ec86b86ceb850a6f56ca55fb6cdec860#commitcomment-10556028","id":10556028,"user":{"login":"Fernando-Jack","id":2591259,"avatar_url":"https://avatars.githubusercontent.com/u/2591259?v=3","gravatar_id":"","url":"https://api.github.com/users/Fernando-Jack","html_url":"https://github.com/Fernando-Jack","followers_url":"https://api.github.com/users/Fernando-Jack/followers","following_url":"https://api.github.com/users/Fernando-Jack/following{/other_user}","gists_url":"https://api.github.com/users/Fernando-Jack/gists{/gist_id}","starred_url":"https://api.github.com/users/Fernando-Jack/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Fernando-Jack/subscriptions","organizations_url":"https://api.github.com/users/Fernando-Jack/orgs","repos_url":"https://api.github.com/users/Fernando-Jack/repos","events_url":"https://api.github.com/users/Fernando-Jack/events{/privacy}","received_events_url":"https://api.github.com/users/Fernando-Jack/received_events","type":"User","site_admin":false},"position":78,"line":350,"path":"api/src/test/java/org/openmrs/api/FormServiceTest.java","commit_id":"60199c46ec86b86ceb850a6f56ca55fb6cdec860","created_at":"2015-04-02T22:53:29Z","updated_at":"2015-04-02T22:53:29Z","body":"I was trying to improve the if-else-condition meanging and readbility. I agree that creating a variable will makes the code more verbose. But I will acrescent more meaning. Just \"true\" or \"false\" doesn't have business meaning associated with it. But the variable isMatched gives me the ideia that something has compatibility. But it is just something I usually do, if you think its better I change it. No problems. Thanks for the review"}}