{"action":"created","comment":{"url":"https://api.github.com/repos/deltegui/fontAPI/pulls/comments/203160392","pull_request_review_id":137996574,"id":203160392,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDIwMzE2MDM5Mg==","diff_hunk":"@@ -0,0 +1,57 @@\n+package com.fontana.api.persistence.fountains;\n+\n+import java.util.ArrayList;\n+import java.util.List;\n+\n+import org.springframework.stereotype.Component;\n+\n+import com.fontana.api.persistence.RepositoryAdapter;\n+import com.fontana.domain.entities.Fountain;\n+import com.fontana.domain.gateways.FountainRepository;\n+import com.fontana.domain.utilities.geolocalization.GeolocalizationUtilities;\n+\n+@Component\n+public class HibernateFountainRepositoryAdapter extends RepositoryAdapter<Fountain, FountainEntity, HibernateFountainsRepository> implements FountainRepository {\n+\n+\t\n+\n+\t\n+\tpublic HibernateFountainRepositoryAdapter(HibernateFountainsRepository fountainRepository) {\n+\t\tsuper(fountainRepository);\n+\t}\n+\t \n+\tpublic FountainEntity  getEntity() {\n+\t\treturn new FountainEntity();\n+\t}\n+\n+\t@Override\n+\tpublic List<Fountain> findAllByLatitudeAndLongitudeInRectangleKm(Double latitude, Double longitude, Double maxDistanceKm) {","path":"src/main/java/com/fontana/api/persistence/fountains/HibernateFountainRepositoryAdapter.java","position":28,"original_position":28,"commit_id":"c33721954a991098cb1e9e87353948e9c04ad8d2","original_commit_id":"c33721954a991098cb1e9e87353948e9c04ad8d2","user":{"login":"deltegui","id":13512924,"node_id":"MDQ6VXNlcjEzNTEyOTI0","avatar_url":"https://avatars0.githubusercontent.com/u/13512924?v=4","gravatar_id":"","url":"https://api.github.com/users/deltegui","html_url":"https://github.com/deltegui","followers_url":"https://api.github.com/users/deltegui/followers","following_url":"https://api.github.com/users/deltegui/following{/other_user}","gists_url":"https://api.github.com/users/deltegui/gists{/gist_id}","starred_url":"https://api.github.com/users/deltegui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/deltegui/subscriptions","organizations_url":"https://api.github.com/users/deltegui/orgs","repos_url":"https://api.github.com/users/deltegui/repos","events_url":"https://api.github.com/users/deltegui/events{/privacy}","received_events_url":"https://api.github.com/users/deltegui/received_events","type":"User","site_admin":false},"body":"Yo lo que haría sería crear objetos de negocio nuevos, como coordenada y área y asignarle responsabilidades a ellos. También vale para los métodos estáticos de GeolocalizationUtilities","created_at":"2018-07-17T20:06:14Z","updated_at":"2018-07-17T20:06:14Z","html_url":"https://github.com/deltegui/fontAPI/pull/5#discussion_r203160392","pull_request_url":"https://api.github.com/repos/deltegui/fontAPI/pulls/5","author_association":"OWNER","_links":{"self":{"href":"https://api.github.com/repos/deltegui/fontAPI/pulls/comments/203160392"},"html":{"href":"https://github.com/deltegui/fontAPI/pull/5#discussion_r203160392"},"pull_request":{"href":"https://api.github.com/repos/deltegui/fontAPI/pulls/5"}},"in_reply_to_id":202950721},"pull_request":{"url":"https://api.github.com/repos/deltegui/fontAPI/pulls/5","id":201748929,"node_id":"MDExOlB1bGxSZXF1ZXN0MjAxNzQ4OTI5","html_url":"https://github.com/deltegui/fontAPI/pull/5","diff_url":"https://github.com/deltegui/fontAPI/pull/5.diff","patch_url":"https://github.com/deltegui/fontAPI/pull/5.patch","issue_url":"https://api.github.com/repos/deltegui/fontAPI/issues/5","number":5,"state":"open","locked":false,"title":"Add basic fountains","user":{"login":"ivanchi88","id":15201222,"node_id":"MDQ6VXNlcjE1MjAxMjIy","avatar_url":"https://avatars0.githubusercontent.com/u/15201222?v=4","gravatar_id":"","url":"https://api.github.com/users/ivanchi88","html_url":"https://github.com/ivanchi88","followers_url":"https://api.github.com/users/ivanchi88/followers","following_url":"https://api.github.com/users/ivanchi88/following{/other_user}","gists_url":"https://api.github.com/users/ivanchi88/gists{/gist_id}","starred_url":"https://api.github.com/users/ivanchi88/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ivanchi88/subscriptions","organizations_url":"https://api.github.com/users/ivanchi88/orgs","repos_url":"https://api.github.com/users/ivanchi88/repos","events_url":"https://api.github.com/users/ivanchi88/events{/privacy}","received_events_url":"https://api.github.com/users/ivanchi88/received_events","type":"User","site_admin":false},"body":"","created_at":"2018-07-16T20:19:33Z","updated_at":"2018-07-17T20:06:14Z","closed_at":null,"merged_at":null,"merge_commit_sha":"93cd8b97baf8dd894ca565743ff40f1024a07161","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[{"id":981939019,"node_id":"MDU6TGFiZWw5ODE5MzkwMTk=","url":"https://api.github.com/repos/deltegui/fontAPI/labels/enhancement","name":"enhancement","color":"a2eeef","default":true}],"milestone":null,"commits_url":"https://api.github.com/repos/deltegui/fontAPI/pulls/5/commits","review_comments_url":"https://api.github.com/repos/deltegui/fontAPI/pulls/5/comments","review_comment_url":"https://api.github.com/repos/deltegui/fontAPI/pulls/comments{/number}","comments_url":"https://api.github.com/repos/deltegui/fontAPI/issues/5/comments","statuses_url":"https://api.github.com/repos/deltegui/fontAPI/statuses/c33721954a991098cb1e9e87353948e9c04ad8d2","head":{"label":"deltegui:add-basic-fountains","ref":"add-basic-fountains","sha":"c33721954a991098cb1e9e87353948e9c04ad8d2","user":{"login":"deltegui","id":13512924,"node_id":"MDQ6VXNlcjEzNTEyOTI0","avatar_url":"https://avatars0.githubusercontent.com/u/13512924?v=4","gravatar_id":"","url":"https://api.github.com/users/deltegui","html_url":"https://github.com/deltegui","followers_url":"https://api.github.com/users/deltegui/followers","following_url":"https://api.github.com/users/deltegui/following{/other_user}","gists_url":"https://api.github.com/users/deltegui/gists{/gist_id}","starred_url":"https://api.github.com/users/deltegui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/deltegui/subscriptions","organizations_url":"https://api.github.com/users/deltegui/orgs","repos_url":"https://api.github.com/users/deltegui/repos","events_url":"https://api.github.com/users/deltegui/events{/privacy}","received_events_url":"https://api.github.com/users/deltegui/received_events","type":"User","site_admin":false},"repo":{"id":139492029,"node_id":"MDEwOlJlcG9zaXRvcnkxMzk0OTIwMjk=","name":"fontAPI","full_name":"deltegui/fontAPI","owner":{"login":"deltegui","id":13512924,"node_id":"MDQ6VXNlcjEzNTEyOTI0","avatar_url":"https://avatars0.githubusercontent.com/u/13512924?v=4","gravatar_id":"","url":"https://api.github.com/users/deltegui","html_url":"https://github.com/deltegui","followers_url":"https://api.github.com/users/deltegui/followers","following_url":"https://api.github.com/users/deltegui/following{/other_user}","gists_url":"https://api.github.com/users/deltegui/gists{/gist_id}","starred_url":"https://api.github.com/users/deltegui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/deltegui/subscriptions","organizations_url":"https://api.github.com/users/deltegui/orgs","repos_url":"https://api.github.com/users/deltegui/repos","events_url":"https://api.github.com/users/deltegui/events{/privacy}","received_events_url":"https://api.github.com/users/deltegui/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/deltegui/fontAPI","description":"Fontana API","fork":false,"url":"https://api.github.com/repos/deltegui/fontAPI","forks_url":"https://api.github.com/repos/deltegui/fontAPI/forks","keys_url":"https://api.github.com/repos/deltegui/fontAPI/keys{/key_id}","collaborators_url":"https://api.github.com/repos/deltegui/fontAPI/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/deltegui/fontAPI/teams","hooks_url":"https://api.github.com/repos/deltegui/fontAPI/hooks","issue_events_url":"https://api.github.com/repos/deltegui/fontAPI/issues/events{/number}","events_url":"https://api.github.com/repos/deltegui/fontAPI/events","assignees_url":"https://api.github.com/repos/deltegui/fontAPI/assignees{/user}","branches_url":"https://api.github.com/repos/deltegui/fontAPI/branches{/branch}","tags_url":"https://api.github.com/repos/deltegui/fontAPI/tags","blobs_url":"https://api.github.com/repos/deltegui/fontAPI/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/deltegui/fontAPI/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/deltegui/fontAPI/git/refs{/sha}","trees_url":"https://api.github.com/repos/deltegui/fontAPI/git/trees{/sha}","statuses_url":"https://api.github.com/repos/deltegui/fontAPI/statuses/{sha}","languages_url":"https://api.github.com/repos/deltegui/fontAPI/languages","stargazers_url":"https://api.github.com/repos/deltegui/fontAPI/stargazers","contributors_url":"https://api.github.com/repos/deltegui/fontAPI/contributors","subscribers_url":"https://api.github.com/repos/deltegui/fontAPI/subscribers","subscription_url":"https://api.github.com/repos/deltegui/fontAPI/subscription","commits_url":"https://api.github.com/repos/deltegui/fontAPI/commits{/sha}","git_commits_url":"https://api.github.com/repos/deltegui/fontAPI/git/commits{/sha}","comments_url":"https://api.github.com/repos/deltegui/fontAPI/comments{/number}","issue_comment_url":"https://api.github.com/repos/deltegui/fontAPI/issues/comments{/number}","contents_url":"https://api.github.com/repos/deltegui/fontAPI/contents/{+path}","compare_url":"https://api.github.com/repos/deltegui/fontAPI/compare/{base}...{head}","merges_url":"https://api.github.com/repos/deltegui/fontAPI/merges","archive_url":"https://api.github.com/repos/deltegui/fontAPI/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/deltegui/fontAPI/downloads","issues_url":"https://api.github.com/repos/deltegui/fontAPI/issues{/number}","pulls_url":"https://api.github.com/repos/deltegui/fontAPI/pulls{/number}","milestones_url":"https://api.github.com/repos/deltegui/fontAPI/milestones{/number}","notifications_url":"https://api.github.com/repos/deltegui/fontAPI/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/deltegui/fontAPI/labels{/name}","releases_url":"https://api.github.com/repos/deltegui/fontAPI/releases{/id}","deployments_url":"https://api.github.com/repos/deltegui/fontAPI/deployments","created_at":"2018-07-02T20:38:50Z","updated_at":"2018-07-15T19:44:24Z","pushed_at":"2018-07-17T17:28:28Z","git_url":"git://github.com/deltegui/fontAPI.git","ssh_url":"git@github.com:deltegui/fontAPI.git","clone_url":"https://github.com/deltegui/fontAPI.git","svn_url":"https://github.com/deltegui/fontAPI","homepage":null,"size":190205,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":1,"license":{"key":"bsd-3-clause","name":"BSD 3-Clause \"New\" or \"Revised\" License","spdx_id":"BSD-3-Clause","url":"https://api.github.com/licenses/bsd-3-clause","node_id":"MDc6TGljZW5zZTU="},"forks":0,"open_issues":1,"watchers":0,"default_branch":"master"}},"base":{"label":"deltegui:dev","ref":"dev","sha":"8a058d86bf35cb16a30458fa5880983632d9e833","user":{"login":"deltegui","id":13512924,"node_id":"MDQ6VXNlcjEzNTEyOTI0","avatar_url":"https://avatars0.githubusercontent.com/u/13512924?v=4","gravatar_id":"","url":"https://api.github.com/users/deltegui","html_url":"https://github.com/deltegui","followers_url":"https://api.github.com/users/deltegui/followers","following_url":"https://api.github.com/users/deltegui/following{/other_user}","gists_url":"https://api.github.com/users/deltegui/gists{/gist_id}","starred_url":"https://api.github.com/users/deltegui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/deltegui/subscriptions","organizations_url":"https://api.github.com/users/deltegui/orgs","repos_url":"https://api.github.com/users/deltegui/repos","events_url":"https://api.github.com/users/deltegui/events{/privacy}","received_events_url":"https://api.github.com/users/deltegui/received_events","type":"User","site_admin":false},"repo":{"id":139492029,"node_id":"MDEwOlJlcG9zaXRvcnkxMzk0OTIwMjk=","name":"fontAPI","full_name":"deltegui/fontAPI","owner":{"login":"deltegui","id":13512924,"node_id":"MDQ6VXNlcjEzNTEyOTI0","avatar_url":"https://avatars0.githubusercontent.com/u/13512924?v=4","gravatar_id":"","url":"https://api.github.com/users/deltegui","html_url":"https://github.com/deltegui","followers_url":"https://api.github.com/users/deltegui/followers","following_url":"https://api.github.com/users/deltegui/following{/other_user}","gists_url":"https://api.github.com/users/deltegui/gists{/gist_id}","starred_url":"https://api.github.com/users/deltegui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/deltegui/subscriptions","organizations_url":"https://api.github.com/users/deltegui/orgs","repos_url":"https://api.github.com/users/deltegui/repos","events_url":"https://api.github.com/users/deltegui/events{/privacy}","received_events_url":"https://api.github.com/users/deltegui/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/deltegui/fontAPI","description":"Fontana API","fork":false,"url":"https://api.github.com/repos/deltegui/fontAPI","forks_url":"https://api.github.com/repos/deltegui/fontAPI/forks","keys_url":"https://api.github.com/repos/deltegui/fontAPI/keys{/key_id}","collaborators_url":"https://api.github.com/repos/deltegui/fontAPI/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/deltegui/fontAPI/teams","hooks_url":"https://api.github.com/repos/deltegui/fontAPI/hooks","issue_events_url":"https://api.github.com/repos/deltegui/fontAPI/issues/events{/number}","events_url":"https://api.github.com/repos/deltegui/fontAPI/events","assignees_url":"https://api.github.com/repos/deltegui/fontAPI/assignees{/user}","branches_url":"https://api.github.com/repos/deltegui/fontAPI/branches{/branch}","tags_url":"https://api.github.com/repos/deltegui/fontAPI/tags","blobs_url":"https://api.github.com/repos/deltegui/fontAPI/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/deltegui/fontAPI/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/deltegui/fontAPI/git/refs{/sha}","trees_url":"https://api.github.com/repos/deltegui/fontAPI/git/trees{/sha}","statuses_url":"https://api.github.com/repos/deltegui/fontAPI/statuses/{sha}","languages_url":"https://api.github.com/repos/deltegui/fontAPI/languages","stargazers_url":"https://api.github.com/repos/deltegui/fontAPI/stargazers","contributors_url":"https://api.github.com/repos/deltegui/fontAPI/contributors","subscribers_url":"https://api.github.com/repos/deltegui/fontAPI/subscribers","subscription_url":"https://api.github.com/repos/deltegui/fontAPI/subscription","commits_url":"https://api.github.com/repos/deltegui/fontAPI/commits{/sha}","git_commits_url":"https://api.github.com/repos/deltegui/fontAPI/git/commits{/sha}","comments_url":"https://api.github.com/repos/deltegui/fontAPI/comments{/number}","issue_comment_url":"https://api.github.com/repos/deltegui/fontAPI/issues/comments{/number}","contents_url":"https://api.github.com/repos/deltegui/fontAPI/contents/{+path}","compare_url":"https://api.github.com/repos/deltegui/fontAPI/compare/{base}...{head}","merges_url":"https://api.github.com/repos/deltegui/fontAPI/merges","archive_url":"https://api.github.com/repos/deltegui/fontAPI/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/deltegui/fontAPI/downloads","issues_url":"https://api.github.com/repos/deltegui/fontAPI/issues{/number}","pulls_url":"https://api.github.com/repos/deltegui/fontAPI/pulls{/number}","milestones_url":"https://api.github.com/repos/deltegui/fontAPI/milestones{/number}","notifications_url":"https://api.github.com/repos/deltegui/fontAPI/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/deltegui/fontAPI/labels{/name}","releases_url":"https://api.github.com/repos/deltegui/fontAPI/releases{/id}","deployments_url":"https://api.github.com/repos/deltegui/fontAPI/deployments","created_at":"2018-07-02T20:38:50Z","updated_at":"2018-07-15T19:44:24Z","pushed_at":"2018-07-17T17:28:28Z","git_url":"git://github.com/deltegui/fontAPI.git","ssh_url":"git@github.com:deltegui/fontAPI.git","clone_url":"https://github.com/deltegui/fontAPI.git","svn_url":"https://github.com/deltegui/fontAPI","homepage":null,"size":190205,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":1,"license":{"key":"bsd-3-clause","name":"BSD 3-Clause \"New\" or \"Revised\" License","spdx_id":"BSD-3-Clause","url":"https://api.github.com/licenses/bsd-3-clause","node_id":"MDc6TGljZW5zZTU="},"forks":0,"open_issues":1,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/deltegui/fontAPI/pulls/5"},"html":{"href":"https://github.com/deltegui/fontAPI/pull/5"},"issue":{"href":"https://api.github.com/repos/deltegui/fontAPI/issues/5"},"comments":{"href":"https://api.github.com/repos/deltegui/fontAPI/issues/5/comments"},"review_comments":{"href":"https://api.github.com/repos/deltegui/fontAPI/pulls/5/comments"},"review_comment":{"href":"https://api.github.com/repos/deltegui/fontAPI/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/deltegui/fontAPI/pulls/5/commits"},"statuses":{"href":"https://api.github.com/repos/deltegui/fontAPI/statuses/c33721954a991098cb1e9e87353948e9c04ad8d2"}},"author_association":"COLLABORATOR"}}
{"actor":{"display_login":"deltegui"}}