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

package/src/modifications/relations/ConvertOneToManyRelationModification.ts

@@ -1,14 +1,18 @@
 import { MigrationBuilder } from '@contember/database-migrations'
 import { Model, Schema } from '@contember/schema'
 import { SchemaUpdater, updateEntity, updateField, updateModel, updateSchema } from '../utils/schemaUpdateUtils'
-import { ModificationHandler, ModificationHandlerOptions, ModificationHandlerStatic } from '../ModificationHandler'
+import {
+	createModificationType,
+	Differ,
+	ModificationHandler,
+	ModificationHandlerOptions,
+} from '../ModificationHandler'
 import { isOwningRelation, NamingHelper } from '@contember/schema-utils'
 import { updateRelations } from '../utils/diffUtils'
-import { UpdateFieldNameModification } from '../fields'
 import { NoopModification } from '../NoopModification'
+import { updateFieldNameModification } from '../fields'
 
-export const ConvertOneToManyRelationModification: ModificationHandlerStatic<ConvertOneToManyRelationModificationData> = class {
-	static id = 'convertOneToManyRelation'
+export class ConvertOneToManyRelationModificationHandler implements ModificationHandler<ConvertOneToManyRelationModificationData> {
 	private subModification: ModificationHandler<any>
 
 	constructor(
@@ -18,7 +22,7 @@ export const ConvertOneToManyRelationModification: ModificationHandlerStatic<Con
 	) {
 		const { relation } = this.getRelation()
 		this.subModification = data.newInverseSideFieldName && relation.inversedBy ?
-			new UpdateFieldNameModification(
+			updateFieldNameModification.createHandler(
 				{
 					entityName: relation.target,
 					fieldName: relation.inversedBy,
@@ -77,11 +81,29 @@ export const ConvertOneToManyRelationModification: ModificationHandlerStatic<Con
 		}
 	}
 
-	static createModification(data: ConvertOneToManyRelationModificationData) {
-		return { modification: this.id, ...data }
+	private getRelation() {
+		const entity = this.schema.model.entities[this.data.entityName]
+		const relation = entity.fields[this.data.fieldName]
+		if (relation.type !== Model.RelationType.OneHasOne || !isOwningRelation(relation)) {
+			throw new Error()
+		}
+		return { entity, relation }
 	}
+}
+
+export interface ConvertOneToManyRelationModificationData {
+	entityName: string
+	fieldName: string
+	newInverseSideFieldName?: string
+}
+
+export const convertOneToManyRelationModification = createModificationType({
+	id: 'convertOneToManyRelation',
+	handler: ConvertOneToManyRelationModificationHandler,
+})
 
-	static createDiff(originalSchema: Schema, updatedSchema: Schema) {
+export class ConvertOneToManyRelationDiffer implements Differ {
+	createDiff(originalSchema: Schema, updatedSchema: Schema) {
 		return updateRelations(originalSchema, updatedSchema, ({ originalRelation, updatedRelation, updatedEntity }) => {
 			if (
 				isOwningRelation(originalRelation) &&
@@ -90,7 +112,7 @@ export const ConvertOneToManyRelationModification: ModificationHandlerStatic<Con
 				updatedRelation.type === Model.RelationType.ManyHasOne
 			) {
 				const isInverseSideRenamed = originalRelation.inversedBy && updatedRelation.inversedBy && originalRelation.inversedBy !== updatedRelation.inversedBy
-				return ConvertOneToManyRelationModification.createModification({
+				return convertOneToManyRelationModification.createModification({
 					entityName: updatedEntity.name,
 					fieldName: updatedRelation.name,
 					...(isInverseSideRenamed ? {
@@ -101,19 +123,4 @@ export const ConvertOneToManyRelationModification: ModificationHandlerStatic<Con
 			return undefined
 		})
 	}
-
-	private getRelation() {
-		const entity = this.schema.model.entities[this.data.entityName]
-		const relation = entity.fields[this.data.fieldName]
-		if (relation.type !== Model.RelationType.OneHasOne || !isOwningRelation(relation)) {
-			throw new Error()
-		}
-		return { entity, relation }
-	}
-}
-
-export interface ConvertOneToManyRelationModificationData {
-	entityName: string
-	fieldName: string
-	newInverseSideFieldName?: string
 }

package/src/modifications/relations/CreateRelationInverseSideModification.ts

@@ -1,12 +1,11 @@
 import { MigrationBuilder } from '@contember/database-migrations'
 import { Model, Schema } from '@contember/schema'
 import { addField, SchemaUpdater, updateEntity, updateField, updateModel } from '../utils/schemaUpdateUtils'
-import { ModificationHandlerStatic } from '../ModificationHandler'
+import { createModificationType, Differ, ModificationHandler } from '../ModificationHandler'
 import { createFields } from '../utils/diffUtils'
 import { isInverseRelation, isRelation } from '@contember/schema-utils'
 
-export const CreateRelationInverseSideModification: ModificationHandlerStatic<CreateRelationInverseSideModificationData> = class {
-	static id = 'createRelationInverseSide'
+export class CreateRelationInverseSideModificationHandler implements ModificationHandler<CreateRelationInverseSideModificationData> {
 	constructor(private readonly data: CreateRelationInverseSideModificationData, private readonly schema: Schema) {}
 
 	public createSql(builder: MigrationBuilder): void {}
@@ -27,25 +26,28 @@ export const CreateRelationInverseSideModification: ModificationHandlerStatic<Cr
 	describe() {
 		return { message: `Add relation ${this.data.entityName}.${this.data.relation.name}` }
 	}
+}
 
-	static createModification(data: CreateRelationInverseSideModificationData) {
-		return { modification: this.id, ...data }
-	}
+export interface CreateRelationInverseSideModificationData {
+	entityName: string
+	relation: Model.AnyRelation & Model.InverseRelation
+}
 
-	static createDiff(originalSchema: Schema, updatedSchema: Schema) {
+export const createRelationInverseSideModification = createModificationType({
+	id: 'createRelationInverseSide',
+	handler: CreateRelationInverseSideModificationHandler,
+})
+
+export class CreateRelationInverseSideDiffer implements Differ {
+	createDiff(originalSchema: Schema, updatedSchema: Schema) {
 		return createFields(originalSchema, updatedSchema, ({ newField, updatedEntity }) => {
 			if (!isRelation(newField) || !isInverseRelation(newField)) {
 				return undefined
 			}
-			return CreateRelationInverseSideModification.createModification({
+			return createRelationInverseSideModification.createModification({
 				entityName: updatedEntity.name,
 				relation: newField,
 			})
 		})
 	}
 }
-
-export interface CreateRelationInverseSideModificationData {
-	entityName: string
-	relation: Model.AnyRelation & Model.InverseRelation
-}

package/src/modifications/relations/CreateRelationModification.ts

@@ -8,7 +8,12 @@ import {
 } from '@contember/schema-utils'
 import { MigrationBuilder } from '@contember/database-migrations'
 import { addField, SchemaUpdater, updateEntity, updateModel } from '../utils/schemaUpdateUtils'
-import { ModificationHandlerOptions, ModificationHandlerStatic } from '../ModificationHandler'
+import {
+	createModificationType,
+	Differ,
+	ModificationHandler,
+	ModificationHandlerOptions,
+} from '../ModificationHandler'
 import { isIt } from '../../utils/isIt'
 import { createFields } from '../utils/diffUtils'
 import { getPrimaryColumnType } from '../utils/getPrimaryColumnType'
@@ -16,8 +21,7 @@ import { createJunctionTableSql } from '../utils/createJunctionTable'
 import { normalizeManyHasManyRelation, PartialManyHasManyRelation } from './normalization'
 
 
-export const CreateRelationModification: ModificationHandlerStatic<CreateRelationModificationData> = class {
-	static id = 'createRelation'
+export class CreateRelationModificationHandler implements ModificationHandler<CreateRelationModificationData> {
 
 	constructor(
 		private readonly data: CreateRelationModificationData,
@@ -55,18 +59,18 @@ export const CreateRelationModification: ModificationHandlerStatic<CreateRelatio
 			})
 		}
 		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: () => {},
@@ -91,11 +95,32 @@ export const CreateRelationModification: ModificationHandlerStatic<CreateRelatio
 		return { message: `Add relation ${this.data.entityName}.${this.data.owningSide.name}`, failureWarning }
 	}
 
-	static createModification(data: CreateRelationModificationData) {
-		return { modification: this.id, ...data }
+
+	private getNormalizedOwningSide(): Model.AnyOwningRelation {
+		if (this.data.owningSide.type === Model.RelationType.ManyHasMany) {
+			return normalizeManyHasManyRelation(this.data.owningSide)
+		}
+		return this.data.owningSide
 	}
+}
+
+
+export type CreateRelationModificationData = {
+	entityName: string
+	owningSide:
+		| Model.ManyHasOneRelation
+		| Model.OneHasOneOwningRelation
+		| PartialManyHasManyRelation
+	inverseSide?: Model.AnyInverseRelation
+}
 
-	static createDiff(originalSchema: Schema, updatedSchema: Schema) {
+export const createRelationModification = createModificationType({
+	id: 'createRelation',
+	handler: CreateRelationModificationHandler,
+})
+
+export class CreateRelationDiffer implements Differ {
+	createDiff(originalSchema: Schema, updatedSchema: Schema) {
 		return createFields(originalSchema, updatedSchema, ({ newField, updatedEntity }) => {
 			if (!isRelation(newField) || !isOwningRelation(newField)) {
 				return undefined
@@ -106,28 +131,11 @@ export const CreateRelationModification: ModificationHandlerStatic<CreateRelatio
 			if (inverseSide && !(isRelation(inverseSide) && isInverseRelation(inverseSide))) {
 				throw new Error()
 			}
-			return CreateRelationModification.createModification({
+			return createRelationModification.createModification({
 				entityName: updatedEntity.name,
 				owningSide: newField,
 				...(inverseSide ? { inverseSide } : {}),
 			})
 		})
 	}
-
-	private getNormalizedOwningSide(): Model.AnyOwningRelation {
-		if (this.data.owningSide.type === Model.RelationType.ManyHasMany) {
-			return normalizeManyHasManyRelation(this.data.owningSide)
-		}
-		return this.data.owningSide
-	}
-}
-
-
-export type CreateRelationModificationData = {
-	entityName: string
-	owningSide:
-		| Model.ManyHasOneRelation
-		| Model.OneHasOneOwningRelation
-		| PartialManyHasManyRelation
-	inverseSide?: Model.AnyInverseRelation
 }

package/src/modifications/relations/DisableOrphanRemovalModification.ts

@@ -1,12 +1,11 @@
 import { MigrationBuilder } from '@contember/database-migrations'
 import { Model, Schema } from '@contember/schema'
 import { SchemaUpdater, updateEntity, updateField, updateModel } from '../utils/schemaUpdateUtils'
-import { ModificationHandlerStatic } from '../ModificationHandler'
+import { createModificationType, Differ, ModificationHandler } from '../ModificationHandler'
 import { isOwningRelation } from '@contember/schema-utils'
 import { updateRelations } from '../utils/diffUtils'
 
-export const DisableOrphanRemovalModification: ModificationHandlerStatic<DisableOrphanRemovalModificationData> = class {
-	static id = 'disableOrphanRemoval'
+export class DisableOrphanRemovalModificationHandler implements ModificationHandler<DisableOrphanRemovalModificationData> {
 
 	constructor(private readonly data: DisableOrphanRemovalModificationData, private readonly schema: Schema) {}
 
@@ -27,12 +26,20 @@ export const DisableOrphanRemovalModification: ModificationHandlerStatic<Disable
 			message: `Disable orphan removal on ${this.data.entityName}.${this.data.fieldName}`,
 		}
 	}
+}
 
-	static createModification(data: DisableOrphanRemovalModificationData) {
-		return { modification: this.id, ...data }
-	}
+export interface DisableOrphanRemovalModificationData {
+	entityName: string
+	fieldName: string
+}
 
-	static createDiff(originalSchema: Schema, updatedSchema: Schema) {
+export const disableOrphanRemovalModification = createModificationType({
+	id: 'disableOrphanRemoval',
+	handler: DisableOrphanRemovalModificationHandler,
+})
+
+export class DisableOrphanRemovalDiffer implements Differ {
+	createDiff(originalSchema: Schema, updatedSchema: Schema) {
 		return updateRelations(originalSchema, updatedSchema, ({ originalRelation, updatedRelation, updatedEntity }) => {
 			if (
 				isOwningRelation(originalRelation) &&
@@ -42,7 +49,7 @@ export const DisableOrphanRemovalModification: ModificationHandlerStatic<Disable
 				originalRelation.orphanRemoval &&
 				!updatedRelation.orphanRemoval
 			) {
-				return DisableOrphanRemovalModification.createModification({
+				return disableOrphanRemovalModification.createModification({
 					entityName: updatedEntity.name,
 					fieldName: updatedRelation.name,
 				})
@@ -51,7 +58,3 @@ export const DisableOrphanRemovalModification: ModificationHandlerStatic<Disable
 		})
 	}
 }
-export interface DisableOrphanRemovalModificationData {
-	entityName: string
-	fieldName: string
-}

package/src/modifications/relations/EnableOrphanRemovalModification.ts

@@ -1,12 +1,11 @@
 import { MigrationBuilder } from '@contember/database-migrations'
 import { Model, Schema } from '@contember/schema'
 import { SchemaUpdater, updateEntity, updateField, updateModel } from '../utils/schemaUpdateUtils'
-import { ModificationHandlerStatic } from '../ModificationHandler'
+import { createModificationType, Differ, ModificationHandler } from '../ModificationHandler'
 import { isOwningRelation } from '@contember/schema-utils'
 import { updateRelations } from '../utils/diffUtils'
 
-export const EnableOrphanRemovalModification: ModificationHandlerStatic<EnableOrphanRemovalModificationData> = class {
-	static id = 'enableOrphanRemoval'
+export class EnableOrphanRemovalModificationHandler implements ModificationHandler<EnableOrphanRemovalModificationData> {
 	constructor(private readonly data: EnableOrphanRemovalModificationData, private readonly schema: Schema) {}
 
 	public createSql(builder: MigrationBuilder): void {}
@@ -30,11 +29,20 @@ export const EnableOrphanRemovalModification: ModificationHandlerStatic<EnableOr
 		}
 	}
 
-	static createModification(data: EnableOrphanRemovalModificationData) {
-		return { modification: this.id, ...data }
-	}
+}
+
+export interface EnableOrphanRemovalModificationData {
+	entityName: string
+	fieldName: string
+}
 
-	static createDiff(originalSchema: Schema, updatedSchema: Schema) {
+export const enableOrphanRemovalModification = createModificationType({
+	id: 'enableOrphanRemoval',
+	handler: EnableOrphanRemovalModificationHandler,
+})
+
+export class EnableOrphanRemovalDiffer implements Differ {
+	 createDiff(originalSchema: Schema, updatedSchema: Schema) {
 		return updateRelations(originalSchema, updatedSchema, ({ originalRelation, updatedRelation, updatedEntity }) => {
 			if (
 				isOwningRelation(originalRelation) &&
@@ -44,7 +52,7 @@ export const EnableOrphanRemovalModification: ModificationHandlerStatic<EnableOr
 				!originalRelation.orphanRemoval &&
 				updatedRelation.orphanRemoval
 			) {
-				return EnableOrphanRemovalModification.createModification({
+				return enableOrphanRemovalModification.createModification({
 					entityName: updatedEntity.name,
 					fieldName: updatedRelation.name,
 				})
@@ -53,8 +61,3 @@ export const EnableOrphanRemovalModification: ModificationHandlerStatic<EnableOr
 		})
 	}
 }
-
-export interface EnableOrphanRemovalModificationData {
-	entityName: string
-	fieldName: string
-}

package/src/modifications/relations/MakeRelationNotNullModification.ts

@@ -1,13 +1,12 @@
 import { MigrationBuilder } from '@contember/database-migrations'
 import { Model, Schema } from '@contember/schema'
 import { SchemaUpdater, updateEntity, updateField, updateModel } from '../utils/schemaUpdateUtils'
-import { ModificationHandlerStatic } from '../ModificationHandler'
+import { createModificationType, Differ, ModificationHandler } from '../ModificationHandler'
 import { getEntity, tryGetColumnName } from '@contember/schema-utils'
 import { isIt } from '../../utils/isIt'
 import { updateRelations } from '../utils/diffUtils'
 
-export const MakeRelationNotNullModification: ModificationHandlerStatic<MakeRelationNotNullModificationData> = class {
-	static id = 'makeRelationNotNull'
+export class MakeRelationNotNullModificationHandler implements ModificationHandler<MakeRelationNotNullModificationData> {
 	constructor(private readonly data: MakeRelationNotNullModificationData, private readonly schema: Schema) {}
 
 	public createSql(builder: MigrationBuilder): void {
@@ -43,12 +42,21 @@ export const MakeRelationNotNullModification: ModificationHandlerStatic<MakeRela
 			failureWarning: 'Changing to not-null may fail in runtime',
 		}
 	}
+}
 
-	static createModification(data: MakeRelationNotNullModificationData) {
-		return { modification: this.id, ...data }
-	}
+export interface MakeRelationNotNullModificationData {
+	entityName: string
+	fieldName: string
+	// todo fillValue
+}
 
-	static createDiff(originalSchema: Schema, updatedSchema: Schema) {
+export const makeRelationNotNullModification = createModificationType({
+	id: 'makeRelationNotNull',
+	handler: MakeRelationNotNullModificationHandler,
+})
+
+export class MakeRelationNotNullDiffer implements Differ {
+	createDiff(originalSchema: Schema, updatedSchema: Schema) {
 		return updateRelations(originalSchema, updatedSchema, ({ originalRelation, updatedRelation, updatedEntity }) => {
 			if (
 				originalRelation.type === updatedRelation.type &&
@@ -57,7 +65,7 @@ export const MakeRelationNotNullModification: ModificationHandlerStatic<MakeRela
 				!updatedRelation.nullable &&
 				originalRelation.nullable
 			) {
-				return MakeRelationNotNullModification.createModification({
+				return makeRelationNotNullModification.createModification({
 					entityName: updatedEntity.name,
 					fieldName: updatedRelation.name,
 				})
@@ -66,9 +74,3 @@ export const MakeRelationNotNullModification: ModificationHandlerStatic<MakeRela
 		})
 	}
 }
-
-export interface MakeRelationNotNullModificationData {
-	entityName: string
-	fieldName: string
-	// todo fillValue
-}

package/src/modifications/relations/MakeRelationNullableModification.ts

@@ -1,13 +1,12 @@
 import { MigrationBuilder } from '@contember/database-migrations'
 import { Model, Schema } from '@contember/schema'
 import { SchemaUpdater, updateEntity, updateField, updateModel } from '../utils/schemaUpdateUtils'
-import { ModificationHandlerStatic } from '../ModificationHandler'
+import { createModificationType, Differ, ModificationHandler } from '../ModificationHandler'
 import { getEntity, tryGetColumnName } from '@contember/schema-utils'
 import { isIt } from '../../utils/isIt'
 import { updateRelations } from '../utils/diffUtils'
 
-export const MakeRelationNullableModification: ModificationHandlerStatic<MakeRelationNullableModificationData> = class {
-	static id = 'makeRelationNullable'
+export class MakeRelationNullableModificationHandler implements ModificationHandler<MakeRelationNullableModificationData> {
 	constructor(private readonly data: MakeRelationNullableModificationData, private readonly schema: Schema) {}
 
 	public createSql(builder: MigrationBuilder): void {
@@ -43,11 +42,21 @@ export const MakeRelationNullableModification: ModificationHandlerStatic<MakeRel
 		}
 	}
 
-	static createModification(data: MakeRelationNullableModificationData) {
-		return { modification: this.id, ...data }
-	}
+}
+
+export interface MakeRelationNullableModificationData {
+	entityName: string
+	fieldName: string
+}
 
-	static createDiff(originalSchema: Schema, updatedSchema: Schema) {
+
+export const makeRelationNullableModification = createModificationType({
+	id: 'makeRelationNullable',
+	handler: MakeRelationNullableModificationHandler,
+})
+
+export class MakeRelationNullableDiffer implements Differ {
+	createDiff(originalSchema: Schema, updatedSchema: Schema) {
 		return updateRelations(originalSchema, updatedSchema, ({ originalRelation, updatedRelation, updatedEntity }) => {
 			if (
 				originalRelation.type === updatedRelation.type &&
@@ -56,7 +65,7 @@ export const MakeRelationNullableModification: ModificationHandlerStatic<MakeRel
 				updatedRelation.nullable &&
 				!originalRelation.nullable
 			) {
-				return MakeRelationNullableModification.createModification({
+				return makeRelationNullableModification.createModification({
 					entityName: updatedEntity.name,
 					fieldName: updatedRelation.name,
 				})
@@ -65,8 +74,3 @@ export const MakeRelationNullableModification: ModificationHandlerStatic<MakeRel
 		})
 	}
 }
-
-export interface MakeRelationNullableModificationData {
-	entityName: string
-	fieldName: string
-}

package/src/modifications/relations/{ToggleManyHasManyEventLogModification.ts → ToggleJunctionEventLogModification.ts}

@@ -1,7 +1,12 @@
 import { MigrationBuilder } from '@contember/database-migrations'
 import { Model, Schema } from '@contember/schema'
 import { SchemaUpdater, updateEntity, updateField, updateModel } from '../utils/schemaUpdateUtils'
-import { ModificationHandlerOptions, ModificationHandlerStatic } from '../ModificationHandler'
+import {
+	createModificationType,
+	Differ,
+	ModificationHandler,
+	ModificationHandlerOptions,
+} from '../ModificationHandler'
 import {
 	createEventTrigger,
 	createEventTrxTrigger,
@@ -12,8 +17,7 @@ import { isOwningRelation, isRelation } from '@contember/schema-utils'
 import { updateRelations } from '../utils/diffUtils'
 import { isIt } from '../../utils/isIt'
 
-export const ToggleJunctionEventLogModification: ModificationHandlerStatic<ToggleJunctionEventLogModificationData> = class {
-	static id = 'toggleJunctionEventLog'
+export class ToggleJunctionEventLogModificationHandler implements ModificationHandler<ToggleJunctionEventLogModificationData> {
 
 	constructor(
 		private readonly data: ToggleJunctionEventLogModificationData,
@@ -67,12 +71,21 @@ export const ToggleJunctionEventLogModification: ModificationHandlerStatic<Toggl
 			message: `${this.data.enabled ? 'Enable' : 'Disable'} event log for ${this.data.entityName}.${this.data.fieldName}`,
 		}
 	}
+}
 
-	static createModification(data: ToggleJunctionEventLogModificationData) {
-		return { modification: this.id, ...data }
-	}
+export interface ToggleJunctionEventLogModificationData {
+	entityName: string
+	fieldName: string
+	enabled: boolean
+}
 
-	static createDiff(originalSchema: Schema, updatedSchema: Schema) {
+export const toggleJunctionEventLogModification = createModificationType({
+	id: 'toggleJunctionEventLog',
+	handler: ToggleJunctionEventLogModificationHandler,
+})
+
+export class ToggleJunctionEventLogDiffer implements Differ {
+	 createDiff(originalSchema: Schema, updatedSchema: Schema) {
 		return updateRelations(originalSchema, updatedSchema, ({ originalRelation, updatedRelation, updatedEntity }) => {
 			if (
 				originalRelation.type === updatedRelation.type &&
@@ -82,7 +95,7 @@ export const ToggleJunctionEventLogModification: ModificationHandlerStatic<Toggl
 				const newValue = updatedRelation.joiningTable.eventLog?.enabled ?? false
 				const oldValue = originalRelation.joiningTable.eventLog?.enabled ?? false
 				if (newValue !== oldValue) {
-					return ToggleJunctionEventLogModification.createModification({
+					return toggleJunctionEventLogModification.createModification({
 						entityName: updatedEntity.name,
 						fieldName: updatedRelation.name,
 						enabled: newValue,
@@ -94,9 +107,3 @@ export const ToggleJunctionEventLogModification: ModificationHandlerStatic<Toggl
 
 	}
 }
-
-export interface ToggleJunctionEventLogModificationData {
-	entityName: string
-	fieldName: string
-	enabled: boolean
-}

package/src/modifications/relations/UpdateRelationOnDeleteModification.ts

@@ -1,12 +1,11 @@
 import { MigrationBuilder } from '@contember/database-migrations'
 import { Model, Schema } from '@contember/schema'
 import { SchemaUpdater, updateEntity, updateField, updateModel } from '../utils/schemaUpdateUtils'
-import { ModificationHandlerStatic } from '../ModificationHandler'
+import { createModificationType, Differ, ModificationHandler } from '../ModificationHandler'
 import { isIt } from '../../utils/isIt'
 import { updateRelations } from '../utils/diffUtils'
 
-export const UpdateRelationOnDeleteModification: ModificationHandlerStatic<UpdateRelationOnDeleteModificationData> = class {
-	static id = 'updateRelationOnDelete'
+export class UpdateRelationOnDeleteModificationHandler implements ModificationHandler<UpdateRelationOnDeleteModificationData> {
 	constructor(private readonly data: UpdateRelationOnDeleteModificationData, private readonly schema: Schema) {}
 
 	public createSql(builder: MigrationBuilder): void {}
@@ -27,12 +26,21 @@ export const UpdateRelationOnDeleteModification: ModificationHandlerStatic<Updat
 	describe() {
 		return { message: `Change on-delete policy of relation ${this.data.entityName}.${this.data.fieldName}` }
 	}
+}
 
-	static createModification(data: UpdateRelationOnDeleteModificationData) {
-		return { modification: this.id, ...data }
-	}
+export interface UpdateRelationOnDeleteModificationData {
+	entityName: string
+	fieldName: string
+	onDelete: Model.OnDelete
+}
 
-	static createDiff(originalSchema: Schema, updatedSchema: Schema) {
+export const updateRelationOnDeleteModification = createModificationType({
+	id: 'updateRelationOnDelete',
+	handler: UpdateRelationOnDeleteModificationHandler,
+})
+
+export class UpdateRelationOnDeleteDiffer implements Differ {
+	createDiff(originalSchema: Schema, updatedSchema: Schema) {
 		return updateRelations(originalSchema, updatedSchema, ({ originalRelation, updatedRelation, updatedEntity }) => {
 			if (
 				originalRelation.type === updatedRelation.type &&
@@ -40,7 +48,7 @@ export const UpdateRelationOnDeleteModification: ModificationHandlerStatic<Updat
 				isIt<Model.JoiningColumnRelation>(originalRelation, 'joiningColumn') &&
 				updatedRelation.joiningColumn.onDelete !== originalRelation.joiningColumn.onDelete
 			) {
-				return UpdateRelationOnDeleteModification.createModification({
+				return updateRelationOnDeleteModification.createModification({
 					entityName: updatedEntity.name,
 					fieldName: updatedRelation.name,
 					onDelete: updatedRelation.joiningColumn.onDelete,
@@ -50,9 +58,3 @@ export const UpdateRelationOnDeleteModification: ModificationHandlerStatic<Updat
 		})
 	}
 }
-
-export interface UpdateRelationOnDeleteModificationData {
-	entityName: string
-	fieldName: string
-	onDelete: Model.OnDelete
-}