{"action":"created","issue":{"url":"https://api.github.com/repos/mapstraction/mxn/issues/256","labels_url":"https://api.github.com/repos/mapstraction/mxn/issues/256/labels{/name}","comments_url":"https://api.github.com/repos/mapstraction/mxn/issues/256/comments","events_url":"https://api.github.com/repos/mapstraction/mxn/issues/256/events","html_url":"https://github.com/mapstraction/mxn/pull/256","id":15607830,"number":256,"title":"Implement a Nominatim geocoder based on the OpenLayers geocoder.","user":{"login":"ChALkeR","id":291301,"avatar_url":"https://avatars.githubusercontent.com/u/291301?v=3","gravatar_id":"","url":"https://api.github.com/users/ChALkeR","html_url":"https://github.com/ChALkeR","followers_url":"https://api.github.com/users/ChALkeR/followers","following_url":"https://api.github.com/users/ChALkeR/following{/other_user}","gists_url":"https://api.github.com/users/ChALkeR/gists{/gist_id}","starred_url":"https://api.github.com/users/ChALkeR/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ChALkeR/subscriptions","organizations_url":"https://api.github.com/users/ChALkeR/orgs","repos_url":"https://api.github.com/users/ChALkeR/repos","events_url":"https://api.github.com/users/ChALkeR/events{/privacy}","received_events_url":"https://api.github.com/users/ChALkeR/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/mapstraction/mxn/milestones/6","html_url":"https://github.com/mapstraction/mxn/milestones/Release%203.1","labels_url":"https://api.github.com/repos/mapstraction/mxn/milestones/6/labels","id":497418,"number":6,"title":"Release 3.1","description":"","creator":{"login":"gilesc50","id":1397303,"avatar_url":"https://avatars.githubusercontent.com/u/1397303?v=3","gravatar_id":"","url":"https://api.github.com/users/gilesc50","html_url":"https://github.com/gilesc50","followers_url":"https://api.github.com/users/gilesc50/followers","following_url":"https://api.github.com/users/gilesc50/following{/other_user}","gists_url":"https://api.github.com/users/gilesc50/gists{/gist_id}","starred_url":"https://api.github.com/users/gilesc50/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gilesc50/subscriptions","organizations_url":"https://api.github.com/users/gilesc50/orgs","repos_url":"https://api.github.com/users/gilesc50/repos","events_url":"https://api.github.com/users/gilesc50/events{/privacy}","received_events_url":"https://api.github.com/users/gilesc50/received_events","type":"User","site_admin":false},"open_issues":7,"closed_issues":1,"state":"open","created_at":"2013-11-26T16:10:48Z","updated_at":"2015-05-20T18:28:27Z","due_on":null,"closed_at":null},"comments":18,"created_at":"2013-06-16T17:25:01Z","updated_at":"2015-07-30T17:14:34Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mapstraction/mxn/pulls/256","html_url":"https://github.com/mapstraction/mxn/pull/256","diff_url":"https://github.com/mapstraction/mxn/pull/256.diff","patch_url":"https://github.com/mapstraction/mxn/pull/256.patch"},"body":"The only difference is in the method used to load data from Nominatim.\r\n\r\nOpenLayers geocoder uses OpenLayers.Request.GET(). This geocoder tries to use jQuery.ajax if available, then OpenLayers.Request.GET if available, and falls back to plain XMLHttpRequest.\r\n\r\nThe usecase I'm thinking about is if one is already using jQuery while using some other map provider (that does not have a geocoder?). There should be no need to load OpenLayers library (which is just too heavy) in such situations.\r\n\r\nOr if one does not care about very old browsers, he does need even need any hacks or workarounds and can use just plain XMLHttpRequest.\r\n\r\nThis file uses CRLF line terminators, like mxn.openlayers.geocoder.js. There seems to be a mix in the repo, but I decided not to touch mxn.openlayers.geocoder.js and keep mxn.nominatim.geocoder.js also CRLF to ease diffing between them (yes, I know about the --ignore-all-space flag).\r\n\r\nIf you want LF line terminators for all new files, you can change that.\r\n\r\nThis is a redo of #245."},"comment":{"url":"https://api.github.com/repos/mapstraction/mxn/issues/comments/126406603","html_url":"https://github.com/mapstraction/mxn/pull/256#issuecomment-126406603","issue_url":"https://api.github.com/repos/mapstraction/mxn/issues/256","id":126406603,"user":{"login":"ChALkeR","id":291301,"avatar_url":"https://avatars.githubusercontent.com/u/291301?v=3","gravatar_id":"","url":"https://api.github.com/users/ChALkeR","html_url":"https://github.com/ChALkeR","followers_url":"https://api.github.com/users/ChALkeR/followers","following_url":"https://api.github.com/users/ChALkeR/following{/other_user}","gists_url":"https://api.github.com/users/ChALkeR/gists{/gist_id}","starred_url":"https://api.github.com/users/ChALkeR/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ChALkeR/subscriptions","organizations_url":"https://api.github.com/users/ChALkeR/orgs","repos_url":"https://api.github.com/users/ChALkeR/repos","events_url":"https://api.github.com/users/ChALkeR/events{/privacy}","received_events_url":"https://api.github.com/users/ChALkeR/received_events","type":"User","site_admin":false},"created_at":"2015-07-30T17:14:34Z","updated_at":"2015-07-30T17:14:34Z","body":"Hi all.\r\n\r\nMapstraction project seems stalled, so I ported my stuff away from it. I'm now using a thin abstraction layer of my own that supports the only the minimal set of features that I need (map, controls, markers, geocoding) and only Leaflet/Google/Yandex as backends.\r\n\r\nI will not update this pull request, so anyone who is interested is welcome to overtake it and notify me, so I will close this one. If you are an owner or a collaborator, you can just close this pull request without further notice when you feel like doing it."}}