Flows lib honours flowFilePretty setting (#837)

* Flow lib now adheres to pretty flow file, ui also adheres to this as well

* added settings mock object in flows_spec get api test

* reverted api changes, fixed parse of flow only

* try spell flows correctly

editor/js/ui/clipboard.js

@@ -119,8 +119,13 @@ RED.clipboard = (function() {
         var selection = RED.view.selection();
         if (selection.nodes) {
             var nns = RED.nodes.createExportableNodeSet(selection.nodes);
+            if (RED.settings.flowFilePretty) {
+                nns = JSON.stringify(nns,null,4);
+            } else {
+                nns = JSON.stringify(nns);
+            }
             $("#clipboard-export")
-                .val(JSON.stringify(nns))
+                .val(nns)
                 .focus(function() {
                     var textarea = $(this);
                     textarea.select();

red/api/info.js

@@ -37,6 +37,10 @@ module.exports = {
             safeSettings.paletteCategories = settings.paletteCategories;
         }
 
+        if (settings.flowFilePretty) {
+            safeSettings.flowFilePretty = settings.flowFilePretty;
+        }
+
         res.json(safeSettings);
     }
 }

red/runtime/storage/localfilesystem.js

@@ -390,6 +390,9 @@ var localfilesystem = {
                 headers += "// "+i+": "+meta[i]+"\n";
             }
         }
+        if (type === "flows" && settings.flowFilePretty) {
+            body = JSON.stringify(JSON.parse(body),null,4);
+        }
         return promiseDir(fspath.dirname(fn)).then(function () {
             writeFile(fn,headers+body);
         });

test/red/api/flows_spec.js

@@ -36,6 +36,7 @@ describe("flows api", function() {
 
     it('returns flow', function(done) {
         flows.init({
+            settings: {},
             log:{warn:function(){},_:function(){},audit:function(){}},
             nodes:{
                 getFlows: function() { return [1,2,3]; }