refactor: remove non-functional attribute

Functionality related to the differentiation between single/double-click
is handled by the dialog being visible via self.dlg.isVisible().

Author: koebi

ORStools/utils/maptools.py

@@ -77,7 +77,6 @@ def __init__(self, dlg):
         self.dlg.routing_preference_combo.currentIndexChanged.connect(self._toggle_preview)
         self.dlg.routing_travel_combo.currentIndexChanged.connect(self._toggle_preview)
 
-        self.last_click = "single-click"
         self.dragging_vertex = None
         self.moved_idxs = 0
         self.error_idxs = 0
@@ -175,95 +174,94 @@ def canvasReleaseEvent(self, event: QEvent) -> None:
         point = self.toMapCoordinates(event.pos())
         self.points.append(point)
 
-        if self.last_click == "single-click":
-            if self.dragging_vertex:
-                try:
+        if self.dragging_vertex:
+            try:
+                self.dragging_vertex = False
+                QApplication.restoreOverrideCursor()
+                crs = self.dlg.canvas.mapSettings().destinationCrs()
+
+                annotation = self.dlg._linetool_annotate_point(point, self.move_i, crs=crs)
+                self.dlg.annotations.insert(self.move_i, annotation)
+                self.dlg.project.annotationManager().addAnnotation(annotation)
+
+                transformer = transform.transformToWGS(crs)
+                point_wgs = transformer.transform(point)
+
+                items = [
+                    self.dlg.routing_fromline_list.item(x).text()
+                    for x in range(self.dlg.routing_fromline_list.count())
+                ]
+                backup = items.copy()
+                items[self.move_i] = (
+                    f"Point {self.move_i}: {point_wgs.x():.6f}, {point_wgs.y():.6f}"
+                )
+
+                self.dlg.routing_fromline_list.clear()
+                for i, x in enumerate(items):
+                    coords = x.split(":")[1]
+                    item = f"Point {i}:{coords}"
+                    self.dlg.routing_fromline_list.addItem(item)
+                self.create_rubber_band()
+                self.save_last_point(point, annotation)
+            except ApiError as e:
+                if self.get_error_code(e) == 2010:
                     self.dragging_vertex = False
-                    QApplication.restoreOverrideCursor()
-                    crs = self.dlg.canvas.mapSettings().destinationCrs()
-
-                    annotation = self.dlg._linetool_annotate_point(point, self.move_i, crs=crs)
-                    self.dlg.annotations.insert(self.move_i, annotation)
-                    self.dlg.project.annotationManager().addAnnotation(annotation)
-
-                    transformer = transform.transformToWGS(crs)
-                    point_wgs = transformer.transform(point)
-
-                    items = [
-                        self.dlg.routing_fromline_list.item(x).text()
-                        for x in range(self.dlg.routing_fromline_list.count())
-                    ]
-                    backup = items.copy()
-                    items[self.move_i] = (
-                        f"Point {self.move_i}: {point_wgs.x():.6f}, {point_wgs.y():.6f}"
-                    )
-
                     self.dlg.routing_fromline_list.clear()
                     for i, x in enumerate(items):
                         coords = x.split(":")[1]
                         item = f"Point {i}:{coords}"
                         self.dlg.routing_fromline_list.addItem(item)
-                    self.create_rubber_band()
-                    self.save_last_point(point, annotation)
-                except ApiError as e:
-                    if self.get_error_code(e) == 2010:
+                    self.dlg._reindex_list_items()
+                    self.radius_message_box(e)
+                else:
+                    raise e
+            except Exception as e:
+                if "Connection refused" in str(e):
+                    self.api_key_message_bar()
+                else:
+                    raise e
+        # Not moving release
+        else:
+            try:
+                if not self.dlg.isVisible():
+                    self.idx -= self.error_idxs
+                    self.dlg.create_vertex(point, self.idx)
+                    self.idx += 1
+                    self.error_idxs = 0
+
+                    if self.dlg.routing_fromline_list.count() > 1:
+                        self.create_rubber_band()
                         self.dragging_vertex = False
+            except ApiError as e:
+                if self.get_error_code(e) == 2010:
+                    self.error_idxs += 1
+                    num = len(self.dlg.routing_fromline_list) - 1
+
+                    if num < 2:
                         self.dlg.routing_fromline_list.clear()
-                        for i, x in enumerate(backup):
-                            coords = x.split(":")[1]
-                            item = f"Point {i}:{coords}"
-                            self.dlg.routing_fromline_list.addItem(item)
-                        self.dlg._reindex_list_items()
-                        self.radius_message_box(e)
-                    else:
-                        raise e
-                except Exception as e:
-                    if "Connection refused" in str(e):
-                        self.api_key_message_bar()
+                        self.dlg._clear_annotations()
                     else:
-                        raise e
-            # Not moving release
-            else:
-                try:
-                    if not self.dlg.isVisible():
-                        self.idx -= self.error_idxs
-                        self.dlg.create_vertex(point, self.idx)
-                        self.idx += 1
-                        self.error_idxs = 0
-
-                        if self.dlg.routing_fromline_list.count() > 1:
-                            self.create_rubber_band()
-                            self.dragging_vertex = False
-                except ApiError as e:
-                    if self.get_error_code(e) == 2010:
-                        self.error_idxs += 1
-                        num = len(self.dlg.routing_fromline_list) - 1
-
-                        if num < 2:
-                            self.dlg.routing_fromline_list.clear()
-                            self.dlg._clear_annotations()
-                        else:
-                            self.dlg.routing_fromline_list.takeItem(num)
-                            self.dlg._reindex_list_items()
-                            self.create_rubber_band()
-
-                        self.radius_message_box(e)
-                    else:
-                        raise e
-                except Exception as e:
-                    if "Connection refused" in str(e):
-                        self.api_key_message_bar()
-                    else:
-                        raise e
+                        self.dlg.routing_fromline_list.takeItem(num)
+                        self.dlg._reindex_list_items()
+                        self.create_rubber_band()
+
+                    self.radius_message_box(e)
+                else:
+                    raise e
+            except Exception as e:
+                if "Connection refused" in str(e):
+                    self.api_key_message_bar()
+                else:
+                    raise e
 
-        self.last_click = "single-click"
 
     def canvasDoubleClickEvent(self, e: QEvent) -> None:
         """
         Populate line list widget with coordinates, end point moving and show dialog again.
         """
         self.dlg.show()
-        self.last_click = "double-click"
+        # not resetting the cursor (and thus the cursor icon still being "+") suggest that you can still set points, which you can't.
+        # Thus, we should set another, different cursor icon here?
 
     def create_rubber_band(self) -> None:
         if self.dlg.optimization_group.isChecked() and self.dlg.routing_fromline_list.count() == 2: