fix(frontend:files): update file metadata timestamps on save and align OnlyOffice state change handlers

johaven · johaven · commit db768e14452f · 2025-12-07T23:18:24.000+01:00
diff --git a/backend/src/applications/files/interfaces/only-office-config.interface.ts b/backend/src/applications/files/interfaces/only-office-config.interface.ts
@@ -165,7 +165,7 @@ export interface OnlyOfficeConfig {
     onAppReady?: (event: object) => void
     onCollaborativeChanges?: (event: object) => void
     onDocumentReady?: (event: object) => void
-    onDocumentStateChange?: (event: object) => void
+    onDocumentStateChange?: (event: { data: boolean }) => void
     onDownloadAs?: (event: object) => void
     onError?: (event: object) => void
     onInfo?: (event: object) => void
@@ -191,7 +191,7 @@ export interface OnlyOfficeConfig {
     onRequestSelectSpreadsheet?: (event: object) => void
     onRequestSendNotify?: (event: object) => void
     onRequestSharingSettings?: (event: object) => void
-    onRequestStartFilling: (event: object) => void
+    onRequestStartFilling?: (event: object) => void
     onRequestUsers?: (event: object) => void
     onSubmit?: (event: object) => void
     onWarning?: (event: object) => void
diff --git a/frontend/src/app/applications/files/components/utils/only-office.component.ts b/frontend/src/app/applications/files/components/utils/only-office.component.ts
@@ -17,6 +17,7 @@ export class OnlyOfficeComponent implements OnInit, OnChanges, OnDestroy {
   @Input() documentServerUrl: string
   @Input() config: OnlyOfficeConfig
   @Output() loadError = new EventEmitter<string>()
+  @Output() wasSaved = new EventEmitter<string>()
   private isFirstOnChanges = true
 
   ngOnInit(): void {
@@ -51,6 +52,7 @@ export class OnlyOfficeComponent implements OnInit, OnChanges, OnDestroy {
     if (window?.DocEditor?.instances[this.id]) {
       window.DocEditor.instances[this.id].destroyEditor()
       window.DocEditor.instances[this.id] = undefined
+      delete window.DocEditor.instances[this.id]
     }
   }
 
@@ -70,6 +72,7 @@ export class OnlyOfficeComponent implements OnInit, OnChanges, OnDestroy {
       }
 
       const config: OnlyOfficeConfig = JSON.parse(JSON.stringify(this.config))
+      config.events = { onDocumentStateChange: (e: { data: boolean }) => (e.data ? this.wasSaved.emit() : null) }
       window.DocEditor.instances[this.id] = window.DocsAPI.DocEditor(this.id, config)
     } catch (err) {
       console.error(err)
diff --git a/frontend/src/app/applications/files/components/viewers/files-viewer-document.component.ts b/frontend/src/app/applications/files/components/viewers/files-viewer-document.component.ts
@@ -35,6 +35,7 @@ import { OnlyOfficeComponent } from '../utils/only-office.component'
           [documentServerUrl]="documentConfig.documentServerUrl"
           [config]="documentConfig.config"
           (loadError)="loadError($event)"
+          (wasSaved)="onSave()"
         ></app-files-onlyoffice-document>
       </div>
     }
@@ -95,4 +96,8 @@ export class FilesViewerDocumentComponent implements OnInit, OnDestroy {
     this.layout.closeDialog()
     this.layout.sendNotification('error', 'Unable to open document', errorMessage)
   }
+
+  onSave() {
+    this.file.updateHTimeAgo()
+  }
 }
diff --git a/frontend/src/app/applications/files/components/viewers/files-viewer-text.component.ts b/frontend/src/app/applications/files/components/viewers/files-viewer-text.component.ts
@@ -203,6 +203,7 @@ export class FilesViewerTextComponent implements OnInit, OnDestroy {
         if (exit) {
           this.onClose().catch(console.error)
         }
+        this.file().updateHTimeAgo()
       },
       error: (e: HttpErrorResponse) => {
         this.isSaving.set(false)
diff --git a/frontend/src/app/applications/files/models/file.model.ts b/frontend/src/app/applications/files/models/file.model.ts
@@ -13,7 +13,7 @@ import {
 import type { FileLockProps, FileProps } from '@sync-in-server/backend/src/applications/files/interfaces/file-props.interface'
 import type { File } from '@sync-in-server/backend/src/applications/files/schemas/file.interface'
 import { SPACE_OPERATION } from '@sync-in-server/backend/src/applications/spaces/constants/spaces'
-import { popFromObject } from '@sync-in-server/backend/src/common/shared'
+import { currentTimeStamp, popFromObject } from '@sync-in-server/backend/src/common/shared'
 import type { Observable } from 'rxjs'
 import { convertBytesToText, getNewly } from '../../../common/utils/functions'
 import { dJs } from '../../../common/utils/time'
@@ -92,8 +92,7 @@ export class FileModel implements File {
     Object.assign(this, props)
     this.path = `${basePath}/${this.path !== '.' ? `${this.path}/` : ''}${this.root?.alias || this.name}`
     this.mime = this.getMime(this.mime, inShare)
-    this.hTimeAgo = dJs(this.mtime).fromNow(true)
-    this.newly = getNewly(this.mtime)
+    this.updateHTimeAgo(this.mtime)
     this.setMimeUrl()
     this.setHSize()
     this.setRoot(inShare)
@@ -125,6 +124,12 @@ export class FileModel implements File {
     this.setMimeUrl()
   }
 
+  updateHTimeAgo(mtime?: number) {
+    mtime ??= currentTimeStamp(null, true)
+    this.hTimeAgo = dJs(mtime).fromNow(true)
+    this.newly = getNewly(mtime)
+  }
+
   private getType(inShare: boolean) {
     return this.isDir ? (inShare ? mimeDirectoryShare : mimeDirectory) : mimeFile
   }

Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@ export class OnlyOfficeComponent implements OnInit, OnChanges, OnDestroy {`
`17`	`17`	`@Input() documentServerUrl: string`
`18`	`18`	`@Input() config: OnlyOfficeConfig`
`19`	`19`	`@Output() loadError = new EventEmitter<string>()`
	`20`	`+ @Output() wasSaved = new EventEmitter<string>()`
`20`	`21`	`private isFirstOnChanges = true`
`21`	`22`
`22`	`23`	`ngOnInit(): void {`
`@@ -51,6 +52,7 @@ export class OnlyOfficeComponent implements OnInit, OnChanges, OnDestroy {`
`51`	`52`	`if (window?.DocEditor?.instances[this.id]) {`
`52`	`53`	`window.DocEditor.instances[this.id].destroyEditor()`
`53`	`54`	`window.DocEditor.instances[this.id] = undefined`
	`55`	`+ delete window.DocEditor.instances[this.id]`
`54`	`56`	`}`
`55`	`57`	`}`
`56`	`58`
`@@ -70,6 +72,7 @@ export class OnlyOfficeComponent implements OnInit, OnChanges, OnDestroy {`
`70`	`72`	`}`
`71`	`73`
`72`	`74`	`const config: OnlyOfficeConfig = JSON.parse(JSON.stringify(this.config))`
	`75`	`+ config.events = { onDocumentStateChange: (e: { data: boolean }) => (e.data ? this.wasSaved.emit() : null) }`
`73`	`76`	`window.DocEditor.instances[this.id] = window.DocsAPI.DocEditor(this.id, config)`
`74`	`77`	`} catch (err) {`
`75`	`78`	`console.error(err)`
Original file line number	Diff line number	Diff line change
`@@ -35,6 +35,7 @@ import { OnlyOfficeComponent } from '../utils/only-office.component'`
`35`	`35`	`[documentServerUrl]="documentConfig.documentServerUrl"`
`36`	`36`	`[config]="documentConfig.config"`
`37`	`37`	`(loadError)="loadError($event)"`
	`38`	`+ (wasSaved)="onSave()"`
`38`	`39`	`></app-files-onlyoffice-document>`
`39`	`40`	`</div>`
`40`	`41`	`}`
`@@ -95,4 +96,8 @@ export class FilesViewerDocumentComponent implements OnInit, OnDestroy {`
`95`	`96`	`this.layout.closeDialog()`
`96`	`97`	`this.layout.sendNotification('error', 'Unable to open document', errorMessage)`
`97`	`98`	`}`
	`99`	`+`
	`100`	`+ onSave() {`
	`101`	`+ this.file.updateHTimeAgo()`
	`102`	`+ }`
`98`	`103`	`}`
Original file line number	Diff line number	Diff line change
`@@ -203,6 +203,7 @@ export class FilesViewerTextComponent implements OnInit, OnDestroy {`
`203`	`203`	`if (exit) {`
`204`	`204`	`this.onClose().catch(console.error)`
`205`	`205`	`}`
	`206`	`+ this.file().updateHTimeAgo()`
`206`	`207`	`},`
`207`	`208`	`error: (e: HttpErrorResponse) => {`
`208`	`209`	`this.isSaving.set(false)`