Fixed regression bug with combination of splitStrings and renameProperties option

https://github.com/javascript-obfuscator/javascript-obfuscator/issues/729

(cherry picked from commit a0dca0256b)

CHANGELOG.md

@@ -1,5 +1,9 @@
 Change Log
 
+v1.12.1
+---
+* Fixed regression bug with combination of `splitStrings` and `renameProperties` option. https://github.com/javascript-obfuscator/javascript-obfuscator/issues/729
+
 v1.12.0
 ---
 * **New option:** `optionsPreset` allows to set options preset

src/JavaScriptObfuscator.ts

@@ -217,12 +217,11 @@ export class JavaScriptObfuscator implements IJavaScriptObfuscator {
             astTree = this.runNodeTransformationStage(astTree, NodeTransformationStage.ControlFlowFlattening);
         }
 
-        astTree = this.runNodeTransformationStage(astTree, NodeTransformationStage.Converting);
-
         if (this.options.renameProperties) {
             astTree = this.runNodeTransformationStage(astTree, NodeTransformationStage.RenameProperties);
         }
 
+        astTree = this.runNodeTransformationStage(astTree, NodeTransformationStage.Converting);
         astTree = this.runNodeTransformationStage(astTree, NodeTransformationStage.RenameIdentifiers);
         astTree = this.runNodeTransformationStage(astTree, NodeTransformationStage.StringArray);
 

test/functional-tests/node-transformers/rename-properties-transformers/rename-properties-transformer/RenamePropertiesTransformer.spec.ts

@@ -294,7 +294,12 @@ describe('RenamePropertiesTransformer', () => {
             });
 
             describe('Variant #8: integration with `splitStrings` option', () => {
-                const propertyRegExp: RegExp = /'a': *'long' *\+ *'Prop' *\+ *'erty' *\+ *'Valu' *\+ *'e'/;
+                const propertyRegExp: RegExp = new RegExp(
+                    'const foo *= *{' +
+                        '\'a\': *\'long\' *\\+ *\'Prop\' *\\+ *\'erty\' *\\+ *\'Valu\' *\\+ *\'e\'' +
+                    '};' +
+                    'foo\\[\'a\'];'
+                );
 
                 let obfuscatedCode: string;
 

test/functional-tests/node-transformers/rename-properties-transformers/rename-properties-transformer/fixtures/split-strings-integration.js

@@ -1,3 +1,4 @@
 const foo = {
     longPropertyName: 'longPropertyValue'
 };
+foo.longPropertyName;