{"action":"created","issue":{"url":"https://api.github.com/repos/graphql-java/graphql-java/issues/164","repository_url":"https://api.github.com/repos/graphql-java/graphql-java","labels_url":"https://api.github.com/repos/graphql-java/graphql-java/issues/164/labels{/name}","comments_url":"https://api.github.com/repos/graphql-java/graphql-java/issues/164/comments","events_url":"https://api.github.com/repos/graphql-java/graphql-java/issues/164/events","html_url":"https://github.com/graphql-java/graphql-java/pull/164","id":160510395,"number":164,"title":"Fix for #130 - Adds support for Boolean property accessors with 'get' prefix in PropertyDataFetcher","user":{"login":"ayhanap","id":5447639,"avatar_url":"https://avatars.githubusercontent.com/u/5447639?v=3","gravatar_id":"","url":"https://api.github.com/users/ayhanap","html_url":"https://github.com/ayhanap","followers_url":"https://api.github.com/users/ayhanap/followers","following_url":"https://api.github.com/users/ayhanap/following{/other_user}","gists_url":"https://api.github.com/users/ayhanap/gists{/gist_id}","starred_url":"https://api.github.com/users/ayhanap/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ayhanap/subscriptions","organizations_url":"https://api.github.com/users/ayhanap/orgs","repos_url":"https://api.github.com/users/ayhanap/repos","events_url":"https://api.github.com/users/ayhanap/events{/privacy}","received_events_url":"https://api.github.com/users/ayhanap/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/graphql-java/graphql-java/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-06-15T20:00:13Z","updated_at":"2016-08-19T22:48:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/graphql-java/graphql-java/pulls/164","html_url":"https://github.com/graphql-java/graphql-java/pull/164","diff_url":"https://github.com/graphql-java/graphql-java/pull/164.diff","patch_url":"https://github.com/graphql-java/graphql-java/pull/164.patch"},"body":"As mentioned in #130 some languages and frameworks uses \"get\" prefix for `Boolean `types. This was also the case for me. I use Lombok for generating accessors which also uses this convention. Even IDE's generate getters for `Boolean` with \"get\"\r\n\r\nThe prefix 'is' generally used for primitive `boolean`, Lombok has a option to disable this but not for `Boolean`. So I think this functionality should be a part of `PropertyDataFetcher`. Otherwise it will be error prone or some unnecessary configuration/code for these fields."},"comment":{"url":"https://api.github.com/repos/graphql-java/graphql-java/issues/comments/241154039","html_url":"https://github.com/graphql-java/graphql-java/pull/164#issuecomment-241154039","issue_url":"https://api.github.com/repos/graphql-java/graphql-java/issues/164","id":241154039,"user":{"login":"markphilpot","id":46191,"avatar_url":"https://avatars.githubusercontent.com/u/46191?v=3","gravatar_id":"","url":"https://api.github.com/users/markphilpot","html_url":"https://github.com/markphilpot","followers_url":"https://api.github.com/users/markphilpot/followers","following_url":"https://api.github.com/users/markphilpot/following{/other_user}","gists_url":"https://api.github.com/users/markphilpot/gists{/gist_id}","starred_url":"https://api.github.com/users/markphilpot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/markphilpot/subscriptions","organizations_url":"https://api.github.com/users/markphilpot/orgs","repos_url":"https://api.github.com/users/markphilpot/repos","events_url":"https://api.github.com/users/markphilpot/events{/privacy}","received_events_url":"https://api.github.com/users/markphilpot/received_events","type":"User","site_admin":false},"created_at":"2016-08-19T22:48:13Z","updated_at":"2016-08-19T22:48:13Z","body":"Thanks for the PR!  Let me know if the comments make sense."}}
{"actor":{"display_login":"markphilpot"}}