Do not add quotes to object names if not necessary when reorganising (#…

…282) Co-authored-by: James Pearson <james.pearson@norriq.com>
waldo1001 · Jul 12, 2023 · 364ca34 · 364ca34
1 parent 2120036
commit 364ca34
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/src/NAVObject.ts b/src/NAVObject.ts
@@ -448,10 +448,19 @@ export class NAVObject {
         if (objectText.indexOf("\"" + this.objectName + "\"") >= 0) {
             return objectText.replace(searchPattern, fixedObjectName);
         } else {
-            return objectText.replace(searchPattern, "\"" + fixedObjectName + "\"");
+            if (this.objectNameRequiresQuotes(fixedObjectName)) {
+                return objectText.replace(searchPattern, "\"" + fixedObjectName + "\"");
+            }
+            else {
+                return objectText.replace(searchPattern, fixedObjectName);
+            }
         }
     }
 
+    private objectNameRequiresQuotes(objectName: string): boolean {
+        return !(/^[A-Za-z0-9]+$/.test(objectName));
+    }
+
     private escapeRegExp(str) {
         // Ref. https://stackoverflow.com/questions/3446170/escape-string-for-use-in-javascript-regex
         return str.replace(/[\-\[\]\/\{\}\(\)\*\+\?\.\\\^\$\|]/g, "\\$&");