Add comments

controllers/config.js

@@ -3,19 +3,16 @@
 const fs = require('fs');
 
 /**
- * config.js controller
- *
- * @description: A set of functions called "actions" of the `config` plugin.
+ * Main controllers for config import/export.
  */
 
 module.exports = {
-
   /**
-   * Default action.
+   * Export all config, from db to filesystem.
    *
-   * @return {Object}
+   * @param {object} ctx - Request context object.
+   * @returns {void}
    */
-
   export: async (ctx) => {
     const coreStoreAPI = strapi.query('core_store');
     const coreStore = await coreStoreAPI.find({ _limit: -1 });
@@ -29,6 +26,12 @@ module.exports = {
     });
   },
 
+  /**
+   * Import all config, from filesystem to db.
+   *
+   * @param {object} ctx - Request context object.
+   * @returns {void}
+   */
   import: async (ctx) => {
     // Check for existance of the config file destination dir.
     if (!fs.existsSync(strapi.plugins.config.config.destination)) {

services/config.js

@@ -4,22 +4,28 @@ const fs = require('fs');
 const util = require('util');
 
 /**
- * config.js service
- *
- * @description: A set of functions similar to controller's actions to avoid code duplication.
+ * Main services for config import/export.
  */
 
 module.exports = {
+  /**
+   * Write a single config file.
+   *
+   * @param {string} configName - The name of the config file.
+   * @param {string} fileContents - The JSON content of the config file.
+   * @returns {void}
+   */
   writeConfigFile: async (configName, fileContents) => {
+    // Check if the config should be excluded.
     const shouldExclude = strapi.plugins.config.config.exclude.includes(configName);
     if (shouldExclude) return;
 
+    // Check if the JSON content should be minified. 
     const json = 
       !strapi.plugins.config.config.minify ? 
         JSON.stringify(JSON.parse(fileContents), null, 2)
         : fileContents;
 
-    // Create the export folder if it does not yet exist.
     if (!fs.existsSync(strapi.plugins.config.config.destination)) {
       fs.mkdirSync(strapi.plugins.config.config.destination, { recursive: true });
     }
@@ -28,6 +34,12 @@ module.exports = {
     await writeFile(`${strapi.plugins.config.config.destination}${configName}.json`, json);
   },
 
+  /**
+   * Read from a config file.
+   *
+   * @param {string} configName - The name of the config file.
+   * @returns {object} The JSON content of the config file.
+   */
   readConfigFile: async (configName) => {
     const readFile = util.promisify(fs.readFile);
     return await readFile(`${strapi.plugins.config.config.destination}${configName}.json`)
@@ -39,27 +51,28 @@ module.exports = {
       });
   },
 
+  /**
+   * Import a config file into the db.
+   *
+   * @param {string} configName - The name of the config file.
+   * @returns {void}
+   */
   importFromFile: async (configName) => {
+    // Check if the config should be excluded.
     const shouldExclude = strapi.plugins.config.config.exclude.includes(configName);
     if (shouldExclude) return;
 
     const coreStoreAPI = strapi.query('core_store');
     const fileContents = await strapi.plugins.config.services.config.readConfigFile(configName);
 
-    try {
-      const configExists = await strapi
+    const configExists = await strapi
       .query('core_store')
       .findOne({ key: configName });
 
-      if (!configExists) {
-        await coreStoreAPI.create({ key: configName, value: fileContents });
-      } else {
-        await coreStoreAPI.update({ key: configName }, { value: fileContents });
-      }
-
-      return { success: true };
-    } catch (err) {
-      throw new Error(err);
+    if (!configExists) {
+      await coreStoreAPI.create({ key: configName, value: fileContents });
+    } else {
+      await coreStoreAPI.update({ key: configName }, { value: fileContents });
     }
   }
 };