{"action":"created","comment":{"url":"https://api.github.com/repos/cgeo/cgeo/pulls/comments/37921346","id":37921346,"diff_hunk":"@@ -15,27 +17,92 @@\n \n import java.io.File;\n \n+/**\n+ * Represent an Image along with Title and Description.\n+ * Mostly used for representing image in/from logs.\n+ */\n public class Image implements Parcelable {\n-    private final String url;\n-    private final String title;\n-    private final String description;\n+    @NonNull private Uri uri;","path":"main/src/cgeo/geocaching/Image.java","position":21,"original_position":21,"commit_id":"545e875beb08650d9726629e5311ae4170b887ee","original_commit_id":"545e875beb08650d9726629e5311ae4170b887ee","user":{"login":"samueltardieu","id":44656,"avatar_url":"https://avatars.githubusercontent.com/u/44656?v=3","gravatar_id":"","url":"https://api.github.com/users/samueltardieu","html_url":"https://github.com/samueltardieu","followers_url":"https://api.github.com/users/samueltardieu/followers","following_url":"https://api.github.com/users/samueltardieu/following{/other_user}","gists_url":"https://api.github.com/users/samueltardieu/gists{/gist_id}","starred_url":"https://api.github.com/users/samueltardieu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/samueltardieu/subscriptions","organizations_url":"https://api.github.com/users/samueltardieu/orgs","repos_url":"https://api.github.com/users/samueltardieu/repos","events_url":"https://api.github.com/users/samueltardieu/events{/privacy}","received_events_url":"https://api.github.com/users/samueltardieu/received_events","type":"User","site_admin":false},"body":"Can't this field (and the two other ones) be `final` as well?","created_at":"2015-08-25T21:21:43Z","updated_at":"2015-08-25T21:21:43Z","html_url":"https://github.com/cgeo/cgeo/pull/5152#discussion_r37921346","pull_request_url":"https://api.github.com/repos/cgeo/cgeo/pulls/5152","_links":{"self":{"href":"https://api.github.com/repos/cgeo/cgeo/pulls/comments/37921346"},"html":{"href":"https://github.com/cgeo/cgeo/pull/5152#discussion_r37921346"},"pull_request":{"href":"https://api.github.com/repos/cgeo/cgeo/pulls/5152"}}},"pull_request":{"url":"https://api.github.com/repos/cgeo/cgeo/pulls/5152","id":43329209,"html_url":"https://github.com/cgeo/cgeo/pull/5152","diff_url":"https://github.com/cgeo/cgeo/pull/5152.diff","patch_url":"https://github.com/cgeo/cgeo/pull/5152.patch","issue_url":"https://api.github.com/repos/cgeo/cgeo/issues/5152","number":5152,"state":"open","locked":false,"title":"Refactor: use Image class instead of local variables.","user":{"login":"kumy","id":176794,"avatar_url":"https://avatars.githubusercontent.com/u/176794?v=3","gravatar_id":"","url":"https://api.github.com/users/kumy","html_url":"https://github.com/kumy","followers_url":"https://api.github.com/users/kumy/followers","following_url":"https://api.github.com/users/kumy/following{/other_user}","gists_url":"https://api.github.com/users/kumy/gists{/gist_id}","starred_url":"https://api.github.com/users/kumy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kumy/subscriptions","organizations_url":"https://api.github.com/users/kumy/orgs","repos_url":"https://api.github.com/users/kumy/repos","events_url":"https://api.github.com/users/kumy/events{/privacy}","received_events_url":"https://api.github.com/users/kumy/received_events","type":"User","site_admin":false},"body":"Do you agree with such update?\r\n","created_at":"2015-08-25T19:56:55Z","updated_at":"2015-08-25T21:21:43Z","closed_at":null,"merged_at":null,"merge_commit_sha":"21baf796f9b31488f691562018ffcd04f2d5a088","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/cgeo/cgeo/pulls/5152/commits","review_comments_url":"https://api.github.com/repos/cgeo/cgeo/pulls/5152/comments","review_comment_url":"https://api.github.com/repos/cgeo/cgeo/pulls/comments{/number}","comments_url":"https://api.github.com/repos/cgeo/cgeo/issues/5152/comments","statuses_url":"https://api.github.com/repos/cgeo/cgeo/statuses/545e875beb08650d9726629e5311ae4170b887ee","head":{"label":"kumy:refactor-images","ref":"refactor-images","sha":"545e875beb08650d9726629e5311ae4170b887ee","user":{"login":"kumy","id":176794,"avatar_url":"https://avatars.githubusercontent.com/u/176794?v=3","gravatar_id":"","url":"https://api.github.com/users/kumy","html_url":"https://github.com/kumy","followers_url":"https://api.github.com/users/kumy/followers","following_url":"https://api.github.com/users/kumy/following{/other_user}","gists_url":"https://api.github.com/users/kumy/gists{/gist_id}","starred_url":"https://api.github.com/users/kumy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kumy/subscriptions","organizations_url":"https://api.github.com/users/kumy/orgs","repos_url":"https://api.github.com/users/kumy/repos","events_url":"https://api.github.com/users/kumy/events{/privacy}","received_events_url":"https://api.github.com/users/kumy/received_events","type":"User","site_admin":false},"repo":{"id":23391796,"name":"cgeo","full_name":"kumy/cgeo","owner":{"login":"kumy","id":176794,"avatar_url":"https://avatars.githubusercontent.com/u/176794?v=3","gravatar_id":"","url":"https://api.github.com/users/kumy","html_url":"https://github.com/kumy","followers_url":"https://api.github.com/users/kumy/followers","following_url":"https://api.github.com/users/kumy/following{/other_user}","gists_url":"https://api.github.com/users/kumy/gists{/gist_id}","starred_url":"https://api.github.com/users/kumy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kumy/subscriptions","organizations_url":"https://api.github.com/users/kumy/orgs","repos_url":"https://api.github.com/users/kumy/repos","events_url":"https://api.github.com/users/kumy/events{/privacy}","received_events_url":"https://api.github.com/users/kumy/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/kumy/cgeo","description":"c:geo - The powerful Android geocaching app.","fork":true,"url":"https://api.github.com/repos/kumy/cgeo","forks_url":"https://api.github.com/repos/kumy/cgeo/forks","keys_url":"https://api.github.com/repos/kumy/cgeo/keys{/key_id}","collaborators_url":"https://api.github.com/repos/kumy/cgeo/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/kumy/cgeo/teams","hooks_url":"https://api.github.com/repos/kumy/cgeo/hooks","issue_events_url":"https://api.github.com/repos/kumy/cgeo/issues/events{/number}","events_url":"https://api.github.com/repos/kumy/cgeo/events","assignees_url":"https://api.github.com/repos/kumy/cgeo/assignees{/user}","branches_url":"https://api.github.com/repos/kumy/cgeo/branches{/branch}","tags_url":"https://api.github.com/repos/kumy/cgeo/tags","blobs_url":"https://api.github.com/repos/kumy/cgeo/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/kumy/cgeo/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/kumy/cgeo/git/refs{/sha}","trees_url":"https://api.github.com/repos/kumy/cgeo/git/trees{/sha}","statuses_url":"https://api.github.com/repos/kumy/cgeo/statuses/{sha}","languages_url":"https://api.github.com/repos/kumy/cgeo/languages","stargazers_url":"https://api.github.com/repos/kumy/cgeo/stargazers","contributors_url":"https://api.github.com/repos/kumy/cgeo/contributors","subscribers_url":"https://api.github.com/repos/kumy/cgeo/subscribers","subscription_url":"https://api.github.com/repos/kumy/cgeo/subscription","commits_url":"https://api.github.com/repos/kumy/cgeo/commits{/sha}","git_commits_url":"https://api.github.com/repos/kumy/cgeo/git/commits{/sha}","comments_url":"https://api.github.com/repos/kumy/cgeo/comments{/number}","issue_comment_url":"https://api.github.com/repos/kumy/cgeo/issues/comments{/number}","contents_url":"https://api.github.com/repos/kumy/cgeo/contents/{+path}","compare_url":"https://api.github.com/repos/kumy/cgeo/compare/{base}...{head}","merges_url":"https://api.github.com/repos/kumy/cgeo/merges","archive_url":"https://api.github.com/repos/kumy/cgeo/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/kumy/cgeo/downloads","issues_url":"https://api.github.com/repos/kumy/cgeo/issues{/number}","pulls_url":"https://api.github.com/repos/kumy/cgeo/pulls{/number}","milestones_url":"https://api.github.com/repos/kumy/cgeo/milestones{/number}","notifications_url":"https://api.github.com/repos/kumy/cgeo/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/kumy/cgeo/labels{/name}","releases_url":"https://api.github.com/repos/kumy/cgeo/releases{/id}","created_at":"2014-08-27T14:06:37Z","updated_at":"2015-04-22T22:40:20Z","pushed_at":"2015-08-25T19:55:16Z","git_url":"git://github.com/kumy/cgeo.git","ssh_url":"git@github.com:kumy/cgeo.git","clone_url":"https://github.com/kumy/cgeo.git","svn_url":"https://github.com/kumy/cgeo","homepage":"www.cgeo.org","size":184177,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"cgeo:master","ref":"master","sha":"d8a1970976368997314752d7de6360abd3cec47b","user":{"login":"cgeo","id":907088,"avatar_url":"https://avatars.githubusercontent.com/u/907088?v=3","gravatar_id":"","url":"https://api.github.com/users/cgeo","html_url":"https://github.com/cgeo","followers_url":"https://api.github.com/users/cgeo/followers","following_url":"https://api.github.com/users/cgeo/following{/other_user}","gists_url":"https://api.github.com/users/cgeo/gists{/gist_id}","starred_url":"https://api.github.com/users/cgeo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cgeo/subscriptions","organizations_url":"https://api.github.com/users/cgeo/orgs","repos_url":"https://api.github.com/users/cgeo/repos","events_url":"https://api.github.com/users/cgeo/events{/privacy}","received_events_url":"https://api.github.com/users/cgeo/received_events","type":"Organization","site_admin":false},"repo":{"id":2028724,"name":"cgeo","full_name":"cgeo/cgeo","owner":{"login":"cgeo","id":907088,"avatar_url":"https://avatars.githubusercontent.com/u/907088?v=3","gravatar_id":"","url":"https://api.github.com/users/cgeo","html_url":"https://github.com/cgeo","followers_url":"https://api.github.com/users/cgeo/followers","following_url":"https://api.github.com/users/cgeo/following{/other_user}","gists_url":"https://api.github.com/users/cgeo/gists{/gist_id}","starred_url":"https://api.github.com/users/cgeo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cgeo/subscriptions","organizations_url":"https://api.github.com/users/cgeo/orgs","repos_url":"https://api.github.com/users/cgeo/repos","events_url":"https://api.github.com/users/cgeo/events{/privacy}","received_events_url":"https://api.github.com/users/cgeo/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/cgeo/cgeo","description":"c:geo - The powerful Android geocaching app.","fork":false,"url":"https://api.github.com/repos/cgeo/cgeo","forks_url":"https://api.github.com/repos/cgeo/cgeo/forks","keys_url":"https://api.github.com/repos/cgeo/cgeo/keys{/key_id}","collaborators_url":"https://api.github.com/repos/cgeo/cgeo/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/cgeo/cgeo/teams","hooks_url":"https://api.github.com/repos/cgeo/cgeo/hooks","issue_events_url":"https://api.github.com/repos/cgeo/cgeo/issues/events{/number}","events_url":"https://api.github.com/repos/cgeo/cgeo/events","assignees_url":"https://api.github.com/repos/cgeo/cgeo/assignees{/user}","branches_url":"https://api.github.com/repos/cgeo/cgeo/branches{/branch}","tags_url":"https://api.github.com/repos/cgeo/cgeo/tags","blobs_url":"https://api.github.com/repos/cgeo/cgeo/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/cgeo/cgeo/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/cgeo/cgeo/git/refs{/sha}","trees_url":"https://api.github.com/repos/cgeo/cgeo/git/trees{/sha}","statuses_url":"https://api.github.com/repos/cgeo/cgeo/statuses/{sha}","languages_url":"https://api.github.com/repos/cgeo/cgeo/languages","stargazers_url":"https://api.github.com/repos/cgeo/cgeo/stargazers","contributors_url":"https://api.github.com/repos/cgeo/cgeo/contributors","subscribers_url":"https://api.github.com/repos/cgeo/cgeo/subscribers","subscription_url":"https://api.github.com/repos/cgeo/cgeo/subscription","commits_url":"https://api.github.com/repos/cgeo/cgeo/commits{/sha}","git_commits_url":"https://api.github.com/repos/cgeo/cgeo/git/commits{/sha}","comments_url":"https://api.github.com/repos/cgeo/cgeo/comments{/number}","issue_comment_url":"https://api.github.com/repos/cgeo/cgeo/issues/comments{/number}","contents_url":"https://api.github.com/repos/cgeo/cgeo/contents/{+path}","compare_url":"https://api.github.com/repos/cgeo/cgeo/compare/{base}...{head}","merges_url":"https://api.github.com/repos/cgeo/cgeo/merges","archive_url":"https://api.github.com/repos/cgeo/cgeo/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/cgeo/cgeo/downloads","issues_url":"https://api.github.com/repos/cgeo/cgeo/issues{/number}","pulls_url":"https://api.github.com/repos/cgeo/cgeo/pulls{/number}","milestones_url":"https://api.github.com/repos/cgeo/cgeo/milestones{/number}","notifications_url":"https://api.github.com/repos/cgeo/cgeo/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/cgeo/cgeo/labels{/name}","releases_url":"https://api.github.com/repos/cgeo/cgeo/releases{/id}","created_at":"2011-07-11T06:02:47Z","updated_at":"2015-08-23T10:22:02Z","pushed_at":"2015-08-25T20:31:13Z","git_url":"git://github.com/cgeo/cgeo.git","ssh_url":"git@github.com:cgeo/cgeo.git","clone_url":"https://github.com/cgeo/cgeo.git","svn_url":"https://github.com/cgeo/cgeo","homepage":"www.cgeo.org","size":210405,"stargazers_count":616,"watchers_count":616,"language":"Java","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":402,"mirror_url":null,"open_issues_count":462,"forks":402,"open_issues":462,"watchers":616,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/cgeo/cgeo/pulls/5152"},"html":{"href":"https://github.com/cgeo/cgeo/pull/5152"},"issue":{"href":"https://api.github.com/repos/cgeo/cgeo/issues/5152"},"comments":{"href":"https://api.github.com/repos/cgeo/cgeo/issues/5152/comments"},"review_comments":{"href":"https://api.github.com/repos/cgeo/cgeo/pulls/5152/comments"},"review_comment":{"href":"https://api.github.com/repos/cgeo/cgeo/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/cgeo/cgeo/pulls/5152/commits"},"statuses":{"href":"https://api.github.com/repos/cgeo/cgeo/statuses/545e875beb08650d9726629e5311ae4170b887ee"}}}}