Fixed when doc reconnect might get wrong OT operation, reorder initialize instructions in proper sequence

This commit is contained in:
Wu Cheng-Han 2015-09-25 13:59:28 +08:00
parent 9acd070581
commit ab6deb8e2e

View file

@ -998,6 +998,18 @@ socket.on('refresh', function (data) {
lastchangeui = ui.infobar.lastchange; lastchangeui = ui.infobar.lastchange;
updateLastChange(); updateLastChange();
checkPermission(); checkPermission();
if (!loaded) {
changeMode(currentMode);
loaded = true;
emitUserStatus(); //send first user status
updateOnlineStatus(); //update first online status
setTimeout(function () {
//work around editor not refresh or doc not fully loaded
windowResizeInner();
//work around might not scroll to hash
scrollToHash();
}, 1);
}
}); });
var EditorClient = ot.EditorClient; var EditorClient = ot.EditorClient;
@ -1013,43 +1025,18 @@ socket.on('doc', function (obj) {
saveInfo(); saveInfo();
if (bodyMismatch) { if (bodyMismatch) {
if (cmClient)
cmClient.editorAdapter.ignoreNextChange = true;
if (body) if (body)
editor.setValue(body); editor.setValue(body);
else else
editor.setValue(""); editor.setValue("");
} }
if (!cmClient) {
cmClient = window.cmClient = new EditorClient(
obj.revision, obj.clients,
new SocketIOAdapter(socket), new CodeMirrorAdapter(editor)
);
} else {
cmClient.revision = obj.revision;
cmClient.initializeClients(obj.clients);
if (bodyMismatch) {
cmClient.undoManager.undoStack.length = 0;
cmClient.undoManager.redoStack.length = 0;
}
}
if (!loaded) { if (!loaded) {
editor.clearHistory(); editor.clearHistory();
ui.spinner.hide(); ui.spinner.hide();
ui.content.fadeIn(); ui.content.fadeIn();
changeMode();
loaded = true;
emitUserStatus(); //send first user status
updateOnlineStatus(); //update first online status
setTimeout(function () {
//work around editor not refresh
editor.refresh();
//work around cursor not refresh
for (var i = 0; i < onlineUsers.length; i++) {
buildCursor(onlineUsers[i]);
}
//work around might not scroll to hash
scrollToHash();
}, 1);
} else { } else {
//if current doc is equal to the doc before disconnect //if current doc is equal to the doc before disconnect
if (bodyMismatch) if (bodyMismatch)
@ -1061,6 +1048,22 @@ socket.on('doc', function (obj) {
lastInfo.history = null; lastInfo.history = null;
} }
if (!cmClient) {
cmClient = window.cmClient = new EditorClient(
obj.revision, obj.clients,
new SocketIOAdapter(socket), new CodeMirrorAdapter(editor)
);
} else {
if (bodyMismatch) {
cmClient.undoManager.undoStack.length = 0;
cmClient.undoManager.redoStack.length = 0;
}
cmClient.revision = obj.revision;
cmClient.setState(new ot.Client.Synchronized());
cmClient.initializeClientList();
cmClient.initializeClients(obj.clients);
}
if (bodyMismatch) { if (bodyMismatch) {
isDirty = true; isDirty = true;
updateView(); updateView();