Fixed incorrect variable declarations in some cases when there is no initializer (#952)

Perryvw · web-flow · commit 237a78835464 · 2020-12-21T15:32:15.000+01:00
diff --git a/src/transformation/utils/lua-ast.ts b/src/transformation/utils/lua-ast.ts
@@ -176,7 +176,9 @@ export function createLocalOrExportedOrGlobalDeclaration(
             if (scope.type === ScopeType.Switch || (!isFunctionDeclaration && hasMultipleReferences(scope, lhs))) {
                 // Split declaration and assignment of identifiers that reference themselves in their declaration
                 declaration = lua.createVariableDeclarationStatement(lhs, undefined, tsOriginal);
-                assignment = lua.createAssignmentStatement(lhs, rhs, tsOriginal);
+                if (rhs) {
+                    assignment = lua.createAssignmentStatement(lhs, rhs, tsOriginal);
+                }
             } else {
                 declaration = lua.createVariableDeclarationStatement(lhs, rhs, tsOriginal);
             }
diff --git a/test/unit/hoisting.spec.ts b/test/unit/hoisting.spec.ts
@@ -254,3 +254,14 @@ test("Hoisting Shorthand Property", () => {
         return foo();`;
     expect(util.transpileAndExecute(code)).toBe("foobar");
 });
+
+// https://github.com/TypeScriptToLua/TypeScriptToLua/issues/944
+test("Hoisting variable without initializer", () => {
+    util.testFunction`
+        function foo() {
+            return x;
+        }
+        let x: number;
+        return foo();
+    `.expectToMatchJsResult();
+});
