Merge pull request #91 from Loop3D/copilot/fix-qgis-4-compatibility

Fix QGIS 4 compatibility: replace removed PyQt6.Qt monolithic import

Author: lachlangrose

.github/workflows/linter.yml

@@ -38,7 +38,7 @@ jobs:
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
-          pip install black ruff
+          pip install black "ruff==0.15.22"
       - name: Autoformat with black
         run: |
           black .

loopstructural/__about__.py

@@ -55,7 +55,7 @@ def plugin_metadata_as_dict() -> dict:
         config.read(PLG_METADATA_FILE.resolve(), encoding="UTF-8")
         return {s: dict(config.items(s)) for s in config.sections()}
     else:
-        raise IOError("Plugin metadata.txt not found at: %s" % PLG_METADATA_FILE)
+        raise OSError("Plugin metadata.txt not found at: %s" % PLG_METADATA_FILE)
 
 
 # ############################################################################
@@ -66,7 +66,7 @@ def plugin_metadata_as_dict() -> dict:
 __plugin_md__: dict = plugin_metadata_as_dict()
 
 __author__: str = __plugin_md__.get("general").get("author")
-__copyright__: str = "2024 - {0}, {1}".format(date.today().year, __author__)
+__copyright__: str = f"2024 - {date.today().year}, {__author__}"
 __email__: str = __plugin_md__.get("general").get("email")
 __icon_path__: Path = DIR_PLUGIN_ROOT.resolve() / __plugin_md__.get("general").get("icon")
 __keywords__: list = [

loopstructural/debug_manager.py

@@ -245,13 +245,13 @@ def log_params(self, context_label: str, params: Any):
         """
         try:
             self.plugin.log(
-                message=f"[map2loop] {context_label} parameters: {str(params)}",
+                message=f"[map2loop] {context_label} parameters: {params!s}",
                 log_level=0,
             )
         except Exception as err:
             self.plugin.log(
                 message=(
-                    f"[map2loop] {context_label} parameters (stringified due to {err}): {str(params)}"
+                    f"[map2loop] {context_label} parameters (stringified due to {err}): {params!s}"
                 ),
                 log_level=0,
             )

loopstructural/gui/compatibility.py

@@ -0,0 +1,20 @@
+# compat.py
+from qgis.PyQt.QtCore import QVariant
+
+try:
+    from qgis.PyQt.QtCore import QMetaType
+
+    # We create a proxy class to mimic the old QVariant.Type behavior
+    class QVariantProxy:
+        Type = QMetaType.Type
+        # Add common types here if needed
+        Int = QMetaType.Type.Int
+        Double = QMetaType.Type.Double
+        String = QMetaType.Type.QString
+        Bool = QMetaType.Type.Bool
+
+    # In QGIS 4, we use our proxy
+    QVariantCompat = QVariantProxy
+except (ImportError, AttributeError):
+    # In QGIS 3, QVariant already has .Type
+    QVariantCompat = QVariant

loopstructural/gui/data_conversion/configuration.py

@@ -2,8 +2,9 @@
 
 from __future__ import annotations
 
+from collections.abc import Iterable, MutableMapping
 from copy import deepcopy
-from typing import Any, Dict, Iterable, MutableMapping
+from typing import Any, Dict
 
 
 class Config:

loopstructural/gui/data_conversion/data_conversion_widget.py

@@ -4,12 +4,15 @@
 
 import os
 import re
+from collections.abc import Iterable, Mapping
 from dataclasses import dataclass
-from typing import Any, Dict, Iterable, List, Mapping, Optional, Tuple
+from typing import Any, Dict, List, Optional, Tuple
 
 from LoopDataConverter import Datatype, InputData, LoopConverter, SurveyName
-from PyQt5.QtCore import Qt, QTimer
-from PyQt5.QtWidgets import (
+from qgis.core import QgsMapLayerProxyModel, QgsProject, QgsVectorLayer
+from qgis.gui import QgsMapLayerComboBox
+from qgis.PyQt.QtCore import Qt, QTimer
+from qgis.PyQt.QtWidgets import (
     QComboBox,
     QDialog,
     QDialogButtonBox,
@@ -21,8 +24,6 @@
     QVBoxLayout,
     QWidget,
 )
-from qgis.core import QgsMapLayerProxyModel, QgsProject, QgsVectorLayer
-from qgis.gui import QgsMapLayerComboBox
 
 from ...main.helpers import ColumnMatcher
 from ...main.vectorLayerWrapper import QgsLayerFromDataFrame, QgsLayerFromGeoDataFrame

loopstructural/gui/dlg_settings.py

@@ -12,7 +12,7 @@
 from qgis.core import Qgis, QgsApplication
 from qgis.gui import QgsOptionsPageWidget, QgsOptionsWidgetFactory
 from qgis.PyQt import uic
-from qgis.PyQt.Qt import QUrl
+from qgis.PyQt.QtCore import QUrl
 from qgis.PyQt.QtGui import QDesktopServices, QIcon
 
 # project
@@ -30,7 +30,7 @@
 # ########## Globals ###############
 # ##################################
 
-FORM_CLASS, _ = uic.loadUiType(Path(__file__).parent / "{}.ui".format(Path(__file__).stem))
+FORM_CLASS, _ = uic.loadUiType(Path(__file__).parent / f"{Path(__file__).stem}.ui")
 
 
 # ############################################################################
@@ -48,7 +48,7 @@ def __init__(self, parent):
 
         # load UI and set objectName
         self.setupUi(self)
-        self.setObjectName("mOptionsPage{}".format(__title__))
+        self.setObjectName(f"mOptionsPage{__title__}")
 
         _report_context_message = quote(
             "> Reported from plugin settings\n\n"

loopstructural/gui/loop_widget.py

@@ -5,7 +5,7 @@
 interface for interacting with LoopStructural features inside QGIS.
 """
 
-from PyQt5.QtWidgets import QTabWidget, QVBoxLayout, QWidget
+from qgis.PyQt.QtWidgets import QTabWidget, QVBoxLayout, QWidget
 
 from .modelling.modelling_widget import ModellingWidget
 from .visualisation.visualisation_widget import VisualisationWidget

loopstructural/gui/map2loop_tools/basal_contacts_widget.py

@@ -2,9 +2,9 @@
 
 import os
 
-from PyQt5.QtWidgets import QMessageBox, QWidget
 from qgis.core import QgsProject, QgsVectorFileWriter
 from qgis.PyQt import uic
+from qgis.PyQt.QtWidgets import QMessageBox, QWidget
 
 from ...main.helpers import ColumnMatcher, get_layer_names
 from ...main.m2l_api import extract_basal_contacts
@@ -277,11 +277,11 @@ def set_parameters(self, params):
         params : dict
             Dictionary of parameters to set.
         """
-        if 'geology_layer' in params and params['geology_layer']:
+        if params.get('geology_layer'):
             self.geologyLayerComboBox.setLayer(params['geology_layer'])
-        if 'faults_layer' in params and params['faults_layer']:
+        if params.get('faults_layer'):
             self.faultsLayerComboBox.setLayer(params['faults_layer'])
-        if 'ignore_units' in params and params['ignore_units']:
+        if params.get('ignore_units'):
             self.ignoreUnitsLineEdit.setText(', '.join(params['ignore_units']))
         if 'all_contacts' in params:
             self.allContactsCheckBox.setChecked(params['all_contacts'])
@@ -310,7 +310,7 @@ def _is_null_like(v):
                 if v is None:
                     return True
                 # PyQGIS QVariant null check
-                if hasattr(v, "isNull") and callable(getattr(v, "isNull")) and v.isNull():
+                if hasattr(v, "isNull") and callable(v.isNull) and v.isNull():
                     return True
                 # Empty strings or literal "NULL" (case-insensitive)
                 if isinstance(v, str):

loopstructural/gui/map2loop_tools/dialogs.py

@@ -4,7 +4,7 @@
 instead of QGIS processing algorithms.
 """
 
-from PyQt5.QtWidgets import QDialog, QDialogButtonBox, QVBoxLayout
+from qgis.PyQt.QtWidgets import QDialog, QDialogButtonBox, QVBoxLayout
 
 
 class SamplerDialog(QDialog):
@@ -23,7 +23,9 @@ def setup_ui(self):
         from .sampler_widget import SamplerWidget
 
         layout = QVBoxLayout(self)
-        self.widget = SamplerWidget(self, data_manager=self.data_manager, debug_manager=self.debug_manager)
+        self.widget = SamplerWidget(
+            self, data_manager=self.data_manager, debug_manager=self.debug_manager
+        )
         layout.addWidget(self.widget)
 
         # Replace the run button with dialog buttons