- Fixed bug when settings were not registered until app restart.

- Fixed issue with HOME "~/" paths.
- Changed code flow. Splits everything to modules for easier maintenance and development. Also makes it easier to other apps to adopt the functionality.

Resolves #1 and #2.

Author: rendertom

CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to the Adobe Script Runner extension will be documented in t
 
 ## [Unreleased]
 
+- Fixed bug when settings were not registered until app restart.
+- Fixed issue with HOME "~/" paths.
+- Changed code flow. Splits everything to modules for easier maintenance and development. Also makes it easier to other apps to adopt the functionality.
+
 ## Released
 
 ## [0.1.2] 2018-03-14

extension.js

@@ -0,0 +1,7 @@
+const vscode = require('vscode');
+
+function update() {
+	return vscode.workspace.getConfiguration('adobeScriptRunner');
+}
+
+module.exports.update = update;

lib/config.js

@@ -0,0 +1,29 @@
+const vscode = require('vscode');
+
+const editor = {
+    getActiveTextEditor() {
+        return vscode.window.activeTextEditor;
+    },
+    
+    hasPath(activeTextEditor) {
+        return !activeTextEditor.document.isUntitled;
+    },
+    
+    isDirty(activeTextEditor) {
+        return activeTextEditor.document.isDirty && this.hasPath(activeTextEditor);
+    },
+    
+    getText(activeTextEditor) {
+        return activeTextEditor.document.getText();
+    },
+    
+    getPath(activeTextEditor) {
+        return activeTextEditor.document.fileName;
+    },
+    
+    save(activeTextEditor) {
+        activeTextEditor.document.save();
+    }
+}
+
+module.exports = editor;

lib/editor.js

@@ -0,0 +1,19 @@
+const buildCommand = require('./static/buildCommand.js');
+const hostApps = require('./static/hostApps.js');
+const vscode = require('vscode');
+
+/**
+ * @description Method is called when extension is activated.
+ *              Extension is activated the very first time the command is executed.
+ * @param {any} context vscode.ExtensionContext
+ */
+function activate(context) {
+	hostApps.forEach(hostApp => {
+		vscode.commands.registerCommand(
+			`adobeScriptRunner.${hostApp.shortName}`,
+			() => buildCommand(hostApp)
+		);
+	});
+}
+
+exports.activate = activate;

lib/extension.js

@@ -0,0 +1,16 @@
+const vscode = require('vscode');
+const message = {
+    success (message) {
+        vscode.window.showInformationMessage(message);
+    },
+    
+    info (message) {
+        vscode.window.showWarningMessage(message);
+    },
+    
+    error (message) {
+        vscode.window.showErrorMessage(string);
+    }
+}
+
+module.exports = message;

lib/message.js

@@ -0,0 +1,38 @@
+const cp = require('child_process');
+const getScriptFile = require('./getScriptFile.js');
+const getShellCommand = require('./getShellCommand.js');
+
+const config = require('../config.js');
+const message = require('../message.js');
+
+/**
+ * @description Implementation of the activation command.
+ * @param {any} hostApp Object, entry from hostApps[hostApp].
+ * @returns {boolean} Nothing on success. 'null' on error.
+ */
+function buildCommand(hostApp) {
+	try {
+		const settings = config.update();
+		const scriptFile = getScriptFile(settings);
+		const command = getShellCommand(hostApp, scriptFile, settings);
+
+		console.log('Running shell command:', command);
+		cp.exec(command, (err, result, raw) => {
+			if (err) {
+				return console.error(err);
+			}
+			console.log(result);
+			console.log(raw);
+		});
+
+		message.success(`Script sent to ${hostApp.appName}`);
+
+	} catch (err) {
+		if (err) {
+			message.error(err.toString());
+			console.log(err);
+		}
+	}
+}
+
+module.exports = buildCommand;