fix: change PrimaryColumn decorator to clone passed options (typeorm#…

…4571) Closes: typeorm#4570
haggholm · Sep 5, 2019 · 3cf470d · 3cf470d
1 parent 9e3d664
commit 3cf470d
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/src/decorator/columns/PrimaryColumn.ts b/src/decorator/columns/PrimaryColumn.ts
@@ -31,7 +31,7 @@ export function PrimaryColumn(typeOrOptions?: ColumnType|ColumnOptions, options?
         if (typeof typeOrOptions === "string") {
             type = <ColumnType> typeOrOptions;
         } else {
-            options = <ColumnOptions> typeOrOptions;
+            options = Object.assign({}, <ColumnOptions> typeOrOptions);
         }
         if (!options) options = {} as ColumnOptions;
 

diff --git a/test/github-issues/4570/issue-4570.ts b/test/github-issues/4570/issue-4570.ts
@@ -0,0 +1,19 @@
+import "reflect-metadata";
+
+import {expect} from "chai";
+import {ColumnOptions, PrimaryColumn} from "../../../src";
+
+describe("github issues > #4570 Fix PrimaryColumn decorator modifies passed option", () => {
+    it("should not modify passed options to PrimaryColumn", () => {
+        const options: ColumnOptions = {type: "varchar" };
+        const clone = Object.assign({}, options);
+
+        class Entity {
+            @PrimaryColumn(options)
+            pkey: string;
+        }
+
+        expect(Entity).to.be;
+        expect(clone).to.be.eql(options);
+    });
+});