fix: extra logging in documents content migration (#5868)

Signed-off-by: Alexander Onnikov <Alexander.Onnikov@xored.com>
This commit is contained in:
Alexander Onnikov 2024-06-20 21:27:34 +07:00 committed by GitHub
parent dedff23b31
commit c88adbc5a9
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 6 additions and 7 deletions

View File

@ -275,11 +275,7 @@ async function migrateTeamspacesMixins (client: MigrationClient): Promise<void>
async function migrateContentField (client: MigrationClient): Promise<void> { async function migrateContentField (client: MigrationClient): Promise<void> {
const ctx = new MeasureMetricsContext('migrate_content_field', {}) const ctx = new MeasureMetricsContext('migrate_content_field', {})
const storage = client.storageAdapter const storage = client.storageAdapter
if (storage === undefined) {
return
}
const documents = await client.find<Document>(DOMAIN_DOCUMENT, { const documents = await client.find<Document>(DOMAIN_DOCUMENT, {
_class: document.class.Document, _class: document.class.Document,
@ -290,17 +286,20 @@ async function migrateContentField (client: MigrationClient): Promise<void> {
try { try {
const ydoc = await loadCollaborativeDoc(storage, client.workspaceId, document.content, ctx) const ydoc = await loadCollaborativeDoc(storage, client.workspaceId, document.content, ctx)
if (ydoc === undefined) { if (ydoc === undefined) {
ctx.error('document content not found', { document: document.name })
continue continue
} }
if (!ydoc.share.has('')) { if (!ydoc.share.has('') || ydoc.share.has('content')) {
continue continue
} }
yDocCopyXmlField(ydoc, '', 'content') yDocCopyXmlField(ydoc, '', 'content')
await saveCollaborativeDoc(storage, client.workspaceId, document.content, ydoc, ctx) await saveCollaborativeDoc(storage, client.workspaceId, document.content, ydoc, ctx)
} catch {} } catch (err) {
ctx.error('error document content migration', { error: err, document: document.name })
}
} }
} }

View File

@ -104,7 +104,7 @@ export interface MigrationClient {
model: ModelDb model: ModelDb
migrateState: Map<string, Set<string>> migrateState: Map<string, Set<string>>
storageAdapter?: StorageAdapter storageAdapter: StorageAdapter
workspaceId: WorkspaceId workspaceId: WorkspaceId
} }