DocumentServer icon indicating copy to clipboard operation
DocumentServer copied to clipboard

OnlyOffice generates corrupted .ods file containing urls due to invalid content.xml

Open hubitor opened this issue 3 years ago • 8 comments

This issue is unique.

  • [X] I have used the search tool and did not find an issue describing my bug.

Operating System

Linux (DEB package)

Version information

7.1.0.215

Expected Behavior

OnlyOffice should handle spreadsheet files in .ods format containing urls without issues and generally generate a valid content.xml regardless of what characters the file contains.

Actual Behavior

I saved a file containing 10 rows with urls, some text and a SUM function as .ods. As I re-opened the file in OnlyOffice 8 rows were missing.

Then I tried to open the file in LibreOffice 7.4 and I got a popup message with this text:

Read Error: Format error discovered in the file in sub-document content.xml at 1,9325(row,col).

and as I clicked OK LibreOffice just closed.

Then I decompressed the file and opened the content.xml in VSC. There were 3 errors:

The reference to entity "l" must end with the ';' delimeter.

Based on this: ERROR: 'The reference to entity "L" must end with the ';' delimiter.' I removed the related ampersands (&), saved the content.xml and compressed the modified .ods file based on this: How does Open Office compress its files?

After that I was able to recover the missing rows (still have to replace the 3 affected urls).

Reproduction Steps

  1. Save a spreadsheet file in .ods format containing urls. E.g. https://www.example.com/blabla?k=&l=&display=10
  2. Close file.
  3. Try to reopen file.

Additional information

I use Arch Linux and the package is the "onlyoffice-bin" from AUR.

hubitor avatar Aug 31 '22 09:08 hubitor

Hi, to be clear - could you provide ods file we can check for this error?

And could you check for this error in flatpak\snap\appimage package? Since we do not provide official Arch AUR and it's made by someone else

ShockwaveNN avatar Aug 31 '22 10:08 ShockwaveNN

Hi, to be clear - could you provide ods file we can check for this error?

Hi, sure. Here you are: test.ods

The file should contain a single url. If you try to open it with OnlyOffice it will be empty. I haven't checked, but the link should be still contained within the content.xml. If you try to open it with LibreOffice a popup error message appears as I described before.

And could you check for this error in flatpak\snap\appimage package? Since we do not provide official Arch AUR and it's made by someone else

No. The AUR package just uses the .deb package you provide and it works fine. I don't want to install an additional package. Besides, I don't think that it has to do anything with this specific issue.

hubitor avatar Aug 31 '22 11:08 hubitor

Hi, sure. Here you are: test.ods

I think this is file after opening in ONLYOFFICE DesktopEditors, I've meant - original file before opening

just uses the .deb package you provide and it works fine

Fair enought

ShockwaveNN avatar Aug 31 '22 11:08 ShockwaveNN

I think this is file after opening in ONLYOFFICE DesktopEditors, I've meant - original file before opening

You can easily create such a file. I have already provided an example above. Just save a file in .ods format e.g. with this content:

https://www.example.com/blabla?k=&l=&display=10

hubitor avatar Aug 31 '22 11:08 hubitor

Oops, sorry, got it.

Able to reproduce it thanks

ShockwaveNN avatar Aug 31 '22 11:08 ShockwaveNN

Seems this problem already resolved in v7.2 which will be released withing a month

I can confirm issue is reproducable in v7.1.1 and is not in v7.2 so you'll need to wait for release

ShockwaveNN avatar Aug 31 '22 11:08 ShockwaveNN

Moving this issue to DocumentServer repo, since it's our base product and problem actual there too

ShockwaveNN avatar Aug 31 '22 11:08 ShockwaveNN

Nice. Thanks!

hubitor avatar Aug 31 '22 11:08 hubitor

DocumentServer v7.2 is released. This issue should be fixed

Feel free to comment or reopen it if you got further questions

ShockwaveNN avatar Sep 23 '22 16:09 ShockwaveNN