fix(sync): actually disable browser broadcast

It was still enabled as we mistyped the `disableBc` option.

We cannot use it right now as the y-websocket room name
does not include the base version etag.

Therefore different tabs may be in different sessions
and would risk syncing never the less.

Observed this when reloading the editor on an outdated session:
* Cleaning up the old yjs document took a while in the beforeDestroy callback.
* The new document was already created.
* The new y-websocket provider issued a sync step 1 via browser broadcast.
* The old y-websocket provider responded and send its full state.

Signed-off-by: Max <[email protected]>

Author: max-nextcloud

src/components/Editor.vue

@@ -484,7 +484,7 @@ export default defineComponent({
 				syncService: this.syncService,
 				fileId: this.fileId,
 				initialSession: this.initialSession,
-				disableBC: true,
+				disableBc: true,
 				awareness: this.awareness,
 			})
 		},

src/composables/useIndexedDbProvider.ts

@@ -39,7 +39,7 @@ export function useIndexedDbProvider(
 	 */
 	function setDirty(val: boolean) {
 		dirty.value = Boolean(val)
-		indexedDbProvider.set('dirty', val ? 1 : 0)
+		return indexedDbProvider.set('dirty', val ? 1 : 0)
 	}
 
 	/**

src/services/SyncServiceProvider.js

@@ -37,7 +37,6 @@ export default function createSyncServiceProvider({
 		initialSession,
 		queue,
 	)
-	disableBc = disableBc ?? !!window?._oc_debug
 	const websocketProvider = new WebsocketProvider(
 		'ws://localhost:1234',
 		'file:' + fileId,