ONLYOFFICE / sdkjs-plugins

The add-ons for ONLYOFFICE Document Server and ONLYOFFICE Desktop Editors.
https://www.onlyoffice.com
Apache License 2.0
136 stars 134 forks source link

InsertAndReplaceContentControls is broken for files being edited by multiple users #74

Open Neverbolt opened 5 years ago

Neverbolt commented 5 years ago

Do you want to request a feature or report a bug? This is a bug with the "InsertAndReplaceContentControls" feature.

What is the current behavior? Step 1: Put the sdkjs-plugins/examples/example_insert_content_and_document folder into sdkjs-plugins/ to install the provided example plugin. (This issue exists with other uses of the "InsertAndReplaceContentControls" command as well, however, this is a good common ground)

Step 2: Create a shared document and open it. Run the "Example insert content & document" plugin. It works as expected and inserts the 7 content controls.

Step 3: Now open the document with two users simultaneously. Place the cursor at the end of the document and run the plugin again. It works as expected and inserts the 7 content controls again.

Step 4: Make a page break (not needed, but makes it easier to visualize) and then type some arbitrary text with one user. Place the cursor of the other user somewhere INSIDE of that text and run the plugin again.

The window freezes and nothing works anymore, no content control is inserted and the page needs to be reloaded.

What is the expected behavior? It should be possible to insert content controls inside of existing text.

Which versions of sdkjs-plugins, and which browser / OS are affected by this issue? Did this work in previous versions of sdkjs-plugins? I have the version 5.2.8.24 and am using the provided docker installation from the "Linux Docker Image" over at https://www.onlyoffice.com/download.aspx

Neverbolt commented 5 years ago

Any updates on this / ways I can help?

flaminestone commented 5 years ago

Hello @Neverbolt, Yes, it is bug, thank you. It should be fixed in future versions

Neverbolt commented 5 years ago

Hello @flaminestone , is there any update yet? or any kind of eta no matter how vague?