Merge pull request #55 from nunocoracao/54-cant-add-homepage-image-if…

…-assets-folder-doesnt-exists 🐛 Can't add homepage image if assets folder doesn't exists
nunocoracao · Feb 10, 2024 · d2551ab · d2551ab
2 parents 1a176d7 + 7f528da
commit d2551ab
Show file tree

Hide file tree

Showing 4 changed files with 39 additions and 25 deletions.
diff --git a/configs/configOptions.json b/configs/configOptions.json
@@ -9,15 +9,15 @@
     },
     {
         "text": "Site's logo",
-        "method": "configLoop",
+        "method": "configImage",
         "file": "./config/_default/languages.en.toml",
         "parent": "params",
         "key": "logo",
         "description": "Site's logo, the logo file should be provided at 2x resolution and supports any image dimensions."
     },
     {
         "text": "Site's secondary logo",
-        "method": "configLoop",
+        "method": "configImage",
         "file": "./config/_default/languages.en.toml",
         "parent": "params",
         "key": "secondaryLogo",

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "blowfish-tools",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "CLI to initialize and configure a Blowfish project.",
   "main": "cli.js",
   "bin": {

diff --git a/src/flow.js b/src/flow.js
@@ -427,19 +427,32 @@ export default class flow {
             process.exit(0);
         }
 
-        utils.run('cp ' + newValue + ' ./assets/', false);
-        newValue = newValue.split('/').pop();
 
-        if (!parent) {
-            data[variable] = newValue
-        } else if (data[parent]) {
-            data[parent][variable] = newValue
+        var processChanges = () => {
+            utils.run('cp ' + newValue + ' ./assets/', false);
+            newValue = newValue.split('/').pop();
+
+            if (!parent) {
+                data[variable] = newValue
+            } else if (data[parent]) {
+                data[parent][variable] = newValue
+            } else {
+                data[parent] = {}
+                data[parent][variable] = newValue
+            }
+
+            utils.saveFileSync(file, toml.stringify(data));
+        }
+
+        if (!utils.directoryExists('./assets/')) {
+            utils.run('mkdir assets', false)
+                .then(() => {
+                    processChanges()
+                });
         } else {
-            data[parent] = {}
-            data[parent][variable] = newValue
+            processChanges()
         }
 
-        utils.saveFileSync(file, toml.stringify(data));
     }
 
     static async configMenus(file) {
@@ -845,7 +858,7 @@ var exitOption = {
 
 var configOptionsJSONList = utils.readAppJsonConfig('configOptions.json');
 
-function createAction (method, file, parent, key, description) {
+function createAction(method, file, parent, key, description) {
     return async (list) => {
         await flow[method](
             file,
@@ -894,11 +907,11 @@ var options = [
 
             var tempList = []
             for (var i in configOptions) {
-                if (configOptions[i].parent === null || configOptions[i].method === 'exit'){
-                    
+                if (configOptions[i].parent === null || configOptions[i].method === 'exit') {
+
                 }
 
-                    tempList.push(configOptions[i])      
+                tempList.push(configOptions[i])
             }
 
             flow.enterConfigMode(tempList);
@@ -912,7 +925,7 @@ var options = [
             var tempList = []
             for (var i in configOptions) {
                 if (configOptions[i].parent === 'author' || configOptions[i].method === 'exit')
-                    tempList.push(configOptions[i])      
+                    tempList.push(configOptions[i])
             }
 
             flow.enterConfigMode(tempList);
@@ -926,7 +939,7 @@ var options = [
             var tempList = []
             for (var i in configOptions) {
                 if (configOptions[i].parent === "homepage" || configOptions[i].method === 'exit')
-                    tempList.push(configOptions[i])      
+                    tempList.push(configOptions[i])
             }
 
             flow.enterConfigMode(tempList);
@@ -940,7 +953,7 @@ var options = [
             var tempList = []
             for (var i in configOptions) {
                 if (configOptions[i].parent === "header" || configOptions[i].method === 'exit')
-                    tempList.push(configOptions[i])      
+                    tempList.push(configOptions[i])
             }
 
             flow.enterConfigMode(tempList);
@@ -954,7 +967,7 @@ var options = [
             var tempList = []
             for (var i in configOptions) {
                 if (configOptions[i].parent === "footer" || configOptions[i].method === 'exit')
-                    tempList.push(configOptions[i])      
+                    tempList.push(configOptions[i])
             }
 
             flow.enterConfigMode(tempList);
@@ -968,7 +981,7 @@ var options = [
             var tempList = []
             for (var i in configOptions) {
                 if (configOptions[i].parent === "article" || configOptions[i].method === 'exit')
-                    tempList.push(configOptions[i])      
+                    tempList.push(configOptions[i])
             }
 
             flow.enterConfigMode(tempList);
@@ -982,7 +995,7 @@ var options = [
             var tempList = []
             for (var i in configOptions) {
                 if (configOptions[i].parent === "list" || configOptions[i].method === 'exit')
-                    tempList.push(configOptions[i])      
+                    tempList.push(configOptions[i])
             }
 
             flow.enterConfigMode(tempList);
@@ -996,7 +1009,7 @@ var options = [
             var tempList = []
             for (var i in configOptions) {
                 if (configOptions[i].parent === "taxonomy" || configOptions[i].method === 'exit')
-                    tempList.push(configOptions[i])      
+                    tempList.push(configOptions[i])
             }
 
             flow.enterConfigMode(tempList);
@@ -1010,7 +1023,7 @@ var options = [
             var tempList = []
             for (var i in configOptions) {
                 if (configOptions[i].parent === "term" || configOptions[i].method === 'exit')
-                    tempList.push(configOptions[i])      
+                    tempList.push(configOptions[i])
             }
 
             flow.enterConfigMode(tempList);
@@ -1024,7 +1037,7 @@ var options = [
             var tempList = []
             for (var i in configOptions) {
                 if (configOptions[i].method === "configImage" || configOptions[i].method === 'exit')
-                    tempList.push(configOptions[i])      
+                    tempList.push(configOptions[i])
             }
 
             flow.enterConfigMode(tempList);

diff --git a/src/utils.js b/src/utils.js
@@ -69,6 +69,7 @@ export default class utils {
         try {
             return fs.existsSync(path);
         } catch (err) {
+            console.log(err)
             return false;
         }
     }