Changeset 22699 in vbox

Timestamp:

Sep 2, 2009 9:32:40 AM (15 years ago)

Author:

vboxsync

Message:

Main/XML: fix bad snapshot description written by earlier trunk builds that confuses VirtualBox versions before 3.1

File:

• trunk/src/VBox/Main/xml/Settings.cpp (modified) (2 diffs)

trunk/src/VBox/Main/xml/Settings.cpp

@@ -1654 +1654 @@
         throw ConfigFileError(this, &elmSnapshot, N_("Required Snapshot/@name attribute is missing"));
 
+    // earlier 3.1 trunk builds had a bug and added Description as an attribute, read it silently and write it back as an element
     elmSnapshot.getAttributeValue("Description", snap.strDescription);
 
@@ -2170 +2171 @@
     pelmSnapshot->setAttribute("uuid", makeString(snap.uuid));
     pelmSnapshot->setAttribute("name", snap.strName);
-    pelmSnapshot->setAttribute("Description", snap.strDescription);
-
     pelmSnapshot->setAttribute("timeStamp", makeString(snap.timestamp));
 
     if (snap.strStateFile.length())
         pelmSnapshot->setAttribute("stateFile", snap.strStateFile);
+
+    if (snap.strDescription.length())
+        pelmSnapshot->createChild("Description")->addContent(snap.strDescription);
 
     writeHardware(*pelmSnapshot, snap.hardware);