feat: show coordinates in routing_from_line_list in project crs

Author: merydian

ORStools/gui/ORStoolsDialog.py

@@ -84,7 +84,7 @@
     PROFILES,
     PREFERENCES,
 )
-from ORStools.utils import maptools, configmanager, transform, gui, exceptions
+from ORStools.utils import maptools, configmanager, gui, exceptions
 from .ORStoolsDialogConfig import ORStoolsDialogConfigMain
 
 MAIN_WIDGET, _ = uic.loadUiType(gui.GuiUtils.get_ui_file_path("ORStoolsDialogUI.ui"))
@@ -415,6 +415,9 @@ def __init__(self, iface: QgisInterface, parent=None) -> None:
             lambda: self.color_duplicate_items(self.routing_fromline_list)
         )
 
+        self.last_crs = self.canvas.mapSettings().destinationCrs()
+        self.canvas.destinationCrsChanged.connect(self._on_crs_changed)
+
         self.load_provider_combo_state()
         self.provider_combo.activated.connect(self.save_selected_provider_state)
 
@@ -435,6 +438,7 @@ def _save_vertices_to_layer(self) -> None:
             self.routing_fromline_list.item(x).text()
             for x in range(self.routing_fromline_list.count())
         ]
+        map_crs = self._iface.mapCanvas().mapSettings().destinationCrs()
 
         if len(items) > 0:
             timestamp = datetime.now().strftime("%Y-%m-%d_%H:%M:%S")
@@ -450,6 +454,7 @@ def _save_vertices_to_layer(self) -> None:
                 feature.setAttributes([idx])
 
                 point_layer.dataProvider().addFeature(feature)
+                point_layer.setCrs(map_crs)
             QgsProject.instance().addMapLayer(point_layer)
             self.canvas.refresh()
 
@@ -530,11 +535,7 @@ def _on_linetool_init(self) -> None:
 
     def create_vertex(self, point, idx):
         """Adds an item to QgsListWidget and annotates the point in the map canvas"""
-        map_crs = self.canvas.mapSettings().destinationCrs()
-
-        transformer = transform.transformToWGS(map_crs)
-        point_wgs = transformer.transform(point)
-        self.routing_fromline_list.addItem(f"Point {idx}: {point_wgs.x():.6f}, {point_wgs.y():.6f}")
+        self.routing_fromline_list.addItem(f"Point {idx}: {point.x():.6f}, {point.y():.6f}")
 
         crs = self.canvas.mapSettings().destinationCrs()
         annotation = self._linetool_annotate_point(point, idx, crs)
@@ -549,17 +550,20 @@ def _reindex_list_items(self) -> None:
         ]
         self.routing_fromline_list.clear()
         self._clear_annotations()
-        crs = QgsCoordinateReferenceSystem(f"EPSG:{4326}")
         project_crs = self.canvas.mapSettings().destinationCrs()
-        for idx, x in enumerate(items):
-            coords = x.split(":")[1]
-            item = f"Point {idx}:{coords}"
-            x, y = (float(i) for i in coords.split(", "))
+        for idx, origin in enumerate(items):
+            origin_coords = origin.split(":")[1]
+            x, y = (float(i) for i in origin_coords.split(", "))
             point = QgsPointXY(x, y)
 
-            self.routing_fromline_list.addItem(item)
-            transform = QgsCoordinateTransform(crs, project_crs, QgsProject.instance())
+            transform = QgsCoordinateTransform(self.last_crs, project_crs, QgsProject.instance())
+
             point = transform.transform(point)
+            
+            coords = f"{point.x():.6f}, {point.y():.6f}"
+            item = f"Point {idx}: {coords}"
+            self.routing_fromline_list.addItem(item)
+
             annotation = self._linetool_annotate_point(point, idx)
             self.annotations.append(annotation)
             self.project.annotationManager().addAnnotation(annotation)
@@ -606,3 +610,8 @@ def show(self):
         """Load the saved state when the window is shown"""
         super().show()
         self.load_provider_combo_state()
+
+    def _on_crs_changed(self) -> None:
+        """Handle CRS change event."""  
+        self._reindex_list_items()
+        self.last_crs = self.canvas.mapSettings().destinationCrs()

ORStools/gui/directions_gui.py

@@ -86,13 +86,12 @@ def _get_avoid_options(avoid_boxes):
 class Directions:
     """Extended functionality for directions endpoint for GUI."""
 
-    def __init__(self, dlg, iface):
+    def __init__(self, dlg):
         """
         :param dlg: Main GUI dialog.
         :type dlg: QDialog
         """
         self.dlg = dlg
-        self.iface = iface
 
         self.options = dict()
 
@@ -103,7 +102,7 @@ def get_request_line_feature(self):
         :returns: coordinate list of line
         :rtype: list
         """
-        map_crs = self.iface.mapCanvas().mapSettings().destinationCrs()
+        map_crs = self.dlg.canvas.mapSettings().destinationCrs()
         transformer = transform.transformToWGS(map_crs)
 
         coordinates = []

ORStools/utils/maptools.py

@@ -32,11 +32,8 @@
 
 from qgis.gui import QgsMapToolEmitPoint, QgsRubberBand
 from qgis.core import (
-    QgsProject,
     QgsPointXY,
-    QgsCoordinateReferenceSystem,
     Qgis,
-    QgsCoordinateTransform,
     QgsWkbTypes,
     QgsAnnotation,
     QgsMarkerSymbol,
@@ -46,7 +43,7 @@
 from qgis.PyQt.QtGui import QColor, QMouseEvent
 
 from ORStools import ROUTE_COLOR
-from ORStools.utils import transform, router
+from ORStools.utils import router
 from ORStools.utils.exceptions import ApiError
 
 
@@ -188,16 +185,13 @@ def canvasReleaseEvent(self, event: QEvent) -> None:
                 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}"
+                    f"Point {self.move_i}: {point.x():.6f}, {point.y():.6f}"
                 )
 
                 self.dlg.routing_fromline_list.clear()
@@ -291,19 +285,15 @@ def create_rubber_band(self) -> None:
             else:
                 self.dlg._clear_annotations()
         else:
-            dest_crs = self.dlg.canvas.mapSettings().destinationCrs()
-            original_crs = QgsCoordinateReferenceSystem("EPSG:4326")
-            transform = QgsCoordinateTransform(original_crs, dest_crs, QgsProject.instance())
             items = [
                 self.dlg.routing_fromline_list.item(x).text()
                 for x in range(self.dlg.routing_fromline_list.count())
             ]
             split = [x.split(":")[1] for x in items]
             coords = [tuple(map(float, coord.split(", "))) for coord in split]
             points_xy = [QgsPointXY(x, y) for x, y in coords]
-            reprojected_point = [transform.transform(point) for point in points_xy]
-            for point in reprojected_point:
-                if point == reprojected_point[-1]:
+            for point in points_xy:
+                if point == points_xy[-1]:
                     self.dlg.rubber_band.addPoint(point, True)
                 else:
                     self.dlg.rubber_band.addPoint(point, False)