diff --git a/src/mol-state/state.ts b/src/mol-state/state.ts index a6cd627ccddf7fb361c180bc55ed05ec2b1c591a..b72476c25ec0ffdb1a37039f3bd409c790ee402d 100644 --- a/src/mol-state/state.ts +++ b/src/mol-state/state.ts @@ -249,6 +249,11 @@ class State { if (!this.inTransaction) this.behaviors.isUpdating.next(true); try { + if (StateBuilder.is(tree)) { + if (tree.editInfo.applied) throw new Error('This builder has already been applied. Create a new builder for further state updates'); + tree.editInfo.applied = true; + } + this.reverted = false; const ret = options && (options.revertIfAborted || options.revertOnError) ? await this._revertibleTreeUpdate(taskCtx, params, options) diff --git a/src/mol-state/state/builder.ts b/src/mol-state/state/builder.ts index c1fd8cbaf38c55168ace43485b634ef2002cb7b2..e249ee738ec42ddcebf871da6dccf74aca584f81 100644 --- a/src/mol-state/state/builder.ts +++ b/src/mol-state/state/builder.ts @@ -21,6 +21,7 @@ interface StateBuilder { namespace StateBuilder { export interface EditInfo { + applied: boolean, sourceTree: StateTree, count: number, lastUpdate?: StateTransform.Ref @@ -108,7 +109,7 @@ namespace StateBuilder { return this.state.state.runTask(this.state.state.updateTree(this, options)); } - constructor(tree: StateTree, state?: State) { this.state = { state, tree: tree.asTransient(), actions: [], editInfo: { sourceTree: tree, count: 0, lastUpdate: void 0 } } } + constructor(tree: StateTree, state?: State) { this.state = { state, tree: tree.asTransient(), actions: [], editInfo: { applied: false, sourceTree: tree, count: 0, lastUpdate: void 0 } } } } export class To<A extends StateObject, T extends StateTransformer = StateTransformer> implements StateBuilder {