{"action":"created","issue":{"url":"https://api.github.com/repos/codice/ddf/issues/262","labels_url":"https://api.github.com/repos/codice/ddf/issues/262/labels{/name}","comments_url":"https://api.github.com/repos/codice/ddf/issues/262/comments","events_url":"https://api.github.com/repos/codice/ddf/issues/262/events","html_url":"https://github.com/codice/ddf/pull/262","id":111717917,"number":262,"title":"DDF-1570 Latitude/longitude ordering should be handled more consisten…","user":{"login":"dcruver","id":5922390,"avatar_url":"https://avatars.githubusercontent.com/u/5922390?v=3","gravatar_id":"","url":"https://api.github.com/users/dcruver","html_url":"https://github.com/dcruver","followers_url":"https://api.github.com/users/dcruver/followers","following_url":"https://api.github.com/users/dcruver/following{/other_user}","gists_url":"https://api.github.com/users/dcruver/gists{/gist_id}","starred_url":"https://api.github.com/users/dcruver/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dcruver/subscriptions","organizations_url":"https://api.github.com/users/dcruver/orgs","repos_url":"https://api.github.com/users/dcruver/repos","events_url":"https://api.github.com/users/dcruver/events{/privacy}","received_events_url":"https://api.github.com/users/dcruver/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"wmcnalli","id":4419383,"avatar_url":"https://avatars.githubusercontent.com/u/4419383?v=3","gravatar_id":"","url":"https://api.github.com/users/wmcnalli","html_url":"https://github.com/wmcnalli","followers_url":"https://api.github.com/users/wmcnalli/followers","following_url":"https://api.github.com/users/wmcnalli/following{/other_user}","gists_url":"https://api.github.com/users/wmcnalli/gists{/gist_id}","starred_url":"https://api.github.com/users/wmcnalli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wmcnalli/subscriptions","organizations_url":"https://api.github.com/users/wmcnalli/orgs","repos_url":"https://api.github.com/users/wmcnalli/repos","events_url":"https://api.github.com/users/wmcnalli/events{/privacy}","received_events_url":"https://api.github.com/users/wmcnalli/received_events","type":"User","site_admin":false},"milestone":null,"comments":8,"created_at":"2015-10-15T22:05:55Z","updated_at":"2015-10-19T22:35:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/codice/ddf/pulls/262","html_url":"https://github.com/codice/ddf/pull/262","diff_url":"https://github.com/codice/ddf/pull/262.diff","patch_url":"https://github.com/codice/ddf/pull/262.patch"},"body":"…tly for CSW records\r\n\r\nThe CSW endpoint assumes incoming bounding box values are in lon/lat order and reverses\r\nthem before ingesting. The endpoint should use lat/lon order exclusively, only changing\r\nthe order when the data comes from a federated system which has been configured for lon/lat.\r\n\r\nThis change fixes a bug in the BoundingBoxReader class which was reversing the axis order\r\nwhen it shouldn't be and not reversing it when it should.  Also updated tests.\n\n<!-- Reviewable:start -->\n[<img src=\"https://reviewable.io/review_button.png\" height=40 alt=\"Review on Reviewable\"/>](https://reviewable.io/reviews/codice/ddf/262)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/codice/ddf/issues/comments/149366937","html_url":"https://github.com/codice/ddf/pull/262#issuecomment-149366937","issue_url":"https://api.github.com/repos/codice/ddf/issues/262","id":149366937,"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},"created_at":"2015-10-19T22:35:42Z","updated_at":"2015-10-19T22:35:42Z","body":"\n\n\n\nReview status: 5 of 6 files reviewed at latest revision, 7 unresolved discussions, some commit checks pending.\n\n---\n\n<sup>**[catalog/spatial/csw/spatial-csw-common/src/main/java/org/codice/ddf/spatial/ogc/csw/catalog/common/BoundingBoxReader.java, line 60 \\[r3\\]](https://reviewable.io:443/reviews/codice/ddf/262#-K10Nps3tvGF7tmQ9-co)** ([raw file](https://github.com/codice/ddf/blob/5f1fc234206bbe6affec7efe56d46f1c9541f0d3/catalog/spatial/csw/spatial-csw-common/src/main/java/org/codice/ddf/spatial/ogc/csw/catalog/common/BoundingBoxReader.java#L60)):</sup>\nI think that a `CswException` (as opposed to null) would be more ideal in this situation.  This shouldn't be outside of the scope of the ticket to change and test.\n<!-- Reviewable comment -K11DcfRB1HfzYmjtKjp:26550902 -->\n\n---\n\n<sup>**[catalog/spatial/csw/spatial-csw-common/src/test/java/org/codice/ddf/spatial/ogc/csw/catalog/common/BoundingBoxReaderTest.java, line 74 \\[r4\\]](https://reviewable.io:443/reviews/codice/ddf/262#-K11Cgr3Qz6PXLcxQNuz)** ([raw file](https://github.com/codice/ddf/blob/e65f556f7189c55a8a09b0a5b1808c0ce20f9d23/catalog/spatial/csw/spatial-csw-common/src/test/java/org/codice/ddf/spatial/ogc/csw/catalog/common/BoundingBoxReaderTest.java#L74)):</sup>\nIt might be good to move some of the setup into an @Before method.   Also, the classes in this package should follow the TestXXXX.java format.  \n<!-- Reviewable comment -K11Cgr3Qz6PXLcxQNv-:-1985046208 -->\n\n---\n\n\nComments from the [review on Reviewable.io](https://reviewable.io:443/reviews/codice/ddf/262)\n<!-- Sent from Reviewable.io -->\n"}}