Importing CSV, converter separates fields at delimiter character inside text fields

[mupaneu]

Do you want to request a feature or report a bug? bug.

What is the current behavior? Importing CSV, the converter regards the delimiter character inside a text field with " " as a field delimiter.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.

1. Prepare a CSV with semicola as field delimiters, " " as text field delimiters, with a ; as part of a text field, between two ".
2. Open a new or existing Sheet (Spreadsheet) in Cryptpad. Select an empty cell with enough rows below. In the ribbon select Data > Get Data > From local TXT / CSV.
3. Import the CSV prepared in 1.
4. The importer will omit the text ; and interpret it to be a field separator.

cf. https://github.com/xwiki-labs/cryptpad/issues/944

What is the expected behavior? Inside " ", a field delimiter character is none. This can be dangerous, the converter destroys content data.

Did this work in previous versions of DocumentServer? Don't know.

DocumentServer version: Don't know, it's integrated in https://cryptpad.digitalcourage.de/.

Operating System: OS: Linux Mint

Browser version: Chromium Version: 103.0.5060.134 (Official Build) for Linux Mint (64-bit) Extensions installed: Privacy Redirect, Textarea Cache, uMatrix, Private Bookmarks Browser tweaks: incognito mode

[ShockwaveNN]

Don't know, it's integrated in https://cryptpad.digitalcourage.de/.

Hi, we already have several reports from cryptpad and can confirm that this is not our product, it's based on ONLYOFFICE but seems to have some modification (we don't know how many and which are) and we don't know which version it's using

So my advice is

1. Try to reproduce your issue on our official DocumentServer/DesktopEditors/personal.onlyoffice.com and if the same problems occur - we're gladly report it to our developer
2. If this problem happens only on cryptpad server and not on ONLYOFFICE products - it's better to contact cryptpad support

[ghost]

Hello!

Sorry to send our users to you. 😁 @mupaneu simply copy/pasted the issue they opened on our repository here but I personally checked on https://api.onlyoffice.com/editors/demopreview?type=spreadsheet#DemoPreview and can confirm the issue.

[ShockwaveNN]

@mthld Ok, thanks, we'll looked in that issue then

[mupaneu]

Hello!

Sorry to send our users to you. grin @mupaneu simply copy/pasted the issue they opened on our repository here but I personally checked on https://api.onlyoffice.com/editors/demopreview?type=spreadsheet#DemoPreview and can confirm the issue.

@mthld Well, I added a link to my ticket with you ...

[VladimirProkofiev52]

Hello! @mupaneu, @mthld! I can confirm - this is bug, issue 58552 in our private issue tracker. Thank you for report.