Changeset 108080 in vbox for trunk/src/VBox/Frontends

Timestamp:

Feb 5, 2025 5:22:27 PM (3 months ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

167374

Message:

FE/Qt: VM settings dialog: Do not show machine data changed warning if serialization failed.

Location:

trunk/src/VBox/Frontends/VirtualBox/src/settings

Files:

• UIAdvancedSettingsDialog.h (modified) (1 diff)
• UIAdvancedSettingsDialogSpecific.cpp (modified) (1 diff)

trunk/src/VBox/Frontends/VirtualBox/src/settings/UIAdvancedSettingsDialog.h

@@ -143 +143 @@
     bool isSerializationInProgress() const { return m_fSerializationIsInProgress; }
 
+    /** Holds whether there were no serialization errors. */
+    bool isSerializationClean() const { return m_fSerializationClean; }
+    /** Resets whether there were no serialization errors. */
+    void resetSerializationClean() { m_fSerializationClean = true; }
+
     /** Returns dialog optional flags. */
     QMap<QString, QVariant> optionalFlags() const { return m_flags; }

trunk/src/VBox/Frontends/VirtualBox/src/settings/UIAdvancedSettingsDialogSpecific.cpp

@@ -580 +580 @@
 
     /* Check if user had changed something and warn him about he will loose settings on reloading: */
-    if (isSettingsChanged() && !msgCenter().confirmSettingsReloading(this))
-        return;
+    if (isSerializationClean() && isSettingsChanged() && !msgCenter().confirmSettingsReloading(this))
+        return;
+
+    /* Make sure serialization reseted: */
+    resetSerializationClean();
 
     /* Reload data: */