GitHub PullRequestReviewCommentEvent at 2014-02-24 13:40:13
Repository: moxie
Actor: martinfilliau
Payload: {"comment":{"url":"https://api.github.com/repos/ox-it/moxie/pulls/comments/9990806","id":9990806,"diff_hunk":"@@ -234,3 +240,40 @@ def as_dict(self):\n         representation.add_curie('hl', RELATIONS_CURIE)\n         representation.add_link('hl:search', url_for('places.search') + \"?type={type}\")\n         return representation.as_dict()\n+\n+\n+class GeoJsonPointsRepresentation(object):\n+\n+    def __init__(self, results):\n+        self.results = results\n+\n+    def as_dict(self):\n+        features = []\n+        for result in self.results:\n+            if result.shape:\n+                f = Feature(id=result.id,\n+                            geometry=wkt_loads(result.shape),\n+                            properties=self._get_feature_properties(result))\n+                features.append(f)\n+            elif result.lat and result.lon:\n+                # if a result does not have a shape, attempt to\n+                # fallback on latitude / longitude\n+                f = Feature(id=result.id,\n+                            geometry=Point(float(result.lat), float(result.lon)),","path":"moxie/places/representations.py","position":43,"original_position":43,"commit_id":"6b2d5143f8bbbd8286e5da81cb3caa550f4cac33","original_commit_id":"6b2d5143f8bbbd8286e5da81cb3caa550f4cac33","user":{"login":"martinfilliau","id":21462,"avatar_url":"https://gravatar.com/avatar/b0c47c732a13bf869b7d8a800ebe847a?d=https%3A%2F%2Fidenticons.github.com%2F37e83c96bcea8158cced5add35c02417.png&r=x","gravatar_id":"b0c47c732a13bf869b7d8a800ebe847a","url":"https://api.github.com/users/martinfilliau","html_url":"https://github.com/martinfilliau","followers_url":"https://api.github.com/users/martinfilliau/followers","following_url":"https://api.github.com/users/martinfilliau/following{/other_user}","gists_url":"https://api.github.com/users/martinfilliau/gists{/gist_id}","starred_url":"https://api.github.com/users/martinfilliau/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martinfilliau/subscriptions","organizations_url":"https://api.github.com/users/martinfilliau/orgs","repos_url":"https://api.github.com/users/martinfilliau/repos","events_url":"https://api.github.com/users/martinfilliau/events{/privacy}","received_events_url":"https://api.github.com/users/martinfilliau/received_events","type":"User","site_admin":false},"body":"Yes you are correct -- I fixed it in the wrong branch: https://github.com/ox-it/moxie/commit/290eda513e2125240ee0cf290b2444a135689bc0\r\nShall I (try to) cherry pick the commit?","created_at":"2014-02-24T13:40:13Z","updated_at":"2014-02-24T13:40:13Z","html_url":"https://github.com/ox-it/moxie/pull/42#discussion_r9990806","pull_request_url":"https://api.github.com/repos/ox-it/moxie/pulls/42","_links":{"self":{"href":"https://api.github.com/repos/ox-it/moxie/pulls/comments/9990806"},"html":{"href":"https://github.com/ox-it/moxie/pull/42#discussion_r9990806"},"pull_request":{"href":"https://api.github.com/repos/ox-it/moxie/pulls/42"}}}}