Merge pull request #205 from Aliyss/patch-1

Static flatten and Code Cleanup

Author: eldadfux

templates/node/lib/client.js.twig

@@ -88,7 +88,7 @@ class Client {
         if (contentType.startsWith('multipart/form-data')) {
             const form = new FormData();
 
-            let flatParams = this.flatten(params);
+            let flatParams = Client.flatten(params);
 
             for (const key in flatParams) {
                 form.append(key, flatParams[key])
@@ -122,26 +122,25 @@ class Client {
                     } else {
                         throw new {{spec.title | caseUcfirst}}Exception(error.response.data.message, error.response.status, error.response.data);
                     }
-                }else{
+                } else {
                     throw new {{spec.title | caseUcfirst}}Exception(error.response.statusText, error.response.status, error.response.data);
                 }
-            }else{
+            } else {
                 throw new {{spec.title | caseUcfirst}}Exception(error.message);
             }
         }
     }
 
-    flatten(data, prefix = '') {
+    static flatten(data, prefix = '') {
         let output = {};
 
         for (const key in data) {
             let value = data[key];
             let finalKey = prefix ? prefix + '[' + key +']' : key;
 
             if (Array.isArray(value)) {
-                output = Object.assign(output, this.flatten(value, finalKey)); // @todo: handle name collision here if needed
-            }
-            else {
+                output = Object.assign(output, Client.flatten(value, finalKey)); // @todo: handle name collision here if needed
+            } else {
                 output[finalKey] = value;
             }
         }
@@ -150,4 +149,4 @@ class Client {
     }
 }
 
-module.exports = Client;
+module.exports = Client;