{"action":"created","comment":{"url":"https://api.github.com/repos/codice/ddf/pulls/comments/69616403","id":69616403,"diff_hunk":"@@ -214,6 +216,11 @@ private Metacard transformPdf(String id, PDDocument pdfDocument, String contentI\n         extractPdfMetadata(pdfDocument, metacard);\n \n         metacard.setThumbnail(generatePdfThumbnail(pdfDocument));\n+\n+        String wktFromPDF = GEO_PDF_PARSER.getWktFromPDF(pdfDocument);\n+        if (wktFromPDF != null) {\n+            metacard.setLocation(wktFromPDF);","path":"catalog/transformer/catalog-transformer-pdf/src/main/java/ddf/catalog/transformer/input/pdf/PdfInputTransformer.java","position":16,"original_position":16,"commit_id":"acf5e2b272a9e8e6bc2fd354ebab08315da93106","original_commit_id":"acf5e2b272a9e8e6bc2fd354ebab08315da93106","user":{"login":"glenhein","id":17837152,"avatar_url":"https://avatars.githubusercontent.com/u/17837152?v=3","gravatar_id":"","url":"https://api.github.com/users/glenhein","html_url":"https://github.com/glenhein","followers_url":"https://api.github.com/users/glenhein/followers","following_url":"https://api.github.com/users/glenhein/following{/other_user}","gists_url":"https://api.github.com/users/glenhein/gists{/gist_id}","starred_url":"https://api.github.com/users/glenhein/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/glenhein/subscriptions","organizations_url":"https://api.github.com/users/glenhein/orgs","repos_url":"https://api.github.com/users/glenhein/repos","events_url":"https://api.github.com/users/glenhein/events{/privacy}","received_events_url":"https://api.github.com/users/glenhein/received_events","type":"User","site_admin":false},"body":"What if extractPdfMetadata already set a location? Should you do a Geography::union instead of just overwriting the value?","created_at":"2016-07-05T18:44:26Z","updated_at":"2016-07-05T18:44:26Z","html_url":"https://github.com/codice/ddf/pull/967#discussion_r69616403","pull_request_url":"https://api.github.com/repos/codice/ddf/pulls/967","_links":{"self":{"href":"https://api.github.com/repos/codice/ddf/pulls/comments/69616403"},"html":{"href":"https://github.com/codice/ddf/pull/967#discussion_r69616403"},"pull_request":{"href":"https://api.github.com/repos/codice/ddf/pulls/967"}}},"pull_request":{"url":"https://api.github.com/repos/codice/ddf/pulls/967","id":76299536,"html_url":"https://github.com/codice/ddf/pull/967","diff_url":"https://github.com/codice/ddf/pull/967.diff","patch_url":"https://github.com/codice/ddf/pull/967.patch","issue_url":"https://api.github.com/repos/codice/ddf/issues/967","number":967,"state":"open","locked":false,"title":"DDF-2260 Added support for GeoPDF WGS84 location parsing","user":{"login":"bdeining","id":6551038,"avatar_url":"https://avatars.githubusercontent.com/u/6551038?v=3","gravatar_id":"","url":"https://api.github.com/users/bdeining","html_url":"https://github.com/bdeining","followers_url":"https://api.github.com/users/bdeining/followers","following_url":"https://api.github.com/users/bdeining/following{/other_user}","gists_url":"https://api.github.com/users/bdeining/gists{/gist_id}","starred_url":"https://api.github.com/users/bdeining/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bdeining/subscriptions","organizations_url":"https://api.github.com/users/bdeining/orgs","repos_url":"https://api.github.com/users/bdeining/repos","events_url":"https://api.github.com/users/bdeining/events{/privacy}","received_events_url":"https://api.github.com/users/bdeining/received_events","type":"User","site_admin":false},"body":"#### What does this PR do?\r\nThis PR adds support to parse WGS84 locations from GeoPDF files when present.\r\n#### Who is reviewing it (please choose AT LEAST two reviewers that need to approve the PR before it can get merged; if a component team is listed, at least one of its members needs to approve)?\r\n@glenhein @jrnorth \r\n#### Choose 2 committers to review/merge the PR (please choose ONLY two committers from below, delete the rest).\r\n@jlcsmith\r\n@kcwire \r\n#### How should this be tested?\r\nInstall DDF, Ingest GeoPDFs that contain `GEOGRAPHIC` projection types, and observe the location is properly set via the Standard Search UI (contact me for example data)\r\n#### Any background context you want to provide?\r\n#### What are the relevant tickets?\r\nhttps://codice.atlassian.net/browse/DDF-2260\r\n#### Screenshots (if appropriate)\r\n#### Checklist:\r\n- [ ] Documentation Updated\r\n- [X] Update / Add Unit Tests\r\n- [ ] Update / Add Integration Tests\r\n\r\n","created_at":"2016-07-05T16:01:01Z","updated_at":"2016-07-05T18:44:26Z","closed_at":null,"merged_at":null,"merge_commit_sha":"ccc3b326c7a625b5570ab22ec650e566f5236ca3","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/codice/ddf/pulls/967/commits","review_comments_url":"https://api.github.com/repos/codice/ddf/pulls/967/comments","review_comment_url":"https://api.github.com/repos/codice/ddf/pulls/comments{/number}","comments_url":"https://api.github.com/repos/codice/ddf/issues/967/comments","statuses_url":"https://api.github.com/repos/codice/ddf/statuses/acf5e2b272a9e8e6bc2fd354ebab08315da93106","head":{"label":"bdeining:DDF-2260","ref":"DDF-2260","sha":"acf5e2b272a9e8e6bc2fd354ebab08315da93106","user":{"login":"bdeining","id":6551038,"avatar_url":"https://avatars.githubusercontent.com/u/6551038?v=3","gravatar_id":"","url":"https://api.github.com/users/bdeining","html_url":"https://github.com/bdeining","followers_url":"https://api.github.com/users/bdeining/followers","following_url":"https://api.github.com/users/bdeining/following{/other_user}","gists_url":"https://api.github.com/users/bdeining/gists{/gist_id}","starred_url":"https://api.github.com/users/bdeining/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bdeining/subscriptions","organizations_url":"https://api.github.com/users/bdeining/orgs","repos_url":"https://api.github.com/users/bdeining/repos","events_url":"https://api.github.com/users/bdeining/events{/privacy}","received_events_url":"https://api.github.com/users/bdeining/received_events","type":"User","site_admin":false},"repo":{"id":40979396,"name":"ddf","full_name":"bdeining/ddf","owner":{"login":"bdeining","id":6551038,"avatar_url":"https://avatars.githubusercontent.com/u/6551038?v=3","gravatar_id":"","url":"https://api.github.com/users/bdeining","html_url":"https://github.com/bdeining","followers_url":"https://api.github.com/users/bdeining/followers","following_url":"https://api.github.com/users/bdeining/following{/other_user}","gists_url":"https://api.github.com/users/bdeining/gists{/gist_id}","starred_url":"https://api.github.com/users/bdeining/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bdeining/subscriptions","organizations_url":"https://api.github.com/users/bdeining/orgs","repos_url":"https://api.github.com/users/bdeining/repos","events_url":"https://api.github.com/users/bdeining/events{/privacy}","received_events_url":"https://api.github.com/users/bdeining/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/bdeining/ddf","description":"The Distributed Data Framework - Distribution, SDK, and Test","fork":true,"url":"https://api.github.com/repos/bdeining/ddf","forks_url":"https://api.github.com/repos/bdeining/ddf/forks","keys_url":"https://api.github.com/repos/bdeining/ddf/keys{/key_id}","collaborators_url":"https://api.github.com/repos/bdeining/ddf/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/bdeining/ddf/teams","hooks_url":"https://api.github.com/repos/bdeining/ddf/hooks","issue_events_url":"https://api.github.com/repos/bdeining/ddf/issues/events{/number}","events_url":"https://api.github.com/repos/bdeining/ddf/events","assignees_url":"https://api.github.com/repos/bdeining/ddf/assignees{/user}","branches_url":"https://api.github.com/repos/bdeining/ddf/branches{/branch}","tags_url":"https://api.github.com/repos/bdeining/ddf/tags","blobs_url":"https://api.github.com/repos/bdeining/ddf/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/bdeining/ddf/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/bdeining/ddf/git/refs{/sha}","trees_url":"https://api.github.com/repos/bdeining/ddf/git/trees{/sha}","statuses_url":"https://api.github.com/repos/bdeining/ddf/statuses/{sha}","languages_url":"https://api.github.com/repos/bdeining/ddf/languages","stargazers_url":"https://api.github.com/repos/bdeining/ddf/stargazers","contributors_url":"https://api.github.com/repos/bdeining/ddf/contributors","subscribers_url":"https://api.github.com/repos/bdeining/ddf/subscribers","subscription_url":"https://api.github.com/repos/bdeining/ddf/subscription","commits_url":"https://api.github.com/repos/bdeining/ddf/commits{/sha}","git_commits_url":"https://api.github.com/repos/bdeining/ddf/git/commits{/sha}","comments_url":"https://api.github.com/repos/bdeining/ddf/comments{/number}","issue_comment_url":"https://api.github.com/repos/bdeining/ddf/issues/comments{/number}","contents_url":"https://api.github.com/repos/bdeining/ddf/contents/{+path}","compare_url":"https://api.github.com/repos/bdeining/ddf/compare/{base}...{head}","merges_url":"https://api.github.com/repos/bdeining/ddf/merges","archive_url":"https://api.github.com/repos/bdeining/ddf/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/bdeining/ddf/downloads","issues_url":"https://api.github.com/repos/bdeining/ddf/issues{/number}","pulls_url":"https://api.github.com/repos/bdeining/ddf/pulls{/number}","milestones_url":"https://api.github.com/repos/bdeining/ddf/milestones{/number}","notifications_url":"https://api.github.com/repos/bdeining/ddf/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/bdeining/ddf/labels{/name}","releases_url":"https://api.github.com/repos/bdeining/ddf/releases{/id}","deployments_url":"https://api.github.com/repos/bdeining/ddf/deployments","created_at":"2015-08-18T15:16:49Z","updated_at":"2016-01-11T14:33:01Z","pushed_at":"2016-07-05T15:57:11Z","git_url":"git://github.com/bdeining/ddf.git","ssh_url":"git@github.com:bdeining/ddf.git","clone_url":"https://github.com/bdeining/ddf.git","svn_url":"https://github.com/bdeining/ddf","homepage":"","size":143040,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":false,"has_downloads":true,"has_wiki":false,"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":"codice:master","ref":"master","sha":"e8df8fb1321d21381bc27f4e0b73821854140944","user":{"login":"codice","id":2386734,"avatar_url":"https://avatars.githubusercontent.com/u/2386734?v=3","gravatar_id":"","url":"https://api.github.com/users/codice","html_url":"https://github.com/codice","followers_url":"https://api.github.com/users/codice/followers","following_url":"https://api.github.com/users/codice/following{/other_user}","gists_url":"https://api.github.com/users/codice/gists{/gist_id}","starred_url":"https://api.github.com/users/codice/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codice/subscriptions","organizations_url":"https://api.github.com/users/codice/orgs","repos_url":"https://api.github.com/users/codice/repos","events_url":"https://api.github.com/users/codice/events{/privacy}","received_events_url":"https://api.github.com/users/codice/received_events","type":"Organization","site_admin":false},"repo":{"id":15173563,"name":"ddf","full_name":"codice/ddf","owner":{"login":"codice","id":2386734,"avatar_url":"https://avatars.githubusercontent.com/u/2386734?v=3","gravatar_id":"","url":"https://api.github.com/users/codice","html_url":"https://github.com/codice","followers_url":"https://api.github.com/users/codice/followers","following_url":"https://api.github.com/users/codice/following{/other_user}","gists_url":"https://api.github.com/users/codice/gists{/gist_id}","starred_url":"https://api.github.com/users/codice/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codice/subscriptions","organizations_url":"https://api.github.com/users/codice/orgs","repos_url":"https://api.github.com/users/codice/repos","events_url":"https://api.github.com/users/codice/events{/privacy}","received_events_url":"https://api.github.com/users/codice/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/codice/ddf","description":"Distributed Data Framework - an open source, modular integration framework.","fork":false,"url":"https://api.github.com/repos/codice/ddf","forks_url":"https://api.github.com/repos/codice/ddf/forks","keys_url":"https://api.github.com/repos/codice/ddf/keys{/key_id}","collaborators_url":"https://api.github.com/repos/codice/ddf/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/codice/ddf/teams","hooks_url":"https://api.github.com/repos/codice/ddf/hooks","issue_events_url":"https://api.github.com/repos/codice/ddf/issues/events{/number}","events_url":"https://api.github.com/repos/codice/ddf/events","assignees_url":"https://api.github.com/repos/codice/ddf/assignees{/user}","branches_url":"https://api.github.com/repos/codice/ddf/branches{/branch}","tags_url":"https://api.github.com/repos/codice/ddf/tags","blobs_url":"https://api.github.com/repos/codice/ddf/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/codice/ddf/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/codice/ddf/git/refs{/sha}","trees_url":"https://api.github.com/repos/codice/ddf/git/trees{/sha}","statuses_url":"https://api.github.com/repos/codice/ddf/statuses/{sha}","languages_url":"https://api.github.com/repos/codice/ddf/languages","stargazers_url":"https://api.github.com/repos/codice/ddf/stargazers","contributors_url":"https://api.github.com/repos/codice/ddf/contributors","subscribers_url":"https://api.github.com/repos/codice/ddf/subscribers","subscription_url":"https://api.github.com/repos/codice/ddf/subscription","commits_url":"https://api.github.com/repos/codice/ddf/commits{/sha}","git_commits_url":"https://api.github.com/repos/codice/ddf/git/commits{/sha}","comments_url":"https://api.github.com/repos/codice/ddf/comments{/number}","issue_comment_url":"https://api.github.com/repos/codice/ddf/issues/comments{/number}","contents_url":"https://api.github.com/repos/codice/ddf/contents/{+path}","compare_url":"https://api.github.com/repos/codice/ddf/compare/{base}...{head}","merges_url":"https://api.github.com/repos/codice/ddf/merges","archive_url":"https://api.github.com/repos/codice/ddf/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/codice/ddf/downloads","issues_url":"https://api.github.com/repos/codice/ddf/issues{/number}","pulls_url":"https://api.github.com/repos/codice/ddf/pulls{/number}","milestones_url":"https://api.github.com/repos/codice/ddf/milestones{/number}","notifications_url":"https://api.github.com/repos/codice/ddf/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/codice/ddf/labels{/name}","releases_url":"https://api.github.com/repos/codice/ddf/releases{/id}","deployments_url":"https://api.github.com/repos/codice/ddf/deployments","created_at":"2013-12-13T20:38:28Z","updated_at":"2016-06-30T23:25:30Z","pushed_at":"2016-07-05T18:29:37Z","git_url":"git://github.com/codice/ddf.git","ssh_url":"git@github.com:codice/ddf.git","clone_url":"https://github.com/codice/ddf.git","svn_url":"https://github.com/codice/ddf","homepage":"http://ddf.codice.org","size":272779,"stargazers_count":39,"watchers_count":39,"language":"Java","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":81,"mirror_url":null,"open_issues_count":16,"forks":81,"open_issues":16,"watchers":39,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/codice/ddf/pulls/967"},"html":{"href":"https://github.com/codice/ddf/pull/967"},"issue":{"href":"https://api.github.com/repos/codice/ddf/issues/967"},"comments":{"href":"https://api.github.com/repos/codice/ddf/issues/967/comments"},"review_comments":{"href":"https://api.github.com/repos/codice/ddf/pulls/967/comments"},"review_comment":{"href":"https://api.github.com/repos/codice/ddf/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/codice/ddf/pulls/967/commits"},"statuses":{"href":"https://api.github.com/repos/codice/ddf/statuses/acf5e2b272a9e8e6bc2fd354ebab08315da93106"}}}}
{"actor":{"display_login":"glenhein"}}