Skip to content

Commit a7ec13d

Browse files
authored
Fix canonicalizer rules added beyond canonicalizer (#790)
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
1 parent 578d3d7 commit a7ec13d

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

src/alterschema/alterschema.cc

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -361,7 +361,9 @@ auto add(SchemaTransformer &bundle, const AlterSchemaMode mode) -> void {
361361
bundle.add<DuplicateAnyOfBranches>();
362362
bundle.add<FlattenNestedAllOf>();
363363
bundle.add<FlattenNestedAnyOf>();
364-
bundle.add<Draft3TypeAny>();
364+
if (mode == AlterSchemaMode::Canonicalizer) {
365+
bundle.add<Draft3TypeAny>();
366+
}
365367
bundle.add<UnsatisfiableInPlaceApplicatorType>();
366368
bundle.add<AllOfFalseSimplify>();
367369
bundle.add<AnyOfFalseSimplify>();
@@ -468,7 +470,9 @@ auto add(SchemaTransformer &bundle, const AlterSchemaMode mode) -> void {
468470
}
469471

470472
bundle.add<DropAllOfEmptySchemas>();
471-
bundle.add<Draft3DropExtendsEmptySchemas>();
473+
if (mode == AlterSchemaMode::Canonicalizer) {
474+
bundle.add<Draft3DropExtendsEmptySchemas>();
475+
}
472476
bundle.add<EmptyObjectAsTrue>();
473477

474478
if (mode == AlterSchemaMode::Canonicalizer) {

0 commit comments

Comments
 (0)