@contember/schema-migrations 1.2.0-alpha.10 → 1.2.0-alpha.11

package/src/modifications/columns/UpdateColumnNameModification.ts

@@ -15,13 +15,13 @@ export class UpdateColumnNameModificationHandler implements ModificationHandler<
 			return
 		}
 		acceptFieldVisitor(this.schema.model, entity, this.data.fieldName, {
-			visitColumn: (entity, column) => {
+			visitColumn: ({ entity, column }) => {
 				builder.renameColumn(entity.tableName, column.columnName, this.data.columnName)
 			},
-			visitManyHasOne: (entity, relation) => {
+			visitManyHasOne: ({ entity, relation }) => {
 				builder.renameColumn(entity.tableName, relation.joiningColumn.columnName, this.data.columnName)
 			},
-			visitOneHasOneOwning: (entity, relation) => {
+			visitOneHasOneOwning: ({ entity, relation }) => {
 				builder.renameColumn(entity.tableName, relation.joiningColumn.columnName, this.data.columnName)
 			},
 			visitManyHasManyInverse: () => {
@@ -43,47 +43,47 @@ export class UpdateColumnNameModificationHandler implements ModificationHandler<
 		return updateModel(
 			updateEntity(
 				this.data.entityName,
-				updateField(this.data.fieldName, ({
-													  field,
-													  entity,
-												  }) => acceptFieldVisitor<Model.AnyField>(this.schema.model, entity, field, {
-					visitColumn: (entity, column) => {
-						return {
-							...column,
-							columnName: this.data.columnName,
-						}
-					},
-					visitManyHasOne: (entity, relation) => {
-						return {
-							...relation,
-							joiningColumn: {
-								...relation.joiningColumn,
-								columnName: this.data.columnName,
-							},
-						}
-					},
-					visitOneHasOneOwning: (entity, relation) => {
-						return {
-							...relation,
-							joiningColumn: {
-								...relation.joiningColumn,
+				updateField(this.data.fieldName, ({ field, entity }) =>
+					acceptFieldVisitor<Model.AnyField>(this.schema.model, entity, field, {
+						visitColumn: ({ column }) => {
+							return {
+								...column,
 								columnName: this.data.columnName,
-							},
-						}
-					},
-					visitManyHasManyInverse: () => {
-						throw new Error('Cannot rename column of many-to-many relation')
-					},
-					visitManyHasManyOwning: () => {
-						throw new Error('Cannot rename column of many-to-many relation')
-					},
-					visitOneHasMany: () => {
-						throw new Error('Cannot rename column of one-to-many relation')
-					},
-					visitOneHasOneInverse: () => {
-						throw new Error('Cannot rename column of one-to-one inverse relation')
+							}
+						},
+						visitManyHasOne: ({ entity, relation }) => {
+							return {
+								...relation,
+								joiningColumn: {
+									...relation.joiningColumn,
+									columnName: this.data.columnName,
+								},
+							}
+						},
+						visitOneHasOneOwning: ({ entity, relation }) => {
+							return {
+								...relation,
+								joiningColumn: {
+									...relation.joiningColumn,
+									columnName: this.data.columnName,
+								},
+							}
+						},
+						visitManyHasManyInverse: () => {
+							throw new Error('Cannot rename column of many-to-many relation')
+						},
+						visitManyHasManyOwning: () => {
+							throw new Error('Cannot rename column of many-to-many relation')
+						},
+						visitOneHasMany: () => {
+							throw new Error('Cannot rename column of one-to-many relation')
+						},
+						visitOneHasOneInverse: () => {
+							throw new Error('Cannot rename column of one-to-one inverse relation')
+						},
 					},
-				})),
+					),
+				),
 			),
 		)
 	}

package/src/modifications/constraints/CreateUniqueConstraintModification.ts

@@ -16,10 +16,10 @@ export class CreateUniqueConstraintModificationHandler implements ModificationHa
 		const fields = this.data.unique.fields
 		const columns = fields.map(fieldName => {
 			return acceptFieldVisitor(this.schema.model, entity, fieldName, {
-				visitColumn: ({}, column) => {
+				visitColumn: ({ column }) => {
 					return column.columnName
 				},
-				visitManyHasOne: ({}, relation) => {
+				visitManyHasOne: ({ relation }) => {
 					return relation.joiningColumn.columnName
 				},
 				visitOneHasMany: () => {

package/src/modifications/fields/RemoveFieldModification.ts

@@ -19,19 +19,19 @@ export class RemoveFieldModificationHandler implements ModificationHandler<Remov
 			return
 		}
 		acceptFieldVisitor(this.schema.model, this.data.entityName, this.data.fieldName, {
-			visitColumn: (entity, column) => {
+			visitColumn: ({ entity, column }) => {
 				builder.dropColumn(entity.tableName, column.columnName)
 			},
-			visitManyHasOne: (entity, relation, {}, _) => {
+			visitManyHasOne: ({ entity, relation }) => {
 				builder.dropColumn(entity.tableName, relation.joiningColumn.columnName)
 			},
 			visitOneHasMany: () => {},
-			visitOneHasOneOwning: (entity, relation, {}, _) => {
+			visitOneHasOneOwning: ({ entity, relation }) => {
 				builder.dropConstraint(entity.tableName, NamingHelper.createUniqueConstraintName(entity.name, [relation.name]))
 				builder.dropColumn(entity.tableName, relation.joiningColumn.columnName)
 			},
 			visitOneHasOneInverse: () => {},
-			visitManyHasManyOwning: ({}, relation, {}, _) => {
+			visitManyHasManyOwning: ({ relation }) => {
 				builder.dropTable(relation.joiningTable.tableName)
 			},
 			visitManyHasManyInverse: () => {},

package/src/modifications/fields/UpdateFieldNameModification.ts

@@ -79,33 +79,33 @@ export class UpdateFieldNameModificationHandler implements ModificationHandler<U
 				}
 
 				return acceptFieldVisitor<Model.AnyField>(this.schema.model, entity, field, {
-					visitColumn: (entity, field) => field,
-					visitManyHasOne: (entity, relation, targetEntity, targetRelation) => {
+					visitColumn: ({ column }) => column,
+					visitManyHasOne: ({ targetEntity, relation, targetRelation }) => {
 						return isUpdatedRelation(targetEntity, targetRelation)
 							? { ...relation, inversedBy: this.data.newFieldName }
 							: relation
 					},
-					visitOneHasMany: (entity, relation, targetEntity, targetRelation) => {
+					visitOneHasMany: ({ relation, targetEntity, targetRelation }) => {
 						return isUpdatedRelation(targetEntity, targetRelation)
 							? { ...relation, ownedBy: this.data.newFieldName }
 							: relation
 					},
-					visitOneHasOneOwning: (entity, relation, targetEntity, targetRelation) => {
+					visitOneHasOneOwning: ({ relation, targetEntity, targetRelation }) => {
 						return isUpdatedRelation(targetEntity, targetRelation)
 							? { ...relation, inversedBy: this.data.newFieldName }
 							: relation
 					},
-					visitOneHasOneInverse: (entity, relation, targetEntity, targetRelation) => {
+					visitOneHasOneInverse: ({ relation, targetEntity, targetRelation }) => {
 						return isUpdatedRelation(targetEntity, targetRelation)
 							? { ...relation, ownedBy: this.data.newFieldName }
 							: relation
 					},
-					visitManyHasManyOwning: (entity, relation, targetEntity, targetRelation) => {
+					visitManyHasManyOwning: ({ relation, targetEntity, targetRelation }) => {
 						return isUpdatedRelation(targetEntity, targetRelation)
 							? { ...relation, inversedBy: this.data.newFieldName }
 							: relation
 					},
-					visitManyHasManyInverse: (entity, relation, targetEntity, targetRelation) => {
+					visitManyHasManyInverse: ({ relation, targetEntity, targetRelation }) => {
 						return isUpdatedRelation(targetEntity, targetRelation)
 							? { ...relation, ownedBy: this.data.newFieldName }
 							: relation

package/src/modifications/indexes/CreateIndexModification.ts

@@ -15,13 +15,13 @@ export class CreateIndexModificationHandler implements ModificationHandler<Creat
 		const fields = this.data.index.fields
 		const columns = fields.map(fieldName => {
 			return acceptFieldVisitor(this.schema.model, entity, fieldName, {
-				visitColumn: ({}, column) => {
+				visitColumn: ({ column }) => {
 					return column.columnName
 				},
-				visitManyHasOne: ({}, relation) => {
+				visitManyHasOne: ({ relation }) => {
 					return relation.joiningColumn.columnName
 				},
-				visitOneHasOneOwning: ({}, relation) => {
+				visitOneHasOneOwning: ({ relation }) => {
 					return relation.joiningColumn.columnName
 				},
 				visitOneHasMany: () => {

package/src/modifications/relations/CreateRelationModification.ts

@@ -59,18 +59,18 @@ export class CreateRelationModificationHandler implements ModificationHandler<Cr
 			})
 		}
 		acceptRelationTypeVisitor(this.schema.model, entity, this.getNormalizedOwningSide(), {
-			visitManyHasOne: ({}, relation, {}, _) => {
+			visitManyHasOne: ({ relation }) => {
 				createOwningSide(relation)
 				builder.addIndex(entity.tableName, relation.joiningColumn.columnName)
 			},
 			visitOneHasMany: () => {},
-			visitOneHasOneOwning: ({}, relation, {}, _) => {
+			visitOneHasOneOwning: ({ relation }) => {
 				createOwningSide(relation)
 				const uniqueConstraintName = NamingHelper.createUniqueConstraintName(entity.name, [relation.name])
 				builder.addConstraint(entity.tableName, uniqueConstraintName, { unique: [relation.joiningColumn.columnName] })
 			},
 			visitOneHasOneInverse: () => {},
-			visitManyHasManyOwning: ({}, relation, {}, _) => {
+			visitManyHasManyOwning: ({ relation }) => {
 				createJunctionTableSql(builder, this.options.systemSchema,  entity, targetEntity, relation)
 			},
 			visitManyHasManyInverse: () => {},