self-hosted/focalboard
1
0
Fork 0
mirror of https://github.com/mattermost/focalboard.git synced 2024-12-24 13:43:12 +02:00
Code Issues Releases Activity

Corner case: handle deleted view by another client

Browse Source
This commit is contained in:
Chen-I Lim 2020-12-09 16:52:29 -08:00
parent a154ed561a
commit 4b744bd395
1 changed files with 24 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

41
webapp/src/pages/boardPage.tsx
View File

@ -3,6 +3,7 @@
import React from 'react'
import {IBlock} from '../blocks/block'
import {MutableBoard} from '../blocks/board'
import {sendFlashMessage} from '../components/flashMessages'
import {WorkspaceComponent} from '../components/workspaceComponent'
import mutator from '../mutator'
@ -180,9 +181,14 @@ export default class BoardPage extends React.Component<Props, State> {
const boardTree = await MutableBoardTree.sync(boardId)
if (boardTree && boardTree.board) {
// Default to first view
// Default to first view
boardTree.setActiveView(viewId || boardTree.views[0].id)
// Update url with viewId if it's different
if (boardTree.activeView.id !== this.state.viewId) {
Utils.replaceUrlQueryParam('v', boardTree.activeView.id)
}
// TODO: Handle error (viewId not found)
this.setState({
@ -203,32 +209,33 @@ export default class BoardPage extends React.Component<Props, State> {
}
private async incrementalUpdate(blocks: IBlock[]) {
const {workspaceTree, boardTree} = this.state
const {workspaceTree, boardTree, viewId} = this.state
let newState = {workspaceTree, boardTree}
let newState = {workspaceTree, boardTree, viewId}
const newWorkspaceTree = workspaceTree.mutableCopy()
if (newWorkspaceTree.incrementalUpdate(blocks)) {
newState = {...newState, workspaceTree: newWorkspaceTree}
}
let newBoardTree: MutableBoardTree | undefined
if (boardTree) {
const newBoardTree = MutableBoardTree.incrementalUpdate(boardTree, blocks)
if (newBoardTree) {
newBoardTree.setActiveView(this.state.viewId)
newState = {...newState, boardTree: newBoardTree}
} else {
newState = {...newState, boardTree: undefined}
}
newBoardTree = MutableBoardTree.incrementalUpdate(boardTree, blocks)
} else if (this.state.boardId) {
// Corner case: When the page is viewing a deleted board, that is subsequently un-deleted on another client
const newBoardTree = await MutableBoardTree.sync(this.state.boardId)
if (newBoardTree) {
newBoardTree.setActiveView(this.state.viewId)
newState = {...newState, boardTree: newBoardTree}
} else {
newState = {...newState, boardTree: undefined}
}
newBoardTree = await MutableBoardTree.sync(this.state.boardId)
}
if (newBoardTree) {
newBoardTree.setActiveView(this.state.viewId)
newState = {...newState, boardTree: newBoardTree, viewId: newBoardTree.activeView.id}
} else {
newState = {...newState, boardTree: undefined}
}
// Update url with viewId if it's different
if (newBoardTree && newBoardTree.activeView.id !== this.state.viewId) {
Utils.replaceUrlQueryParam('v', newBoardTree?.activeView.id)
}
this.setState(newState)