{"action":"created","issue":{"url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2632","labels_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2632/labels{/name}","comments_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2632/comments","events_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2632/events","html_url":"https://github.com/checkstyle/checkstyle/pull/2632","id":118084409,"number":2632,"title":"minor: fixed improper put that should be get","user":{"login":"rnveach","id":5427943,"avatar_url":"https://avatars.githubusercontent.com/u/5427943?v=3","gravatar_id":"","url":"https://api.github.com/users/rnveach","html_url":"https://github.com/rnveach","followers_url":"https://api.github.com/users/rnveach/followers","following_url":"https://api.github.com/users/rnveach/following{/other_user}","gists_url":"https://api.github.com/users/rnveach/gists{/gist_id}","starred_url":"https://api.github.com/users/rnveach/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rnveach/subscriptions","organizations_url":"https://api.github.com/users/rnveach/orgs","repos_url":"https://api.github.com/users/rnveach/repos","events_url":"https://api.github.com/users/rnveach/events{/privacy}","received_events_url":"https://api.github.com/users/rnveach/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-11-20T17:10:48Z","updated_at":"2015-11-20T17:26:00Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/checkstyle/checkstyle/pulls/2632","html_url":"https://github.com/checkstyle/checkstyle/pull/2632","diff_url":"https://github.com/checkstyle/checkstyle/pull/2632.diff","patch_url":"https://github.com/checkstyle/checkstyle/pull/2632.patch"},"body":"This line should be a 'get' because regardless of the contents of `current` the next statements will do another put, replacing the original put contents. This is similar to assigning a value to a variable, and then re-assigning the same variable with the same or similar value."},"comment":{"url":"https://api.github.com/repos/checkstyle/checkstyle/issues/comments/158466047","html_url":"https://github.com/checkstyle/checkstyle/pull/2632#issuecomment-158466047","issue_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2632","id":158466047,"user":{"login":"rnveach","id":5427943,"avatar_url":"https://avatars.githubusercontent.com/u/5427943?v=3","gravatar_id":"","url":"https://api.github.com/users/rnveach","html_url":"https://github.com/rnveach","followers_url":"https://api.github.com/users/rnveach/followers","following_url":"https://api.github.com/users/rnveach/following{/other_user}","gists_url":"https://api.github.com/users/rnveach/gists{/gist_id}","starred_url":"https://api.github.com/users/rnveach/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rnveach/subscriptions","organizations_url":"https://api.github.com/users/rnveach/orgs","repos_url":"https://api.github.com/users/rnveach/repos","events_url":"https://api.github.com/users/rnveach/events{/privacy}","received_events_url":"https://api.github.com/users/rnveach/received_events","type":"User","site_admin":false},"created_at":"2015-11-20T17:26:00Z","updated_at":"2015-11-20T17:26:00Z","body":"@mkordas \r\nWe still would need to do a get if it does contain the key to concatenate it. \r\nAlso 'contains' pretty much is calling a 'get' anyways, so I think it is better to call one get instead of 2."}}