From c8c6077a9c636aea3369743ae1385b8c07ba5b21 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Stephan=20B=C3=B6sch-Plepelits?= <skunk@xover.mud.at>
Date: Sat, 21 Mar 2020 17:16:26 +0100
Subject: [PATCH] editLink: decide editor option after clicking (return always
 the same link)

---
 src/editLink.js | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/src/editLink.js b/src/editLink.js
index 492bfdc7..70985c50 100644
--- a/src/editLink.js
+++ b/src/editLink.js
@@ -1,4 +1,4 @@
-window.editLinkRemote = function (type, osm_id) {
+function editLinkRemote (type, osm_id) {
   let id = type.substr(0, 1) + osm_id
 
   global.overpassFrontend.get(
@@ -27,18 +27,24 @@ window.editLinkRemote = function (type, osm_id) {
       }
     }
   )
-
-  return false
 }
 
-module.exports = function editLink (object) {
+window.editLink = function (type, osm_id) {
   switch (global.options.editor) {
     case 'remote':
-      return '<a class="editLink" href="#" onclick="return editLinkRemote(\'' + object.object.type + '\', ' + object.object.osm_id + ')">' + lang('edit') + '</a>'
+      editLinkRemote(type, osm_id)
+      break
     case 'id':
     default:
-      return '<a target="_blank" class="editLink" href="' + global.config.urlOpenStreetMap + '/edit?editor=id&' + object.object.type + '=' + object.object.osm_id + '">' + lang('edit') + '</a>'
+      let url = global.config.urlOpenStreetMap + '/edit?editor=id&' + type + '=' + osm_id
+      window.open(url)
   }
+
+  return false
+}
+
+module.exports = function (object) {
+  return '<a class="editLink" href="#edit" onclick="return editLink(\'' + object.object.type + '\', ' + object.object.osm_id + ')">' + lang('edit') + '</a>'
 }
 
 register_hook('options_orig_data', function (data) {