Added wrapper function for checking, asking and committing changes.

Added wrapper function

Author: Mtk112

arho_feature_template/plugin.py

@@ -17,12 +17,7 @@
 from arho_feature_template.qgis_plugin_tools.tools.i18n import setup_translation
 from arho_feature_template.qgis_plugin_tools.tools.resources import plugin_name
 from arho_feature_template.utils.db_utils import get_existing_database_connection_names
-from arho_feature_template.utils.misc_utils import (
-    PLUGIN_PATH,
-    check_layer_changes,
-    commit_all_layer_changes,
-    prompt_commit_changes,
-)
+from arho_feature_template.utils.misc_utils import PLUGIN_PATH, handle_unsaved_changes
 
 if TYPE_CHECKING:
     from qgis.gui import QgisInterface, QgsMapTool
@@ -187,7 +182,7 @@ def load_existing_land_use_plan(self) -> None:
             QMessageBox.critical(None, "Error", "No database connections found.")
             return
 
-        if check_layer_changes() and (not prompt_commit_changes() or not commit_all_layer_changes()):
+        if not handle_unsaved_changes():
             return
 
         dialog = LoadPlanDialog(None, connections)

arho_feature_template/utils/misc_utils.py

@@ -40,3 +40,18 @@ def commit_all_layer_changes() -> bool:
             all_committed = False
 
     return all_committed
+
+
+def handle_unsaved_changes() -> bool:
+    """
+    Wrapper function to check for unsaved changes, prompt user to commit, and commit changes if chosen.
+    Returns:
+        bool: True if changes are committed or no changes were found;
+            False if user does not want to commit or if commit fails.
+    """
+    if check_layer_changes():
+        if not prompt_commit_changes():
+            return False
+        if not commit_all_layer_changes():
+            return False
+    return True