yonote/yjs
14
0
Fork 0
Code Pull Requests 1 Releases Activity

toDelta doesnt create transaction - fixes #506

Browse Source
This commit is contained in:
Kevin Jahns
2023-03-11 09:13:27 +01:00
parent da8bacfc78
commit 227018f5c7
4 changed files with 23 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
src/types/YText.js
View File

@@ -1018,15 +1018,7 @@ export class YText extends AbstractType {
str = ''
}
}
// snapshots are merged again after the transaction, so we need to keep the
// transalive until we are done
transact(doc, transaction => {
if (snapshot) {
splitSnapshotAffectedStructs(transaction, snapshot)
}
if (prevSnapshot) {
splitSnapshotAffectedStructs(transaction, prevSnapshot)
}
const computeDelta = () => {
while (n !== null) {
if (isVisible(n, snapshot) || (prevSnapshot !== undefined && isVisible(n, prevSnapshot))) {
switch (n.content.constructor) {
@@ -1079,7 +1071,22 @@ export class YText extends AbstractType {
n = n.right
}
packStr()
}, 'cleanup')
}
if (snapshot || prevSnapshot) {
// snapshots are merged again after the transaction, so we need to keep the
// transaction alive until we are done
transact(doc, transaction => {
if (snapshot) {
splitSnapshotAffectedStructs(transaction, snapshot)
}
if (prevSnapshot) {
splitSnapshotAffectedStructs(transaction, prevSnapshot)
}
computeDelta()
}, 'cleanup')
} else {
computeDelta()
}
return ops
}