From 69797e58cb6fc5d5622c55dbd42fab83de1ad668 Mon Sep 17 00:00:00 2001
From: j433866 <j433866@gmail.com>
Date: Wed, 16 Jan 2019 16:57:58 +0000
Subject: [PATCH] Add better error handling. Also now doesn't do anything if
 there's no input

---
 src/core/operations/ConvertCoordinateFormat.mjs | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/src/core/operations/ConvertCoordinateFormat.mjs b/src/core/operations/ConvertCoordinateFormat.mjs
index 770920f4..09e1620c 100644
--- a/src/core/operations/ConvertCoordinateFormat.mjs
+++ b/src/core/operations/ConvertCoordinateFormat.mjs
@@ -5,6 +5,7 @@
  */
 
 import Operation from "../Operation";
+import OperationError from "../errors/OperationError";
 import {FORMATS, convertCoordinates} from "../lib/ConvertCoordinates";
 
 /**
@@ -37,7 +38,6 @@ class ConvertCoordinateFormat extends Operation {
                     "Auto",
                     "Direction Preceding",
                     "Direction Following",
-                    "Space",
                     "\\n",
                     "Comma",
                     "Semi-colon",
@@ -83,8 +83,17 @@ class ConvertCoordinateFormat extends Operation {
      * @returns {string}
      */
     run(input, args) {
-        const [inFormat, inDelim, outFormat, outDelim, incDirection, precision] = args;
-        return convertCoordinates(input, inFormat, inDelim, outFormat, outDelim, incDirection, precision);
+        if (input.replace(/[\s+]/g, "") !== "") {
+            const [inFormat, inDelim, outFormat, outDelim, incDirection, precision] = args;
+            try {
+                const result = convertCoordinates(input, inFormat, inDelim, outFormat, outDelim, incDirection, precision);
+                return result;
+            } catch (error) {
+                throw new OperationError(error);
+            }
+        } else {
+            return input;
+        }
     }
 }