fix: small PR fixes workspace-sync-fields (#6107)

Small fixes of PR #6069
This commit is contained in:
Jérémy M
2024-07-03 19:19:37 +02:00
committed by GitHub
parent c2da7c4c6e
commit 921b311c65
2 changed files with 27 additions and 34 deletions

View File

@ -122,7 +122,7 @@ export class TimelineCalendarEventService {
return { return {
totalNumberOfCalendarEvents: total, totalNumberOfCalendarEvents: total,
timelineCalendarEvents: timelineCalendarEvents, timelineCalendarEvents,
}; };
} }

View File

@ -3,7 +3,10 @@ import { Injectable, Logger } from '@nestjs/common';
import { EntityManager } from 'typeorm'; import { EntityManager } from 'typeorm';
import { WorkspaceSyncContext } from 'src/engine/workspace-manager/workspace-sync-metadata/interfaces/workspace-sync-context.interface'; import { WorkspaceSyncContext } from 'src/engine/workspace-manager/workspace-sync-metadata/interfaces/workspace-sync-context.interface';
import { ComparatorAction } from 'src/engine/workspace-manager/workspace-sync-metadata/interfaces/comparator.interface'; import {
ComparatorAction,
FieldComparatorResult,
} from 'src/engine/workspace-manager/workspace-sync-metadata/interfaces/comparator.interface';
import { FeatureFlagMap } from 'src/engine/core-modules/feature-flag/interfaces/feature-flag-map.interface'; import { FeatureFlagMap } from 'src/engine/core-modules/feature-flag/interfaces/feature-flag-map.interface';
import { WorkspaceMigrationBuilderAction } from 'src/engine/workspace-manager/workspace-migration-builder/interfaces/workspace-migration-builder-action.interface'; import { WorkspaceMigrationBuilderAction } from 'src/engine/workspace-manager/workspace-migration-builder/interfaces/workspace-migration-builder-action.interface';
@ -133,8 +136,6 @@ export class WorkspaceSyncFieldMetadataService {
originalObjectMetadataCollection, originalObjectMetadataCollection,
); );
this.logger.log('Comparing standard objects and fields metadata');
// Loop over all standard objects and compare them with the objects in DB // Loop over all standard objects and compare them with the objects in DB
for (const [ for (const [
standardObjectId, standardObjectId,
@ -155,22 +156,7 @@ export class WorkspaceSyncFieldMetadataService {
computedStandardFieldMetadataCollection, computedStandardFieldMetadataCollection,
); );
for (const fieldComparatorResult of fieldComparatorResults) { this.storeComparatorResults(fieldComparatorResults, storage);
switch (fieldComparatorResult.action) {
case ComparatorAction.CREATE: {
storage.addCreateFieldMetadata(fieldComparatorResult.object);
break;
}
case ComparatorAction.UPDATE: {
storage.addUpdateFieldMetadata(fieldComparatorResult.object);
break;
}
case ComparatorAction.DELETE: {
storage.addDeleteFieldMetadata(fieldComparatorResult.object);
break;
}
}
}
} }
} }
@ -205,20 +191,27 @@ export class WorkspaceSyncFieldMetadataService {
standardFieldMetadataCollection, standardFieldMetadataCollection,
); );
for (const fieldComparatorResult of fieldComparatorResults) { this.storeComparatorResults(fieldComparatorResults, storage);
switch (fieldComparatorResult.action) { }
case ComparatorAction.CREATE: { }
storage.addCreateFieldMetadata(fieldComparatorResult.object);
break; private storeComparatorResults(
} fieldComparatorResults: FieldComparatorResult[],
case ComparatorAction.UPDATE: { storage: WorkspaceSyncStorage,
storage.addUpdateFieldMetadata(fieldComparatorResult.object); ): void {
break; for (const fieldComparatorResult of fieldComparatorResults) {
} switch (fieldComparatorResult.action) {
case ComparatorAction.DELETE: { case ComparatorAction.CREATE: {
storage.addDeleteFieldMetadata(fieldComparatorResult.object); storage.addCreateFieldMetadata(fieldComparatorResult.object);
break; break;
} }
case ComparatorAction.UPDATE: {
storage.addUpdateFieldMetadata(fieldComparatorResult.object);
break;
}
case ComparatorAction.DELETE: {
storage.addDeleteFieldMetadata(fieldComparatorResult.object);
break;
} }
} }
} }