Merge pull request #1522 from kadler15/issue-1521

[preservefiles] Fix global storage JSON dump

src/libcalamares/GlobalStorage.h

@@ -22,7 +22,7 @@ namespace Calamares
 
 /** @brief Storage for data that passes between Calamares modules.
  *
- * The Global Storage is global to the Calamars JobQueue and
+ * The Global Storage is global to the Calamares JobQueue and
  * everything that depends on that: all of its modules use the
  * same instance of the JobQueue, and so of the Global Storage.
  *

src/modules/preservefiles/PreserveFiles.cpp

@@ -137,9 +137,9 @@ PreserveFiles::exec()
         }
         if ( it.type == ItemType::Config )
         {
-            if ( Calamares::JobQueue::instance()->globalStorage()->saveJson( dest ) )
+            if ( !Calamares::JobQueue::instance()->globalStorage()->saveJson( dest ) )
             {
-                cWarning() << "Could not write config for" << dest;
+                cWarning() << "Could not write a JSON dump of global storage to" << dest;
             }
             else
             {

src/modules/preservefiles/preservefiles.conf

@@ -31,10 +31,9 @@
 #       page (may be empty, for instance if no user page is enabled)
 #
 # Special values for the key *from* are:
-# - *log*, for the complete log file (up to the moment the preservefiles
-#   module is run),
-# - *config*, for the Calamares configuration file
-# - *globals*, for a JSON dump of the contents of global storage
+#   - *log*, for the complete log file (up to the moment the preservefiles
+#     module is run),
+#   - *config*, for a JSON dump of the contents of global storage
 ---
 files:
   - /etc/oem-information
@@ -42,7 +41,7 @@ files:
     dest: /root/install.log
     perm: root:wheel:644
   - from: config
-    dest: /root/install.cfg
+    dest: /root/install.json
     perm: root:wheel:400
 
 # The *perm* key contains a default value to apply to all files listed