fix(entity-generator): fix boolean default values

Closes #1917
mikro-orm · Jun 29, 2021 · 219fc0c · 219fc0c
1 parent ed399b1
commit 219fc0c
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/packages/entity-generator/src/SourceFile.ts b/packages/entity-generator/src/SourceFile.ts
@@ -62,7 +62,7 @@ export class SourceFile {
 
   private getPropertyDefinition(prop: EntityProperty, padLeft: number): string {
     // string defaults are usually things like SQL functions
-    const useDefault = prop.default && typeof prop.default !== 'string';
+    const useDefault = prop.default != null && typeof prop.default !== 'string';
     const optional = prop.nullable ? '?' : (useDefault ? '' : '!');
     const ret = `${prop.name}${optional}: ${prop.type}`;
     const padding = ' '.repeat(padLeft);

diff --git a/packages/knex/src/schema/DatabaseTable.ts b/packages/knex/src/schema/DatabaseTable.ts
@@ -168,7 +168,7 @@ export class DatabaseTable {
     }
 
     if (propType === 'boolean') {
-      return !!column.defaultValue;
+      return !['0', 'false', 'f', 'n', 'no', 'off'].includes('' + column.defaultValue);
     }
 
     if (propType === 'number') {

diff --git a/tests/__snapshots__/EntityGenerator.test.ts.snap b/tests/__snapshots__/EntityGenerator.test.ts.snap
@@ -47,7 +47,7 @@ export class Author2 {
 
   @Index({ name: 'author2_terms_accepted_index' })
   @Property()
-  termsAccepted: boolean = true;
+  termsAccepted: boolean = false;
 
   @Property({ nullable: true })
   optional?: boolean;
@@ -560,7 +560,7 @@ export class Author2 {
 
   @Index({ name: 'author2_terms_accepted_index' })
   @Property()
-  termsAccepted: boolean = true;
+  termsAccepted: boolean = false;
 
   @Property({ nullable: true })
   optional?: boolean;
@@ -917,7 +917,7 @@ export class Author3 {
   age?: number;
 
   @Property()
-  termsAccepted!: number;
+  termsAccepted: number = 0;
 
   @Property({ columnType: 'text', nullable: true })
   identities?: string;