Merge pull request #5930 from nextcloud/perf/read-only-sync

nextcloud · Jun 21, 2024 · 3381c80 · 3381c80
2 parents 7c518f6 + a3617e7
commit 3381c80
Showing 1 changed file with 13 additions and 1 deletion.
diff --git a/src/services/PollingBackend.js b/src/services/PollingBackend.js
@@ -27,6 +27,12 @@ const FETCH_INTERVAL_MAX = 5000
  */
 const FETCH_INTERVAL_SINGLE_EDITOR = 5000
 
+/**
+ * Interval to check for changes for read only users
+ * @type {number}
+ */
+const FETCH_INTERVAL_READ_ONLY = 30000
+
 /**
  * Interval to fetch for changes when a browser window is considered invisible by the
  * page visibility API https://developer.mozilla.org/de/docs/Web/API/Page_Visibility_API
@@ -120,7 +126,9 @@ class PollingBackend {
 			}
 			const disconnect = Date.now() - COLLABORATOR_DISCONNECT_TIME
 			const alive = sessions.filter((s) => s.lastContact * 1000 > disconnect)
-			if (alive.length < 2) {
+			if (this.#syncService.connection.state.document.readOnly) {
+				this.maximumReadOnlyTimer()
+			} else if (alive.length < 2) {
 				this.maximumRefetchTimer()
 			} else {
 				this.increaseRefetchTimer()
@@ -193,6 +201,10 @@ class PollingBackend {
 		this.#fetchInterval = FETCH_INTERVAL_SINGLE_EDITOR
 	}
 
+	maximumReadOnlyTimer() {
+		this.#fetchInterval = FETCH_INTERVAL_READ_ONLY
+	}
+
 	visibilitychange() {
 		if (document.visibilityState === 'hidden') {
 			this.#fetchInterval = FETCH_INTERVAL_INVISIBLE