From 0546fc502e277ba8c7775daba068a6870c59ff50 Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Thu, 10 Oct 2024 11:08:37 +0200
Subject: [PATCH] fix: handle background blur in Talk if server doesn't support
 it

- partially reverts ed39a6bef13f7c1312459ebc610ba669d26501e0

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 src/components/CallView/CallView.vue          | 37 +++++++++-
 .../SettingsDialog/SettingsDialog.vue         | 72 ++++++++++++++-----
 2 files changed, 90 insertions(+), 19 deletions(-)

diff --git a/src/components/CallView/CallView.vue b/src/components/CallView/CallView.vue
index 1402ce2980e..68f21ddbec1 100644
--- a/src/components/CallView/CallView.vue
+++ b/src/components/CallView/CallView.vue
@@ -21,7 +21,7 @@
 -->
 
 <template>
-	<div id="call-container">
+	<div id="call-container" :style="callContainerStyle">
 		<ViewerOverlayCallView v-if="isViewerOverlay"
 			:token="token"
 			:model="promotedParticipantModel"
@@ -136,10 +136,12 @@
 
 <script>
 import debounce from 'debounce'
+import { ref } from 'vue'
 
 import { getCapabilities } from '@nextcloud/capabilities'
 import { showMessage } from '@nextcloud/dialogs'
 import { subscribe, unsubscribe } from '@nextcloud/event-bus'
+import { loadState } from '@nextcloud/initial-state'
 
 import Grid from './Grid/Grid.vue'
 import EmptyCallView from './shared/EmptyCallView.vue'
@@ -151,11 +153,15 @@ import VideoVue from './shared/VideoVue.vue'
 import ViewerOverlayCallView from './shared/ViewerOverlayCallView.vue'
 
 import { SIMULCAST } from '../../constants.js'
+import BrowserStorage from '../../services/BrowserStorage.js'
 import { fetchPeers } from '../../services/callsService.js'
 import { EventBus } from '../../services/EventBus.js'
 import { localMediaModel, localCallParticipantModel, callParticipantCollection } from '../../utils/webrtc/index.js'
 import RemoteVideoBlocker from '../../utils/webrtc/RemoteVideoBlocker.js'
 
+const serverVersion = loadState('core', 'config', {}).versionstring ?? '29.0.0'
+const serverSupportsBackgroundBlurred = '29.0.4'.localeCompare(serverVersion) < 1
+
 const supportedReactions = getCapabilities()?.spreed?.config?.call?.['supported-reactions']
 
 export default {
@@ -190,11 +196,16 @@ export default {
 	},
 
 	setup() {
+		const isBackgroundBlurred = ref(serverSupportsBackgroundBlurred
+			? null
+			: BrowserStorage.getItem('background-blurred') !== 'false')
+
 		return {
 			supportedReactions,
 			localMediaModel,
 			localCallParticipantModel,
 			callParticipantCollection,
+			isBackgroundBlurred,
 		}
 	},
 
@@ -344,6 +355,19 @@ export default {
 		presenterVideoBlockerEnabled() {
 			return this.sharedDatas[this.shownRemoteScreenPeerId]?.remoteVideoBlocker?.isVideoEnabled()
 		},
+
+		/**
+		 * Fallback style for versions before v29.0.4
+		 */
+		callContainerStyle() {
+			if (serverSupportsBackgroundBlurred) {
+				return
+			}
+
+			return {
+				'backdrop-filter': this.isBackgroundBlurred ? 'blur(25px)' : 'none',
+			}
+		}
 	},
 
 	watch: {
@@ -442,6 +466,7 @@ export default {
 		callParticipantCollection.on('remove', this._lowerHandWhenParticipantLeaves)
 
 		subscribe('switch-screen-to-id', this._switchScreenToId)
+		subscribe('set-background-blurred', this.setBackgroundBlurred)
 	},
 
 	beforeDestroy() {
@@ -451,6 +476,7 @@ export default {
 		callParticipantCollection.off('remove', this._lowerHandWhenParticipantLeaves)
 
 		unsubscribe('switch-screen-to-id', this._switchScreenToId)
+		unsubscribe('set-background-blurred', this.setBackgroundBlurred)
 	},
 
 	methods: {
@@ -714,6 +740,14 @@ export default {
 			}
 		},
 
+		/**
+		 * Fallback method for versions before v29.0.4
+		 * @param {boolean} value whether background should be blurred
+		 */
+		setBackgroundBlurred(value) {
+			this.isBackgroundBlurred = value
+		},
+
 		isModelWithVideo(callParticipantModel) {
 			return callParticipantModel.attributes.videoAvailable
 				&& this.sharedDatas[callParticipantModel.attributes.peerId].remoteVideoBlocker.isVideoEnabled()
@@ -746,6 +780,7 @@ export default {
 	width: 100%;
 	height: 100%;
 	background-color: $color-call-background;
+	// Default value has changed since v29.0.4: 'blur(25px)' => 'none'
 	backdrop-filter: var(--filter-background-blur);
 }
 
diff --git a/src/components/SettingsDialog/SettingsDialog.vue b/src/components/SettingsDialog/SettingsDialog.vue
index 5da9668f910..b718f51505c 100644
--- a/src/components/SettingsDialog/SettingsDialog.vue
+++ b/src/components/SettingsDialog/SettingsDialog.vue
@@ -101,20 +101,30 @@
 		<NcAppSettingsSection id="performance"
 			:name="t('spreed', 'Performance')"
 			class="app-settings-section">
-			<NcCheckboxRadioSwitch id="blur-call-background"
-				:checked="isBackgroundBlurred === 'yes'"
-				:indeterminate="isBackgroundBlurred === ''"
-				type="checkbox"
+			<template v-if="serverSupportsBackgroundBlurred">
+				<NcCheckboxRadioSwitch id="blur-call-background"
+					:checked="isBackgroundBlurred === 'yes'"
+					:indeterminate="isBackgroundBlurred === ''"
+					type="checkbox"
+					class="checkbox"
+					disabled>
+					{{ t('spreed', 'Blur background image in the call (may increase GPU load)') }}
+				</NcCheckboxRadioSwitch>
+				<a :href="themingUrl"
+					target="_blank"
+					rel="noreferrer nofollow"
+					class="external">
+					{{ t('spreed', 'Background blur for Nextcloud instance can be adjusted in the theming settings.') }} ↗
+				</a>
+			</template>
+			<NcCheckboxRadioSwitch v-else
+				id="blur-call-background"
+				:checked="isBackgroundBlurred !== 'false'"
+				type="switch"
 				class="checkbox"
-				disabled>
+				@update:checked="toggleBackgroundBlurred">
 				{{ t('spreed', 'Blur background image in the call (may increase GPU load)') }}
 			</NcCheckboxRadioSwitch>
-			<a :href="themingUrl"
-				target="_blank"
-				rel="noreferrer nofollow"
-				class="external">
-				{{ t('spreed', 'Background blur for Nextcloud instance can be adjusted in the theming settings.') }} ↗
-			</a>
 		</NcAppSettingsSection>
 		<NcAppSettingsSection v-if="!disableKeyboardShortcuts"
 			id="shortcuts"
@@ -186,11 +196,13 @@
 </template>
 
 <script>
+import { ref } from 'vue'
+
 import axios from '@nextcloud/axios'
 import { getCapabilities } from '@nextcloud/capabilities'
 import { showError, showSuccess } from '@nextcloud/dialogs'
 import { FilePickerVue } from '@nextcloud/dialogs/filepicker.js'
-import { subscribe, unsubscribe } from '@nextcloud/event-bus'
+import { emit, subscribe, unsubscribe } from '@nextcloud/event-bus'
 import { loadState } from '@nextcloud/initial-state'
 import { t } from '@nextcloud/l10n'
 import { generateOcsUrl, generateUrl } from '@nextcloud/router'
@@ -206,7 +218,12 @@ import { PRIVACY } from '../../constants.js'
 import BrowserStorage from '../../services/BrowserStorage.js'
 import { useSettingsStore } from '../../stores/settings.js'
 
-const isBackgroundBlurred = loadState('spreed', 'force_enable_blur_filter', '')
+const serverVersion = loadState('core', 'config', {}).versionstring ?? '29.0.0'
+const serverSupportsBackgroundBlurred = '29.0.4'.localeCompare(serverVersion) < 1
+
+const isBackgroundBlurredState = serverSupportsBackgroundBlurred
+	? loadState('spreed', 'force_enable_blur_filter', '') // 'yes', 'no', ''
+	: BrowserStorage.getItem('background-blurred') // 'true', 'false', null
 const supportTypingStatus = getCapabilities()?.spreed?.config?.chat?.['typing-privacy'] !== undefined
 
 export default {
@@ -223,11 +240,13 @@ export default {
 
 	setup() {
 		const settingsStore = useSettingsStore()
+		const isBackgroundBlurred = ref(isBackgroundBlurredState)
 
 		return {
 			settingsStore,
 			supportTypingStatus,
 			isBackgroundBlurred,
+			serverSupportsBackgroundBlurred,
 		}
 	},
 
@@ -293,12 +312,20 @@ export default {
 
 	created() {
 		const blurred = BrowserStorage.getItem('background-blurred')
-		if (blurred === 'false' && isBackgroundBlurred === '') {
-			console.debug('Blur was disabled intentionally, propagating last choice to server')
-			axios.post(generateOcsUrl('apps/provisioning_api/api/v1/config/users/theming/force_enable_blur_filter'),
-				{ configValue: 'no' })
+		if (serverSupportsBackgroundBlurred) {
+			// Blur is handled by theming app, migrating
+			if (blurred === 'false' && isBackgroundBlurredState === '') {
+				console.debug('Blur was disabled intentionally, propagating last choice to server')
+				axios.post(generateOcsUrl('apps/provisioning_api/api/v1/config/users/theming/force_enable_blur_filter'),
+					{ configValue: 'no' })
+			}
+			BrowserStorage.removeItem('background-blurred')
+		} else {
+			// Fallback to BrowserStorage
+			if (blurred === null) {
+				BrowserStorage.setItem('background-blurred', 'true')
+			}
 		}
-		BrowserStorage.removeItem('background-blurred')
 	},
 
 	mounted() {
@@ -353,6 +380,15 @@ export default {
 			this.privacyLoading = false
 		},
 
+		toggleBackgroundBlurred(value) {
+			if (serverSupportsBackgroundBlurred) {
+				return
+			}
+			this.isBackgroundBlurred = value ? 'true' : 'false'
+			BrowserStorage.setItem('background-blurred', value)
+			emit('set-background-blurred', value)
+		},
+
 		async togglePlaySounds() {
 			this.playSoundsLoading = true
 			try {